diff options
| author | Lukas Reschke <lukas@statuscode.ch> | 2016-12-21 01:39:56 +0300 |
|---|---|---|
| committer | Lukas Reschke <lukas@statuscode.ch> | 2016-12-21 01:40:49 +0300 |
| commit | 7efc6b26140f41dee26bfe241b67a3c9738d265d (patch) | |
| tree | e1c7e6337e59e63ee44f249a21baf5476c06d9cd /autotest-external.sh | |
| parent | e3dfe2cba7ad53c69b443a18926d47e006df09d7 (diff) | |
Set exit code to 1 if integration test is failing
Before it just checked the unit tests and ignored the integration test results. See https://drone.nextcloud.com/nextcloud/server/3374/35 for details.
Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
Diffstat (limited to 'autotest-external.sh')
| -rwxr-xr-x | autotest-external.sh | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/autotest-external.sh b/autotest-external.sh index a6ca077ebb0..1f1ca330ac5 100755 --- a/autotest-external.sh +++ b/autotest-external.sh @@ -210,16 +210,19 @@ EOF rm -rf "coverage-external-html-$1-$name" mkdir "coverage-external-html-$1-$name" "$PHPUNIT" --configuration phpunit-autotest-external.xml --log-junit "autotest-external-results-$1-$name.xml" --coverage-clover "autotest-external-clover-$1-$name.xml" --coverage-html "coverage-external-html-$1-$name" "$FILES_EXTERNAL_BACKEND_PATH/$testToRun" - RESULT=$? else echo "No coverage" "$PHPUNIT" --configuration phpunit-autotest-external.xml --log-junit "autotest-external-results-$1-$name.xml" "$FILES_EXTERNAL_BACKEND_PATH/$testToRun" - RESULT=$? fi else DOEXIT=1 fi + if [[ $? -ne 0 ]]; then + echo "Error during phpunit execution ... terminating" + exit 1 + fi + # calculate stop file stopFile=`echo "$startFile" | sed 's/start/stop/'` echo "stop: $stopFile" |