Differential D55221

mixer/tests: use require.kmods property instead of ad-hoc checks
ClosedPublic
Actions

Authored by siva on Feb 10 2026, 3:07 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Mar 13, 7:18 AM

	Unknown Object (File)
	Wed, Mar 11, 10:22 AM

	Unknown Object (File)
	Wed, Mar 11, 8:14 AM

	Unknown Object (File)
	Wed, Mar 11, 7:23 AM

	Unknown Object (File)
	Wed, Mar 11, 4:23 AM

	Unknown Object (File)
	Mon, Mar 9, 12:57 PM

	Unknown Object (File)
	Sun, Mar 8, 3:15 AM

	Unknown Object (File)
	Feb 23 2026, 1:33 AM

View All 20 Files

Subscribers

Details

Reviewers

christos
lwhsu

Commits

rG3bd7deffdda7: mixer/tests: use require.kmods property instead of ad-hoc checks
rG5fa297f4bcd2: mixer/tests: use require.kmods property instead of ad-hoc checks

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

siva created this revision.Feb 10 2026, 3:07 PM

Herald added a subscriber: imp. · View Herald TranscriptFeb 10 2026, 3:07 PM

siva requested review of this revision.Feb 10 2026, 3:07 PM

Harbormaster completed remote builds in B70581: Diff 171649.Feb 10 2026, 3:07 PM

lwhsu accepted this revision.Feb 10 2026, 7:40 PM

This revision is now accepted and ready to land.Feb 10 2026, 7:40 PM

christos added inline comments.Feb 10 2026, 10:05 PM

usr.sbin/mixer/tests/Makefile
5	Isn't it cleaner for this to be inside the test script?

siva added inline comments.Feb 10 2026, 10:22 PM

usr.sbin/mixer/tests/Makefile
5	It's a bit simpler to add it here just to avoid so much duplication, because there are exactly 5 tests and all of them need this. Currently, it is equivalent.

christos accepted this revision.Feb 10 2026, 11:02 PM

Closed by commit rG5fa297f4bcd2: mixer/tests: use require.kmods property instead of ad-hoc checks (authored by siva). · Explain WhyFeb 12 2026, 3:04 PM

This revision was automatically updated to reflect the committed changes.

siva added a commit: rG5fa297f4bcd2: mixer/tests: use require.kmods property instead of ad-hoc checks.

christos added a commit: rG3bd7deffdda7: mixer/tests: use require.kmods property instead of ad-hoc checks.Feb 17 2026, 12:52 PM

Revision Contents
Changeset List

Path

Size

usr.sbin/

mixer/

tests/

3 lines

12 lines

Diff 171828

usr.sbin/mixer/tests/Makefile

Loading...

usr.sbin/mixer/tests/mixer_test.sh

Loading...