diff options
author | Jim Meyering <meyering@fb.com> | 2016-11-08 19:57:41 -0600 |
---|---|---|
committer | Jim Meyering <meyering@fb.com> | 2016-11-13 08:26:19 -0800 |
commit | 22063c8c083d2d8130efb76482e45fa1976a595a (patch) | |
tree | 18645c547557304f18702e3c07b83fb3793ea135 /tests/misc/printenv.sh | |
parent | 4d45722d1160a04e4d26fd812da4e0140ad3550c (diff) | |
download | coreutils-22063c8c083d2d8130efb76482e45fa1976a595a.tar.xz |
tests: use "returns_" rather than explicit comparison with "$?"
* tests/chmod/no-x.sh: Likewise.
* tests/cp/no-deref-link1.sh: Likewise.
* tests/cp/no-deref-link2.sh: Likewise.
* tests/cp/no-deref-link3.sh: Likewise.
* tests/du/move-dir-while-traversing.sh: Likewise.
* tests/ls/infloop.sh: Likewise.
* tests/ls/stat-failed.sh: Likewise.
* tests/ls/time-style-diag.sh: Likewise.
* tests/misc/chroot-fail.sh: Likewise.
* tests/misc/env-null.sh: Likewise.
* tests/misc/env.sh: Likewise.
* tests/misc/nice-fail.sh: Likewise.
* tests/misc/printenv.sh: Likewise.
* tests/misc/timeout.sh: Likewise.
* tests/tail-2/follow-name.sh: Likewise.
Diffstat (limited to 'tests/misc/printenv.sh')
-rwxr-xr-x | tests/misc/printenv.sh | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/tests/misc/printenv.sh b/tests/misc/printenv.sh index 5df4146c8..2017e5df6 100755 --- a/tests/misc/printenv.sh +++ b/tests/misc/printenv.sh @@ -39,8 +39,7 @@ if env -- printenv | grep '^ENV_TEST' >/dev/null ; then fi # Printing a single variable's value. -env -- printenv ENV_TEST > out -test $? = 1 || fail=1 +returns_ 1 env -- printenv ENV_TEST > out || fail=1 compare /dev/null out || fail=1 echo a > exp || framework_failure_ ENV_TEST=a env -- printenv ENV_TEST > out || fail=1 @@ -60,10 +59,8 @@ EOF compare exp out || fail=1 # Exit status reflects missing variable, but remaining arguments processed. -ENV_TEST1=a env -- printenv ENV_TEST2 ENV_TEST1 > out -test $? = 1 || fail=1 -ENV_TEST1=a env -- printenv ENV_TEST1 ENV_TEST2 >> out -test $? = 1 || fail=1 +ENV_TEST1=a returns_ 1 env -- printenv ENV_TEST2 ENV_TEST1 > out || fail=1 +ENV_TEST1=a returns_ 1 env -- printenv ENV_TEST1 ENV_TEST2 >> out || fail=1 cat <<EOF > exp || framework_failure_ a a @@ -78,8 +75,7 @@ compare exp out || fail=1 # Silently reject invalid env-var names. # Bug present through coreutils 8.0. -env a=b=c printenv a=b > out -test $? = 1 || fail=1 +returns_ 1 env a=b=c printenv a=b > out || fail=1 compare /dev/null out || fail=1 Exit $fail |