diff options
| author | Tomi Ollila <tomi.ollila@iki.fi> | 2015-08-06 12:13:36 +0300 |
|---|---|---|
| committer | David Bremner <david@tethera.net> | 2015-08-07 21:56:39 +0200 |
| commit | 02a2eeb427d6b424029f6e5e5ddad4c6ec987741 (patch) | |
| tree | abdc66ca743a1f21c70238e4c118a51d08c253f1 /test/README | |
| parent | af3eba97fde00d6bc922df87732b3d28f717f120 (diff) | |
test: make script exit (1) if it "fails" to source (.) a file
The files (test) scripts source (with builtin command `.`) provides
information which the scripts depend, and without the `source` to
succeed allowing script to continue may lead to dangerous situations
(e.g. rm -rf "${undefined_variable}"/*).
At the end of all source (.) lines construct ' || exit 1' was added;
In our case the script script will exit if it cannot find (or read) the
file to be sourced. Additionally script would also exits if the last
command of the sourced file exited nonzero.
Diffstat (limited to 'test/README')
| -rw-r--r-- | test/README | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/README b/test/README index daf41600..ce403cef 100644 --- a/test/README +++ b/test/README @@ -138,7 +138,7 @@ Source 'test-lib.sh' After assigning test_description, the test script should source test-lib.sh like this: - . ./test-lib.sh + . ./test-lib.sh || exit 1 This test harness library does the following things: |