]> git.ipfire.org Git - thirdparty/coreutils.git/commitdiff
tests: improve recent seq EPIPE test
authorAssaf Gordon <assafgordon@gmail.com>
Sun, 1 May 2016 19:43:59 +0000 (15:43 -0400)
committerAssaf Gordon <assafgordon@gmail.com>
Sat, 2 Jul 2016 22:08:33 +0000 (18:08 -0400)
Follow-up to commit c92585b1: improve epipe test script, with
suggestions from Bernhard Voelker and Pádraig Brady.
see: http://lists.gnu.org/archive/html/coreutils/2016-04/msg00067.html .

* tests/misc/seq-epipe.sh: Avoid too many subshells,
  stricter validation of errors and output from seq,
  skip if trapping SIGPIPE is not supported.

tests/misc/seq-epipe.sh

index 21e932293447212ca1a79f21ab166d6a40ce889f..9a43068bb34467635f976e3a4d2f1976291c5a91 100755 (executable)
 . "${srcdir=.}/tests/init.sh"; path_prepend_ ./src
 print_ver_ seq
 
+(trap '' PIPE && yes | :) 2>&1 | grep -qF 'Broken pipe' ||
+    skip_ 'trapping SIGPIPE is not supported'
+
 # upon EPIPE with signals ignored, 'seq' should exit with an error.
-(trap '' PIPE;
- timeout 10 sh -c '( (seq inf 2>err ; echo $?>code) | head -n1)'>/dev/null)
+timeout 10 sh -c \
+  'trap "" PIPE && { seq inf 2>err; echo $? >code; } | head -n1' >out
 
 # Exit-code must be 1, indicating 'write error'
-cat << \EOF > exp || framework_failure_
-1
-EOF
-if test -e code ; then
-  compare exp code || fail=1
-else
-  # 'exitcode' file was not created
-  warn_ "missing exit code file (seq failed to detect EPIPE?)"
-  fail=1
-fi
+echo 1 > exp || framework_failure_
+compare exp out || fail=1
+compare exp code || fail=1
 
 # The error message must begin with "standard output:"
 # (but don't hard-code the strerror text)
-compare_dev_null_ /dev/null err
-if ! grep -qE '^seq: standard output: .+$' err ; then
-  warn_ "seq emitted incorrect error on EPIPE"
-  fail=1
-fi
+grep '^seq: standard output: ' err \
+  || { warn_ "seq emitted incorrect error on EPIPE"; \
+       cat err;\
+       fail=1; }
 
 Exit $fail