tests: avoid spurious failures on older shells
authorPádraig Brady <P@draigBrady.com>
Wed, 13 Jan 2010 12:19:26 +0000 (12:19 +0000)
committerJim Meyering <meyering@redhat.com>
Wed, 13 Jan 2010 12:42:49 +0000 (13:42 +0100)
* tests/tail-2/inotify-hash-abuse: Use kill rather than wait
to determine if the tail process is still running.
* tests/tail-2/inotify-hash-abuse2: Ditto.

tests/tail-2/inotify-hash-abuse
tests/tail-2/inotify-hash-abuse2

index ab5a69ecd188a08453cf5ae09b58ae880372278f..65a341211aab78be97e00795b2779ed1ddd2c18d 100755 (executable)
@@ -46,23 +46,14 @@ echo a > 1 || fail=1
 
 # Wait up to 2s for the buggy tail to die,
 # or for the "tail: `1' has appeared;  following end of new file" output
-dead=0
 for i in $(seq 10); do
-  kill -0 $pid || { dead=1; break; }
+  kill -0 $pid || break
   grep 'has appeared;' out > /dev/null && break
   sleep .2
 done
 
-# Fixed tail will not have aborted.  Kill it.
-test $dead = 0 && kill -HUP $pid
-
-wait $pid
-st=$?
-
-case $st in
-  129) ;;
-  *) echo tail died via unexpected signal: $st; fail=1;;
-esac
+# Kill the working tail, or fail if it has already aborted
+kill $pid || fail=1
 
 cat out
 
index 4fe080bd2d4d7e816bb1d4392cc60992c62ad250..2e567c799f133e6fc9c6d89cc307c07f9e76be60 100755 (executable)
@@ -36,15 +36,7 @@ for i in $(seq 200); do
   touch f
 done
 
-# Fixed tail will not have aborted.  Kill it.
-kill -HUP $pid
-
-wait $pid
-st=$?
-
-case $st in
-  129) ;;
-  *) echo tail died via unexpected signal: $st; fail=1;;
-esac
+# Kill the working tail, or fail if it has already aborted
+kill $pid || fail=1
 
 Exit $fail