diff options
author | Bernhard Voelker <mail@bernhard-voelker.de> | 2015-01-14 17:38:12 +0100 |
---|---|---|
committer | Bernhard Voelker <mail@bernhard-voelker.de> | 2015-01-14 17:38:12 +0100 |
commit | e84046dbe6dd26bfcea7edf6fb419bea274635ab (patch) | |
tree | 50160bef015be6a261b0986762e23a449657fe0b /tests/tail-2/inotify-race.sh | |
parent | 58cff8a009ed9b8280c5f35074cef97231286023 (diff) | |
download | coreutils-e84046dbe6dd26bfcea7edf6fb419bea274635ab.tar.xz |
tests: use compare-vs-/dev/null instead of 'test -s'
When some program produces unexpected output, that use of
compare-vs-/dev/null will ensure that the surprising output is
printed in the test's output. With "test -s err" only, one
would have to instrument and rerun in order to see the offending
output.
* cfg.mk (sc_prohibit_and_fail_1): Exempt 'compare' from this check.
* tests/dd/misc.sh: Change "tests -s ... || fail=1" to
"compare /dev/null ... && fail=1".
* tests/misc/nice.sh: Likewise.
* tests/rm/read-only.sh: Likewise.
* tests/tail-2/inotify-race.sh: Likewise.
* tests/touch/no-dereference.sh: Likewise.
Suggested by Jim Meyering in
http://lists.gnu.org/archive/html/coreutils/2015-01/msg00042.html
Diffstat (limited to 'tests/tail-2/inotify-race.sh')
-rwxr-xr-x | tests/tail-2/inotify-race.sh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/tail-2/inotify-race.sh b/tests/tail-2/inotify-race.sh index 576a65218..2b1655c48 100755 --- a/tests/tail-2/inotify-race.sh +++ b/tests/tail-2/inotify-race.sh @@ -68,6 +68,6 @@ pid=$! tail --pid=$pid -f tail.out | (read; kill $pid) -test -s tail.out || fail=1 +compare /dev/null tail.out && fail=1 Exit $fail |