[src/trunk]: src/tests/usr.bin/mixerctl Correct oversight in previous ... red...

[kre <kre%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/385162669e59
branches:  trunk
changeset: 825623:385162669e59
user:      kre <kre%NetBSD.org@localhost>
date:      Tue Jul 25 21:25:03 2017 +0000

description:
Correct oversight in previous ... redirecting into a compound statement
causes the shell to exit if the redirect fails (posix says "may exit"
and /bin/sh does - maybe should give that more thought) - which will
happen if /dev/pad0 does not exist, causing a very messy test abort
(the shell running the test is not supposed to just go away).  So
check tha the device exista and is readable before attempting to open it.

Problem brought to my attention by nat@ - thanks...

diffstat:

 tests/usr.bin/mixerctl/t_mixerctl.sh |  5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)

diffs (19 lines):

diff -r 1a414d5ac80d -r 385162669e59 tests/usr.bin/mixerctl/t_mixerctl.sh
--- a/tests/usr.bin/mixerctl/t_mixerctl.sh      Tue Jul 25 19:44:25 2017 +0000
+++ b/tests/usr.bin/mixerctl/t_mixerctl.sh      Tue Jul 25 21:25:03 2017 +0000
@@ -1,4 +1,4 @@
-# $NetBSD: t_mixerctl.sh,v 1.8 2017/07/18 13:17:37 kre Exp $
+# $NetBSD: t_mixerctl.sh,v 1.9 2017/07/25 21:25:03 kre Exp $
 
 audio_setup() {
        # Open /dev/pad0 so we have a configured audio device.
@@ -21,7 +21,8 @@
        # variables, we would need to put $padpid in a file.)
 
        unset padpid
-       { { cat >/dev/null & } < /dev/pad0 ; } 2>/dev/null && padpid=$!
+       test -r /dev/pad0 && 
+           { { cat >/dev/null & } < /dev/pad0 ; } 2>/dev/null && padpid=$!
 
        (</dev/mixer) >/dev/null 2>&1 ||
            atf_skip "no audio mixer available in kernel"