Skip to content

Commit 03aed6c

Browse files
committed
Fail integration test on any PHP/Framwork Exception
In the past we only did this for warnings in the prod env but those shouldn't happen regardless of if we run those in `dev`, `test` or `prod`. This hopefully protects future tests where we miss this line.
1 parent f02f40a commit 03aed6c

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

.github/workflows/integration.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -132,10 +132,10 @@ jobs:
132132
curl $CURLOPTS http://localhost/domjudge/jury/contests/1/freeze/doNow || true
133133
curl $CURLOPTS http://localhost/domjudge/jury/contests/1/end/doNow || true
134134
curl $CURLOPTS -X POST -d 'finalize_contest[b]=0&finalize_contest[finalizecomment]=gitlab&finalize_contest[finalize]=' http://localhost/domjudge/jury/contests/1/finalize
135-
- name: Verify no errors in prod.log
135+
- name: Verify no errors in symfony {prod,test,dev}.log
136136
shell: bash
137137
run: |
138-
if cat /opt/domjudge/domserver/webapp/var/log/prod.log | egrep '(CRITICAL|ERROR):'; then
138+
if cat /opt/domjudge/domserver/webapp/var/log/*.log | egrep '(CRITICAL|ERROR):'; then
139139
exit 1
140140
fi
141141
- name: Download and perform API check

0 commit comments

Comments
 (0)