From 507b3a02015b529d98c57ca8cd2c6e130a23b083 Mon Sep 17 00:00:00 2001
From: Assaf Gordon <assafgordon@gmail.com>
Date: Mon, 25 Apr 2016 22:14:33 -0400
Subject: [PATCH] tests: improve recent seq EPIPE test

Follow-up to commit c92585b1: improve epipe test script, with
suggestions from Bernhard Voelker.

* tests/misc/seq-epipe.sh: avoid too many subshells,
  stricter validation of errors and output from seq.
---
 tests/misc/seq-epipe.sh | 26 +++++++++-----------------
 1 file changed, 9 insertions(+), 17 deletions(-)

diff --git a/tests/misc/seq-epipe.sh b/tests/misc/seq-epipe.sh
index 21e9322..5c42c46 100755
--- a/tests/misc/seq-epipe.sh
+++ b/tests/misc/seq-epipe.sh
@@ -20,27 +20,19 @@
 print_ver_ seq
 
 # 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
-- 
1.9.1