diff options
author | a1346054 <36859588+a1346054@users.noreply.github.com> | 2022-09-13 18:42:15 +0000 |
---|---|---|
committer | a1346054 <36859588+a1346054@users.noreply.github.com> | 2022-09-13 22:37:18 +0000 |
commit | e04d6b7bf58a95105a8fc79fcde7c0e98875e72a (patch) | |
tree | 6e20913ed5d4af3e1b0ef5a31ef80ee42f651376 /test/root | |
parent | tests: fix unintentional overwriting of array value (diff) | |
download | firejail-e04d6b7bf58a95105a8fc79fcde7c0e98875e72a.tar.gz firejail-e04d6b7bf58a95105a8fc79fcde7c0e98875e72a.tar.zst firejail-e04d6b7bf58a95105a8fc79fcde7c0e98875e72a.zip |
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`
Diffstat (limited to 'test/root')
-rwxr-xr-x | test/root/root.sh | 3 |
1 files changed, 1 insertions, 2 deletions
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 | |||
11 | #******************************** | 11 | #******************************** |
12 | # firecfg | 12 | # firecfg |
13 | #******************************** | 13 | #******************************** |
14 | which less 2>/dev/null | 14 | if command -v less |
15 | if [ "$?" -eq 0 ]; | ||
16 | then | 15 | then |
17 | echo "TESTING: firecfg (test/root/firecfg.exp)" | 16 | echo "TESTING: firecfg (test/root/firecfg.exp)" |
18 | mv /home/netblue/.local/share/applications /home/netblue/.local/share/applications-store | 17 | mv /home/netblue/.local/share/applications /home/netblue/.local/share/applications-store |