tests: directly test for presence of command

Test directly for presence of command instead of indirectly testing
the return code.

Additionally:

* uses a shell builtin `command -v` instead of external `which`
* `command -v` is the standardized version of `which`

10 files changed, 35 insertions, 70 deletions

diff --git a/test/apps-x11-xorg/apps-x11-xorg.sh b/test/apps-x11-xorg/apps-x11-xorg.sh
index 9ed123979..9dcee7aff 100755
--- a/test/apps-x11-xorg/apps-x11-xorg.sh
+++ b/test/apps-x11-xorg/apps-x11-xorg.sh
@@ -7,8 +7,7 @@ export MALLOC_CHECK_=3
 export MALLOC_PERTURB_=$(($RANDOM % 255 + 1))
 export LC_ALL=C
 
-which firefox 2>/dev/null
-if [ "$?" -eq 0 ];
+if command -v firefox
 then
         echo "TESTING: firefox x11 xorg"
         ./firefox.exp
@@ -16,8 +15,7 @@ else
         echo "TESTING SKIP: firefox not found"
 fi
 
-which transmission-gtk 2>/dev/null
-if [ "$?" -eq 0 ];
+if command -v transmission-gtk
 then
         echo "TESTING: transmission-gtk x11 xorg"
         ./transmission-gtk.exp
@@ -25,8 +23,7 @@ else
         echo "TESTING SKIP: transmission-gtk not found"
 fi
 
-which transmission-qt 2>/dev/null
-if [ "$?" -eq 0 ];
+if command -v transmission-qt
 then
         echo "TESTING: transmission-qt x11 xorg"
         ./transmission-qt.exp
@@ -34,8 +31,7 @@ else
         echo "TESTING SKIP: transmission-qt not found"
 fi
 
-which thunderbird 2>/dev/null
-if [ "$?" -eq 0 ];
+if command -v thunderbird
 then
         echo "TESTING: thunderbird x11 xorg"
         ./thunderbird.exp
diff --git a/test/apps-x11/apps-x11.sh b/test/apps-x11/apps-x11.sh
index a3c946ca4..9f8102738 100755
--- a/test/apps-x11/apps-x11.sh
+++ b/test/apps-x11/apps-x11.sh
@@ -11,21 +11,18 @@ echo "TESTING: no x11 (test/apps-x11/x11-none.exp)"
 ./x11-none.exp
 
 
-which xterm 2>/dev/null
-if [ "$?" -eq 0 ];
+if command -v xterm
 then
         echo "TESTING: xterm x11 xorg"
         ./xterm-xorg.exp
 
-        which xpra 2>/dev/null
-        if [ "$?" -eq 0 ];
+        if command -v xpra
         then
         echo "TESTING: xterm x11 xpra"
         ./xterm-xpra.exp
         fi
 
-        which Xephyr 2>/dev/null
-        if [ "$?" -eq 0 ];
+        if command -v Xephyr
         then
         echo "TESTING: xterm x11 xephyr"
         ./xterm-xephyr.exp
@@ -35,14 +32,12 @@ else
 fi
 
 # check xpra/xephyr
-which xpra 2>/dev/null
-if [ "$?" -eq 0 ];
+if command -v xpra
 then
         echo "xpra found"
 else
         echo "xpra not found"
-        which Xephyr 2>/dev/null
-        if [ "$?" -eq 0 ];
+        if command -v Xephyr
         then
                 echo "Xephyr found"
         else
@@ -51,8 +46,7 @@ else
         fi
 fi
 
-which firefox 2>/dev/null
-if [ "$?" -eq 0 ];
+if command -v firefox
 then
         echo "TESTING: firefox x11"
         ./firefox.exp
@@ -60,8 +54,7 @@ else
         echo "TESTING SKIP: firefox not found"
 fi
 
-which chromium 2>/dev/null
-if [ "$?" -eq 0 ];
+if command -v chromium
 then
         echo "TESTING: chromium x11"
         ./chromium.exp
@@ -69,8 +62,7 @@ else
         echo "TESTING SKIP: chromium not found"
 fi
 
-which transmission-gtk 2>/dev/null
-if [ "$?" -eq 0 ];
+if command -v transmission-gtk
 then
         echo "TESTING: transmission-gtk x11"
         ./transmission-gtk.exp
@@ -78,8 +70,7 @@ else
         echo "TESTING SKIP: transmission-gtk not found"
 fi
 
-which thunderbird 2>/dev/null
-if [ "$?" -eq 0 ];
+if command -v thunderbird
 then
         echo "TESTING: thunderbird x11"
         ./thunderbird.exp
