Revert "test: Add system tests for exercise administration"#3065
Revert "test: Add system tests for exercise administration"#3065arkirchner wants to merge 1 commit intomainfrom
Conversation
This reverts commit 152b140.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3065 +/- ##
==========================================
- Coverage 70.18% 70.08% -0.11%
==========================================
Files 215 215
Lines 6850 6850
==========================================
- Hits 4808 4801 -7
- Misses 2042 2049 +7 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| @@ -1,162 +0,0 @@ | |||
| # frozen_string_literal: true | |||
There was a problem hiding this comment.
So I understand that the test is flaky.
Could we use other measures than removing it entirely? I'm thinking about retrying it or marking it as flaky.
There was a problem hiding this comment.
Thanks for pushing me to check this more deeply. We recently switched from Chrome to Firefox in the CI. Reverting this fixed the issue.
We were switching to Firefox because there were issues with Chrome. They seem to be solved. We can switch back to Chrome.
Can you review: openHPI/codeharbor#2105 and #3067
The system test is unstable. Revert for now to allow the CI to pass.
Reverts #3008