Nicer tests output

Author: 1999

test/index.sh

@@ -13,148 +13,201 @@ function test {
     return $status
 }
 
+function showTestResult {
+    if [ $1 -eq 0 ]; then
+        echo "OK"
+    else
+        echo "Failed"
+    fi
+}
+
 MOCHA_VERSION=`mocha --version`
 echo "You're running tests with mocha version $MOCHA_VERSION"
 echo ""
 
 echo 'TESTCASE: native (json) reporter'
 test test/reporter-native-json/index.sh
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: native (tap) reporter'
 test test/reporter-native-tap/index.sh
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: custom (teamcity) reporter'
 test test/reporter-custom-teamcity/index.sh
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: custom (jenkins) reporter'
 test test/reporter-custom-jenkins/index.sh
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: custom (mochawesome) reporter'
 test test/reporter-custom-mochawesome/index.sh
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: custom (allure) reporter'
 test test/reporter-custom-allure/index.sh
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: cli target'
 test test/cli-target/index.sh
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: pwd-based reporter'
 test test/reporter-pwd/index.sh
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: reporter emits events as soon as they come from subprocess'
 test test/reporter-log/index.js
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: parallel'
 test test/parallel/parallel.sh
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: parallel order'
 test test/parallel-order/index.js
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: max parallel tests'
 test test/max-parallel/index.sh
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: max parallel equal 1'
 test test/max-parallel-1/index.sh
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: max parallel tests with empty ones'
 test test/max-parallel-empty/index.sh
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: only tests run'
 test test/only-tests-run/index.js
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: nesting'
 test test/nesting/nesting.sh
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: describe inside describe'
 test test/describe-inside-describe/index.js
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: missing test'
 test test/missing-test/index.js
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: console logs'
 test test/console-log-inject/index.js
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: global hooks'
 test test/global-hooks/index.sh
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: global hooks with required files'
 test test/global-hooks-require/index.sh
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: global hooks with directory as a tests source'
 test test/global-hooks-directory/index.sh
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: --recursive option if no target is set'
 test test/recursive-no-target/index.js
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: total time'
 test test/total-time/index.js
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: timeouts exit code'
 test test/timeouts-exit-code/index.sh
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: no timeouts'
 test test/no-timeouts/index.sh
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: js compilers support'
 test test/js-compilers/index.sh
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: js compilers with files support'
 test test/js-compilers-1/index.sh
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: js compilers with --require support'
 test test/js-compilers-2/index.sh
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: reporter with options'
 test test/reporter-options/index.sh
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: mocha.opts'
 test test/mocha-opts/index.sh
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: syntax errors'
 test test/syntax-errors/index.js
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: --require option support'
 test test/require-option/index.sh
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: run programmatically (base API)'
 test test/run-programmatically/callback/index.js
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: run programmatically (reporter.done is called)'
 test test/run-programmatically/reporter-done/index.js
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: --no-exit option support'
 test test/reporter-end-no-exit/index.js
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: node add-on'
 test test/node-addon/index.sh
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: skip-suite'
 test test/skip-suite/index.sh
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: skip-test'
 test test/skip-test/index.sh
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: --delay option support'
 test test/delay/index.js
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: --retries option support'
 test test/retries/index.js
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: --exit option support'
 test test/exit/index.js
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: --retries plus all tests fail'
 test test/retries-all-fail/index.js
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: --retries and --bail should work well together'
 test test/bail-and-retries/index.js
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: subprocess exits before sending an end message'
 test test/no-subprocess-end/index.js
-echo $?
+showTestResult $?
+
 echo 'TESTCASE: unhandled rejections should not force subprocess to exit'
 test test/q-promises/index.js
-echo $?
+showTestResult $?
 
 echo "Passes: $PASSES Failes: $FAILES"
 

test/parallel/parallel.sh

@@ -6,10 +6,10 @@ OUTPUT=$(dist/bin/cli.js -R spec test/parallel/tests --timeout 10000 --slow 1000
 
 TIMESTAMP_FINISH="$(date +%s)"
 TIMESTAMP_DIFF=`expr $TIMESTAMP_FINISH - $TIMESTAMP_START`
-echo "Tests running time was $TIMESTAMP_DIFF seconds"
 
 if [[ $TIMESTAMP_DIFF -lt 10 ]]; then
     exit 0
 else
+    echo "Tests running time was $TIMESTAMP_DIFF seconds"
     exit 1
 fi

test/timeouts-exit-code/index.sh

@@ -9,7 +9,7 @@ MOCHA_STATUS_CODE=$?
 # currently mocha exits with 2 if 2 tests failed, 3 if 3 tests failed etc
 # though it's a strange behaviour, mocha-parallel-tests behaviour should be the same
 if [ $MPT_STATUS_CODE -eq $MOCHA_STATUS_CODE ]; then
-    echo "Timeout status code is the same as in mocha"
+    exit 0
 else
     echo "Timeout status code is not valid: $MPT_STATUS_CODE (actual) vs $MOCHA_STATUS_CODE (expected)"
     exit 1