Page MenuHomeFreeBSD

mac_bsdextended: ATFify the tests
ClosedPublic

Authored by asomers on Apr 2 2020, 4:45 AM.
Tags
None
Referenced Files
F106275341: D24257.diff
Sat, Dec 28, 9:10 AM
Unknown Object (File)
Fri, Dec 27, 5:37 AM
Unknown Object (File)
Tue, Dec 24, 4:00 AM
Unknown Object (File)
Thu, Dec 19, 3:44 PM
Unknown Object (File)
Sat, Nov 30, 12:03 AM
Unknown Object (File)
Nov 19 2024, 7:10 PM
Unknown Object (File)
Nov 17 2024, 6:56 AM
Unknown Object (File)
Nov 14 2024, 10:17 PM
Subscribers

Details

Summary

mac_bsdextended: ATFify the tests

This is a precondition to fixing bug 244229

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This is a precondition to fixing bug 244229

I understand why this was a precondition to fixing bug 244229 after reading the bug, but without that context, I wouldn't know why. Could you please clarify that in the commit message?

tests/sys/mac/bsdextended/Makefile
6 ↗(On Diff #70107)

nit: why not sort matches_test above ugidfw_test, since ATF < TAP, sorting-wise?

tests/sys/mac/bsdextended/matches_test.sh
70 ↗(On Diff #70107)

Why isn't this filename parameterized at the global level?

316 ↗(On Diff #70107)

Why not set this property in the Makefile?

375 ↗(On Diff #70107)

Wouldn't it be a good idea here (and elsewhere) to check the return code from ugidfw, etc?

410 ↗(On Diff #70107)

Why not use atf_check with an inverted exit code check?

tests/sys/mac/bsdextended/matches_test.sh
14 ↗(On Diff #70107)

Where is this called in the new code?

asomers added inline comments.
tests/sys/mac/bsdextended/matches_test.sh
70 ↗(On Diff #70107)

Because it's the result of mktemp. Though, now that we're using ATF, it doesn't have to be. I'll change it.

316 ↗(On Diff #70107)

I generally prefer to set those things in the test file because it's more granular. It's done per testcase here, whereas it must be per test program if done in the Makefile. Do you think I should change it?

375 ↗(On Diff #70107)

Yeah, probably.

410 ↗(On Diff #70107)

Sure. It will remove the failure descriptions, but those aren't very useful anyway.

asomers marked an inline comment as done.

mac_bsdextended: ATFify the tests

The new tests have more complete setup and cleanup, are more granular, and
correctly annotate expected failures and skipped tests. A follow-up commit
will resolve a conflict with the fusefs tests (bug 244229)

ping @ngie @lwhsu . Are you satisfied with the latest changes?

This revision was not accepted when it landed; it landed in state Needs Review.Apr 26 2020, 3:51 PM
This revision was automatically updated to reflect the committed changes.