Skip to content

Commit dea70f1

Browse files
author
Michael Vasseur
committed
Revert "Don't mask the deprecation failure"
This reverts commit c586c15.
1 parent c586c15 commit dea70f1

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

gitlab/unit-tests.sh

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -36,12 +36,11 @@ php $phpcov webapp/bin/phpunit -c webapp/phpunit.xml.dist webapp/tests/$unittest
3636
UNITSUCCESS=$?
3737
set -e
3838
CNT=0
39-
THRESHOLD=32
4039
if [ $CODECOVERAGE -eq 1 ]; then
4140
CNT=$(sed -n '/Generating code coverage report/,$p' "$GITLABARTIFACTS"/phpunit.out | grep -v DoctrineTestBundle | grep -cv ^$)
4241
FILE=deprecation.txt
4342
sed -n '/Generating code coverage report/,$p' "$GITLABARTIFACTS"/phpunit.out > ${CI_PROJECT_DIR}/$FILE
44-
if [ $CNT -le $THRESHOLD ]; then
43+
if [ $CNT -le 32 ]; then
4544
STATE=success
4645
else
4746
STATE=failure
@@ -67,7 +66,7 @@ curl https://api.github.com/repos/domjudge/domjudge/statuses/$CI_COMMIT_SHA \
6766
-H "Authorization: token $GH_BOT_TOKEN_OBSCURED" \
6867
-H "Accept: application/vnd.github.v3+json" \
6968
-d "{\"state\": \"$STATE\", \"target_url\": \"${CI_PIPELINE_URL}/test_report\", \"description\":\"Unit tests\", \"context\": \"unit_tests ($version)\"}"
70-
if [ $UNITSUCCESS -ne 0 ] || [ $CNT -le $THRESHOLD ]; then
69+
if [ $UNITSUCCESS -ne 0 ]; then
7170
exit 1
7271
fi
7372

0 commit comments

Comments
 (0)