grub-devel.gnu.org archive mirror
 help / color / mirror / Atom feed
From: Leo Sandoval via Grub-devel <grub-devel@gnu.org>
To: grub-devel@gnu.org
Cc: Leo Sandoval <lsandova@redhat.com>
Subject: [PATCH 0/3] tests: improvements and fixes for the test suite
Date: Fri, 19 Sep 2025 15:45:02 -0600	[thread overview]
Message-ID: <20250919214516.1693171-1-lsandova@redhat.com> (raw)

With the goal to have a all-pass 'make check', these patches improve a bit
tests, increasing bash verbosity and skipping those where tools are not available.
Finally it removes the -w parameter on the mkfs.hfsplus.

Taking into account Andrew's series [1] and this series, I am still seeing
1 failure and 3 errors as shown below. So this work is no over but exposing
these may lead to further fixes.

Finally, all these tests were done on a Fedora 42 VM. If containers are used
like those at [2], I am seeing a bunch of errors due to the lack of the
/dev/loop0 device but I will update the corresponding thread.

[1] https://lists.nongnu.org/archive/html/grub-devel/2025-08/msg00249.html
[2] https://lists.gnu.org/archive/html/grub-devel/2025-06/msg00139.html

    =================================
    GRUB 2.13: ./test-suite.log
    =================================

    # TOTAL: 89
    # PASS:  80
    # SKIP:  5
    # XFAIL: 0
    # FAIL:  1
    # XPASS: 0
    # ERROR: 3


    FAIL: zfs_test
    ==============

    + '[' x0 = x ']'
    + '[' 0 '!=' 0 ']'
    + which zpool
    + ./grub-fs-tester zfs
    Defaulting to 4K blocksize (ashift=12) for '/dev/loop0'
    1024+0 records in
    1024+0 records out
    1048576 bytes (1.0 MB, 1.0 MiB) copied, 0.0128335 s, 81.7 MB/s
    Device proc: Filesystem type procfs - Sector size 512B - Total size 0KiB
    Device loop0: Filesystem type zfs - Label `grub_testieufiue r' - Last modification time 2025-09-19 19:30:17 Friday, UUID aeaca1e49e9167be - Sector size 512B - Total size 5939200KiB
    Device host: Filesystem type hostfs - Sector size 512B - Total size 0KiB
    TIME FAIL
    DIR          20250919193013 sdir/
    DIR          20250919193013 0/
    5242879      20250919193013 1.img
    5242879      20250919193013 .?*\!"#%@$%&'()+ ,-.:;<=>^{_}[]`|~.
    5242879      20250919193013 éàèüöäëñкирилица莭莽茝Ελληνικά 
    5242879      20250919193013 qwertzuiopasdfghjklyxcvbnm1234567890qwertzuiopasdfghjklyxcvbnm1234567890oiewqfiewioqoiqoiurqruewqoiuwoieoiiuewqroreqiufieiuwrnureweriuvceoiroiewqoiricdsalkcndsakfirefoiwqeoircorejwoij\
    freoijojoiewjfwnfcoirenfoirefnreoifenoiwfnoi
    5242879      20250919193013 CaSe
    5242879      20250919193013 cAsE
    1048576      20250919193013 american-english
    5242879      20250919193013 sym
    ???????????? 20250919193013 longsym
    ???????????? 20250919193013 psym
    5242879      20250919193013 Ελληνικάкирилица éàèüöäëñ莭莽茝
    5242879      20250919193013 hard
    total 25
    drwxr-xr-x. 4 root root 16 Sep 19 19:30 grub fs
    NAME                 SIZE  ALLOC   FREE    CAP  DEDUP  HEALTH  ALTROOT
    grub_testieufiue r  5.66G  43.8M  5.61G     0%  1.00x  ONLINE  /tmp/grub-fs-tester.20250919213012734622369.zfs.ztl/zfs_ro
    FAIL zfs_test (exit status: 1)


    ERROR: nilfs2_test
    ==================

    + '[' x0 = x ']'
    + '[' 0 '!=' 0 ']'
    + which mkfs.nilfs2
    + ./grub-fs-tester nilfs2
    mount.nilfs2: preparing failed: No such device
    MOUNT FAILED.
    ERROR nilfs2_test (exit status: 99)

    ERROR: exfat_test
    =================

    + '[' x0 = x ']'
    + '[' 0 '!=' 0 ']'
    + which mkfs.exfat
    + ./grub-fs-tester exfat
    exfatprogs version : 1.2.8
    Creating exFAT filesystem(/dev/loop0, cluster size=512)

    Writing volume boot record: done
    Writing backup volume boot record: done
    Fat table creation: done
    Allocation bitmap creation: done
    Upcase table creation: done
    Writing root directory entry: done
    Synchronizing...

    exFAT format complete!
    mount: /tmp/grub-fs-tester.20250919212812088905104.exfat.WLh/exfat_rw: unknown filesystem type 'exfat-fuse'.
           dmesg(1) may have more information after failed mount system call.
    MOUNT FAILED.
    ERROR exfat_test (exit status: 99)

    ERROR: zfs_zstd_test
    ====================

    zfs zstd compression not supported; cannot test zfs zstd.
    ERROR zfs_zstd_test (exit status: 99)


