Page MenuHomeFreeBSD

makefs: tests: Fix timestamp-related tests
ClosedPublic

Authored by jlduran on Dec 31 2025, 1:40 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Feb 10, 8:06 AM
Unknown Object (File)
Sun, Feb 1, 9:21 PM
Unknown Object (File)
Jan 29 2026, 1:13 PM
Unknown Object (File)
Jan 25 2026, 11:29 PM
Unknown Object (File)
Jan 16 2026, 9:16 AM
Unknown Object (File)
Jan 15 2026, 4:22 PM
Unknown Object (File)
Jan 13 2026, 1:29 AM
Unknown Object (File)
Jan 12 2026, 1:06 PM
Subscribers

Details

Summary

Use a variable for mtree that includes the DEFAULT_MTREE_KEYWORDS, in
order to avoid missing the important "time" keyword.

Unskip the "failing" tests, and comment out the specific failing checks,
these will be addressed (or at least discussed) in a different revision.

For MS-DOS tests, use an even value timestamp, as precision is 2s.

For ZFS tests, use import_image instead of mount_image consistently.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

markj added inline comments.
usr.sbin/makefs/tests/makefs_zfs_tests.sh
964

These options are all redundant, i.e., atf_check will verify them by default. Generally I omit them on purpose to make test code a bit neater. But I guess it's better to be consistent with other tests of the same type.

This revision is now accepted and ready to land.Jan 1 2026, 3:19 PM
bnovkov added inline comments.
usr.sbin/makefs/tests/makefs_cd9660_tests.sh
40

I would consider moving the MTREE variable definition to makefs_tests_common.sh since all its contents are the same in all three scripts, but I won't insist.

jlduran added inline comments.
usr.sbin/makefs/tests/makefs_cd9660_tests.sh
40

Yes! It will be addressed upon commit.

usr.sbin/makefs/tests/makefs_zfs_tests.sh
964

I agree, and I'll try to address this suggestion in D54427; however, there is a high probability it may not be accepted.

This revision was automatically updated to reflect the committed changes.
jlduran marked an inline comment as done.