Page MenuHomeFreeBSD

pf tests: Improve assertion messaging of mbuf tests
ClosedPublic

Authored by igoro on Sun, Aug 3, 2:38 PM.
Tags
None
Referenced Files
F127086201: D51716.id160907.diff
Wed, Aug 27, 11:55 AM
F127017537: D51716.diff
Tue, Aug 26, 3:32 PM
Unknown Object (File)
Sun, Aug 24, 7:49 PM
Unknown Object (File)
Sun, Aug 24, 9:43 AM
Unknown Object (File)
Wed, Aug 20, 1:54 PM
Unknown Object (File)
Sat, Aug 16, 11:43 AM
Unknown Object (File)
Thu, Aug 14, 4:53 AM
Unknown Object (File)
Sat, Aug 9, 3:23 AM

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 65942
Build 62825: arc lint + arc unit

Event Timeline

igoro requested review of this revision.Sun, Aug 3, 2:38 PM

A minuscule improvement to get the following upon atf_check_equal failure:

Error Message:
1 != $(jexec alcatraz sysctl -n net.dummymbuf.hits) (1 != 2)

instead of this:

Error Message:
1 != 2 (1 != 2)

It's still non-ideal for these specific test cases, but reveals a best practice if any copy-pasting is needed in the future.

Wanted to do it like a year ago. I'm thinking to improve the atf manual page to make it clear (if possible), right now it presents it with double quotes which might end up being used such way. But if we check the code behind then it's obvious that shell expansion was intended to be left for later.

This revision is now accepted and ready to land.Mon, Aug 4, 6:30 AM