diff --git a/test/apps/apps.sh b/test/apps/apps.sh
index 83e977ba0..ed4b7604a 100755
--- a/test/apps/apps.sh
+++ b/test/apps/apps.sh
@@ -11,8 +11,7 @@ LIST="firefox midori chromium opera transmission-qt qbittorrent uget-gtk filezil
 LIST+="vlc fbreader deluge gnome-mplayer xchat wine kcalc ktorrent hexchat"
 
 for app in $LIST; do
-        which $app 2>/dev/null
-        if [ "$?" -eq 0 ];
+        if command -v "$app"
         then
                 echo "TESTING: $app"
                 ./$app.exp
diff --git a/test/environment/environment.sh b/test/environment/environment.sh
index da9c170b9..f6acada73 100755
--- a/test/environment/environment.sh
+++ b/test/environment/environment.sh
@@ -39,8 +39,7 @@ echo "TESTING: environment variables (test/environment/env.exp)"
 echo "TESTING: firejail in firejail - single sandbox (test/environment/firejail-in-firejail.exp)"
 ./firejail-in-firejail.exp
 
-which aplay 2>/dev/null
-if [ "$?" -eq 0 ] && [ "$(aplay -l | grep -c "List of PLAYBACK")" -gt 0 ];
+if command -v aplay && [ "$(aplay -l | grep -c "List of PLAYBACK")" -gt 0 ];
 then
         echo "TESTING: sound (test/environment/sound.exp)"
         ./sound.exp
@@ -54,8 +53,7 @@ echo "TESTING: nice (test/environment/nice.exp)"
 echo "TESTING: quiet (test/environment/quiet.exp)"
 ./quiet.exp
 
-which strace 2>/dev/null
-if [ "$?" -eq 0 ];
+if command -v strace
 then
         echo "TESTING: --allow-debuggers (test/environment/allow-debuggers.exp)"
         ./allow-debuggers.exp
@@ -67,8 +65,7 @@ fi
 #       $ sudo apt-get install ibus-table-array30
 #       $ ibus-setup
 
-find ~/.config/ibus/bus | grep unix-0
-if [ "$?" -eq 0 ];
+if find ~/.config/ibus/bus | grep unix-0
 then
         echo "TESTING: ibus (test/environment/ibus.exp)"
         ./ibus.exp
diff --git a/test/filters/filters.sh b/test/filters/filters.sh
index 04d7080d6..1c6b812ff 100755
--- a/test/filters/filters.sh
+++ b/test/filters/filters.sh
@@ -96,8 +96,7 @@ echo "TESTING: seccomp errno (test/filters/seccomp-errno.exp)"
 echo "TESTING: seccomp su (test/filters/seccomp-su.exp)"
 ./seccomp-su.exp
 
-which strace 2>/dev/null
-if [ $? -eq 0 ]; then
+if command -v strace; then
         echo "TESTING: seccomp ptrace (test/filters/seccomp-ptrace.exp)"
         ./seccomp-ptrace.exp
 else
diff --git a/test/overlay/overlay.sh b/test/overlay/overlay.sh
index 490b180e1..9bc458f50 100755
--- a/test/overlay/overlay.sh
+++ b/test/overlay/overlay.sh
@@ -22,8 +22,7 @@ rm -fr ~/_firejail_test_*
 ./fs-tmpfs.exp
 rm -fr ~/_firejail_test_*
 
-which firefox 2>/dev/null
-if [ "$?" -eq 0 ];
+if command -v firefox
 then
         echo "TESTING: overlay firefox"
         ./firefox.exp
@@ -31,8 +30,7 @@ else
         echo "TESTING SKIP: firefox not found"
 fi
 
-which firefox 2>/dev/null
-if [ "$?" -eq 0 ];
+if command -v firefox
 then
         echo "TESTING: overlay firefox x11 xorg"
         ./firefox.exp
@@ -42,14 +40,12 @@ fi
 
 
 # check xpra/xephyr
-which xpra 2>/dev/null
-if [ "$?" -eq 0 ];
+if command -v xpra
 then
         echo "xpra found"
 else
         echo "xpra not found"
-        which Xephyr 2>/dev/null
-        if [ "$?" -eq 0 ];
+        if command -v Xephyr
         then
                 echo "Xephyr found"
         else
@@ -58,8 +54,7 @@ else
         fi
 fi
 
-which firefox 2>/dev/null
-if [ "$?" -eq 0 ];
+if command -v firefox
 then
         echo "TESTING: overlay firefox x11"
         ./firefox-x11.exp
