mirror of
https://github.com/zeek/zeek.git
synced 2025-10-06 08:38:20 +00:00
Changing some external testing scripts.
- The absolute path canonifier was overzealously canonifying relevant log fields, so it's no longer generally applied to diffing all baselines. I don't think there's any logs that require local filesystem path names that aren't already tested by a unit test, but if any show up in the future, they can be canonified on a case-by-case basis. - Removed some logs from being diff'd in the diff-all script because they're either already covered by a unit test (load_scripts.log) or because of difficulty/maintainenance tradeoff (prof.log). Baselines for the external bro-testing repo still need updating.
This commit is contained in:
parent
870bdf796d
commit
357341c887
2 changed files with 8 additions and 7 deletions
2
testing/external/scripts/diff-all
vendored
2
testing/external/scripts/diff-all
vendored
|
@ -15,6 +15,7 @@ fi
|
|||
rc=0;
|
||||
|
||||
for i in $@; do
|
||||
if [[ "$i" != "loaded_scripts.log" && "$i" != "prof.log" ]]; then
|
||||
if ! btest-diff $i; then
|
||||
echo "" >>$diag
|
||||
echo "#### btest-diff $i" >>$diag
|
||||
|
@ -22,6 +23,7 @@ for i in $@; do
|
|||
cat $diag.tmp >>$diag
|
||||
rc=1
|
||||
fi
|
||||
fi
|
||||
done
|
||||
|
||||
exit $rc
|
||||
|
|
|
@ -5,4 +5,3 @@
|
|||
`dirname $0`/diff-remove-timestamps \
|
||||
| `dirname $0`/diff-remove-uids \
|
||||
| `dirname $0`/diff-remove-mime-types \
|
||||
| `dirname $0`/diff-remove-abspath
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue