test/notmuch-test: simple streamlining
authorTomi Ollila <tomi.ollila@iki.fi>
Thu, 9 Jun 2016 19:20:56 +0000 (22:20 +0300)
committerDavid Bremner <david@tethera.net>
Tue, 28 Jun 2016 07:23:39 +0000 (09:23 +0200)
A few simple things that applies well to test/notmuch-test.sh

- Shell does pathname expansion also without doing `echo ...` in subshell.

- Redirections >/dev/null 2>/dev/null hide (improbable but) potential
  serious errors; adding -f to rm instead.

- Inter-file capitalization consistency in comments.

- Unnecesary space removal.

test/notmuch-test

index 988dce6c59cb5533d4a703fd52748f030c6b0d41..e7d3151ce8cbb528c68d6526ba4c39a035d5a1eb 100755 (executable)
@@ -17,14 +17,14 @@ fi
 
 cd "$(dirname "$0")"
 
-TESTS=${NOTMUCH_TESTS:-`echo T[0-9][0-9][0-9]-*.sh`}
+TESTS=${NOTMUCH_TESTS:-T[0-9][0-9][0-9]-*.sh}
 
 # Clean up any results from a previous run
-rm -r test-results >/dev/null 2>/dev/null
+rm -rf test-results
 
-# test for timeout utility
+# Test for timeout utility
 if command -v timeout >/dev/null; then
-    TEST_TIMEOUT_CMD="timeout 2m "
+    TEST_TIMEOUT_CMD="timeout 2m"
     echo "INFO: using 2 minute timeout for tests"
 else
     TEST_TIMEOUT_CMD=""