Page MenuHomeFreeBSD

tests/sys/netgraph/ng_macfilter_test: Fix invalid TAP output
ClosedPublic

Authored by arichardson on Mar 4 2021, 9:51 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Mar 7, 11:57 PM
Unknown Object (File)
Jan 30 2024, 2:10 AM
Unknown Object (File)
Jan 30 2024, 2:10 AM
Unknown Object (File)
Jan 13 2024, 8:36 PM
Unknown Object (File)
Dec 28 2023, 4:36 AM
Unknown Object (File)
Dec 20 2023, 3:00 AM
Unknown Object (File)
Dec 12 2023, 10:35 PM
Unknown Object (File)
Nov 20 2023, 6:12 PM

Details

Summary

This should allow the test to pass in Jenkins. Testing it locally now
reports "passed" instead of "invalid TAP data".

While touching this file also fix some shellcheck warnings that were
pointed out by my IDE.

Diff Detail

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

Event Timeline

_test_next () { TSTNR=$((TSTNR + 1)); }

is incorrect. It should be $(($TSTNR + 1)).

_test_next () { TSTNR=$((TSTNR + 1)); }

is incorrect. It should be $(($TSTNR + 1)).

It's not incorrect, just an unnecessary change. I made that change because shellcheck pointed it out to me (https://github.com/koalaman/shellcheck/wiki/SC2004).

  • Fix remaining SC2004 warnings
  • Fix uses of undefined variables (noticed by shellcheck)

ping? This test has been failing for the past 294 builds. I'd really like to get Jenkins back to green so we can catch real regressions.

This revision is now accepted and ready to land.Mar 11 2021, 3:02 PM
lwhsu added inline comments.
tests/sys/netgraph/ng_macfilter_test.sh
233

I feel this is easy to be forgot, but also not easy to change. The better approach might be migrating the test cases to use atf-sh.