The tests correctly skip if no snd_dummy neither mixer is found.
But the cleanup is still called with the skip condition and this one by calling mixer fails.
Details
Details
- Reviewers
christos - Group Reviewers
tests - Commits
- rGb8aeb7101724: mixer(8) tests: Fix cleanup routine
rG080c85127e3f: mixer(8) tests: Fix cleanup routine
Before:
root@host:/usr/tests # mv /boot/kernel/snd_dummy.ko /boot/kernel/snd_dummy.ko.bak root@host:/usr/tests # kyua test usr.sbin/mixer usr.sbin/mixer/mixer_test:d_flag -> broken: Test case cleanup did not terminate successfully [0.014s] usr.sbin/mixer/mixer_test:o_flag -> broken: Test case cleanup did not terminate successfully [0.015s] usr.sbin/mixer/mixer_test:mute -> skipped: cannot load snd_dummy.ko [0.014s] usr.sbin/mixer/mixer_test:recsrc -> skipped: cannot load snd_dummy.ko [0.015s] usr.sbin/mixer/mixer_test:volume -> skipped: cannot load snd_dummy.ko [0.015s] Results file id is usr_tests.20240830-160704-611764 Results saved to /root/.kyua/store/results.usr_tests.20240830-160704-611764.db 3/5 passed (2 failed)
After:
root@host:/usr/tests # kyua test usr.sbin/mixer usr.sbin/mixer/mixer_test:d_flag -> skipped: cannot load snd_dummy.ko [0.015s] usr.sbin/mixer/mixer_test:mute -> skipped: cannot load snd_dummy.ko [0.015s] usr.sbin/mixer/mixer_test:o_flag -> skipped: cannot load snd_dummy.ko [0.015s] usr.sbin/mixer/mixer_test:recsrc -> skipped: cannot load snd_dummy.ko [0.016s] usr.sbin/mixer/mixer_test:volume -> skipped: cannot load snd_dummy.ko [0.015s] Results file id is usr_tests.20240830-161822-481902 Results saved to /root/.kyua/store/results.usr_tests.20240830-161822-481902.db 5/5 passed (0 failed)
Diff Detail
Diff Detail
- Lint
Lint Skipped - Unit
Tests Skipped
Event Timeline
Comment Actions
LGTM. I will commit the patch and add you as the author, as it needs some style(9) fix as well. Thanks!