diff --git a/test/private-lib/private-lib.sh b/test/private-lib/private-lib.sh
index d168c2b1b..86ed7b06d 100755
--- a/test/private-lib/private-lib.sh
+++ b/test/private-lib/private-lib.sh
@@ -11,8 +11,7 @@ LIST="gnome-logs gnome-system-log gnome-nettool pavucontrol dig evince whois gal
 
 
 for app in $LIST; do
-        which $app 2>/dev/null
-        if [ "$?" -eq 0 ];
+        if command -v "$app"
         then
                 echo "TESTING: private-lib $app"
                 ./$app.exp
diff --git a/test/root/root.sh b/test/root/root.sh
index e8c0ec1ac..e3b48d15b 100755
--- a/test/root/root.sh
+++ b/test/root/root.sh
@@ -11,8 +11,7 @@ export LC_ALL=C
 #********************************
 # firecfg
 #********************************
-which less 2>/dev/null
-if [ "$?" -eq 0 ];
+if command -v less
 then
         echo "TESTING: firecfg (test/root/firecfg.exp)"
         mv /home/netblue/.local/share/applications /home/netblue/.local/share/applications-store
diff --git a/test/sysutils/sysutils.sh b/test/sysutils/sysutils.sh
index a903c7c6b..bfe723047 100755
--- a/test/sysutils/sysutils.sh
+++ b/test/sysutils/sysutils.sh
@@ -7,8 +7,7 @@ export MALLOC_CHECK_=3
 export MALLOC_PERTURB_=$(($RANDOM % 255 + 1))
 export LC_ALL=C
 
-which cpio 2>/dev/null
-if [ "$?" -eq 0 ];
+if command -v cpio
 then
         echo "TESTING: cpio"
         ./cpio.exp
@@ -16,8 +15,7 @@ else
         echo "TESTING SKIP: cpio not found"
 fi
 
-#which strings
-#if [ "$?" -eq 0 ];
+#if command -v strings
 #then
 #       echo "TESTING: strings"
 #       ./strings.exp
@@ -25,8 +23,7 @@ fi
 #       echo "TESTING SKIP: strings not found"
 #fi
 
-which gzip 2>/dev/null
-if [ "$?" -eq 0 ];
+if command -v gzip
 then
         echo "TESTING: gzip"
         ./gzip.exp
@@ -34,8 +31,7 @@ else
         echo "TESTING SKIP: gzip not found"
 fi
 
-which xzdec 2>/dev/null
-if [ "$?" -eq 0 ];
+if command -v xzdec
 then
         echo "TESTING: xzdec"
         ./xzdec.exp
@@ -43,8 +39,7 @@ else
         echo "TESTING SKIP: xzdec not found"
 fi
 
-which xz 2>/dev/null
-if [ "$?" -eq 0 ];
+if command -v xz
 then
         echo "TESTING: xz"
         ./xz.exp
@@ -52,8 +47,7 @@ else
         echo "TESTING SKIP: xz not found"
 fi
 
-which less 2>/dev/null
-if [ "$?" -eq 0 ];
+if command -v less
 then
         echo "TESTING: less"
         ./less.exp
@@ -61,8 +55,7 @@ else
         echo "TESTING SKIP: less not found"
 fi
 
-which file 2>/dev/null
-if [ "$?" -eq 0 ];
+if command -v file
 then
         echo "TESTING: file"
         ./file.exp
@@ -70,8 +63,7 @@ else
         echo "TESTING SKIP: file not found"
 fi
 
-which tar 2>/dev/null
-if [ "$?" -eq 0 ];
+if command -v tar
 then
         echo "TESTING: tar"
         ./tar.exp
@@ -79,8 +71,7 @@ else
         echo "TESTING SKIP: tar not found"
 fi
 
-which ping 2>/dev/null
-if [ "$?" -eq 0 ];
+if command -v ping
 then
         echo "TESTING: ping"
         ./ping.exp
diff --git a/test/utils/utils.sh b/test/utils/utils.sh
index 102c8df17..ad36d7053 100755
--- a/test/utils/utils.sh
+++ b/test/utils/utils.sh
@@ -33,8 +33,7 @@ echo "TESTING: version (test/utils/version.exp)"
 echo "TESTING: help (test/utils/help.exp)"
 ./help.exp
 
-which man 2>/dev/null
-if [ "$?" -eq 0 ];
+if command -v man
 then
         echo "TESTING: man (test/utils/man.exp)"
         ./man.exp