Leo Sandoval (3):
  tests: skip tests if required tools are not available
  tests: include verbosity on tests.in checks
  tests: remove -w param from mkfs.hfsplus

 tests/ahci_test.in                   | 2 +-
 tests/asn1_test.in                   | 2 +-
 tests/btrfs_test.in                  | 4 ++--
 tests/cdboot_test.in                 | 2 +-
 tests/core_compress_test.in          | 2 +-
 tests/cpio_test.in                   | 4 ++--
 tests/ehci_test.in                   | 2 +-
 tests/erofs_test.in                  | 4 ++--
 tests/example_scripted_test.in       | 2 +-
 tests/exfat_test.in                  | 4 ++--
 tests/ext234_test.in                 | 8 ++++----
 tests/f2fs_test.in                   | 4 ++--
 tests/fat_test.in                    | 4 ++--
 tests/fddboot_test.in                | 2 +-
 tests/file_filter_test.in            | 2 +-
 tests/grub_cmd_cryptomount.in        | 4 ++--
 tests/grub_cmd_date.in               | 2 +-
 tests/grub_cmd_regexp.in             | 2 +-
 tests/grub_cmd_set_date.in           | 2 +-
 tests/grub_cmd_sleep.in              | 2 +-
 tests/grub_cmd_test.in               | 2 +-
 tests/grub_func_test.in              | 2 +-
 tests/grub_script_blanklines.in      | 2 +-
 tests/grub_script_blockarg.in        | 2 +-
 tests/grub_script_dollar.in          | 2 +-
 tests/grub_script_expansion.in       | 2 +-
 tests/grub_script_final_semicolon.in | 2 +-
 tests/grub_script_no_commands.in     | 2 +-
 tests/gzcompress_test.in             | 4 ++--
 tests/hddboot_test.in                | 2 +-
 tests/help_test.in                   | 2 +-
 tests/hfs_test.in                    | 4 ++--
 tests/hfsplus_test.in                | 4 ++--
 tests/iso9660_test.in                | 4 ++--
 tests/jfs_test.in                    | 4 ++--
 tests/luks1_test.in                  | 6 +++---
 tests/luks2_test.in                  | 6 +++---
 tests/lzocompress_test.in            | 4 ++--
 tests/minixfs_test.in                | 4 ++--
 tests/netboot_test.in                | 2 +-
 tests/nilfs2_test.in                 | 4 ++--
 tests/ntfs_test.in                   | 6 +++---
 tests/ohci_test.in                   | 2 +-
 tests/partmap_test.in                | 4 ++--
 tests/pata_test.in                   | 2 +-
 tests/pseries_test.in                | 2 +-
 tests/reiserfs_test.in               | 4 ++--
 tests/romfs_test.in                  | 4 ++--
 tests/serial_test.in                 | 2 +-
 tests/squashfs_test.in               | 4 ++--
 tests/syslinux_test.in               | 2 +-
 tests/tar_test.in                    | 4 ++--
 tests/test_sha512sum.in              | 2 +-
 tests/tpm2_key_protector_test.in     | 4 ++--
 tests/udf_test.in                    | 4 ++--
 tests/uhci_test.in                   | 2 +-
 tests/util/grub-fs-tester.in         | 2 +-
 tests/xfs_test.in                    | 4 ++--
 tests/xzcompress_test.in             | 4 ++--
 tests/zfs_test.in                    | 4 ++--
 60 files changed, 94 insertions(+), 94 deletions(-)

-- 
2.50.1


_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel

             reply	other threads:[~2025-09-19 21:46 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-09-19 21:45 Leo Sandoval via Grub-devel [this message]
2025-09-19 21:45 ` [PATCH 1/3] tests: skip tests if required tools are not available Leo Sandoval via Grub-devel
2025-11-09  4:28   ` Glenn Washburn
2025-09-19 21:45 ` [PATCH 2/3] tests: include verbosity on tests.in checks Leo Sandoval via Grub-devel
2025-11-09  4:47   ` Glenn Washburn
2025-09-19 21:45 ` [PATCH 3/3] tests: remove -w param from mkfs.hfsplus Leo Sandoval via Grub-devel
2025-11-09  3:34   ` Glenn Washburn
2025-11-09  3:44     ` Glenn Washburn
2025-11-05 19:25 ` [PATCH 0/3] tests: improvements and fixes for the test suite Daniel Kiper

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20250919214516.1693171-1-lsandova@redhat.com \
    --to=grub-devel@gnu.org \
    --cc=lsandova@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).