Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[stable27] Flaky CI fixes #4391

Merged
merged 7 commits into from Jun 27, 2023
Merged

[stable27] Flaky CI fixes #4391

merged 7 commits into from Jun 27, 2023

Conversation

juliushaertl
Copy link
Member

@juliushaertl juliushaertl commented Jun 26, 2023

Combined backport of #4357 and #4364

  • use proper database for cypress
  • ci(phpunit): Work around flaky self hosted runners
  • ci(cypress): Use cy.request instead of axios
  • ci(cypress): Use cypress select file
  • ci(cypress): Attempt to fix flaky recovery test
  • ci(cypress): Avoid unneeded user removal
  • ci(cypress): Clear page to avoid issues when switching sessions while requests are still being fired

📝 Summary

  • Resolves: #

🖼️ Screenshots

🏚️ Before 🏡 After
B A

🚧 TODO

  • ...

🏁 Checklist

  • Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
  • Sign-off message is added to all commits
  • Tests (unit, integration and/or end-to-end) passing and the changes are covered with tests
  • Documentation (README or documentation) has been updated or is not required

@cypress
Copy link

cypress bot commented Jun 26, 2023

1 failed tests on run #10705 ↗︎

1 141 1 0 Flakiness 0

Details:

[stable27] Flaky CI fixes
Project: Text Commit: 11b3428a0e
Status: Failed Duration: 04:58 💡
Started: Jun 27, 2023 4:17 PM Ended: Jun 27, 2023 4:22 PM
Failed  cypress/e2e/sync.spec.js • 1 failed test

View Output Video

Test Artifacts
Sync > recovers from a lost connection Output Screenshots

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
… requests are still being fired

Signed-off-by: Julius Härtl <jus@bitgrid.net>
@mejo- mejo- merged commit 526f0fc into stable27 Jun 27, 2023
39 of 42 checks passed
@delete-merged-branch delete-merged-branch bot deleted the ci/flaky-stable27 branch June 27, 2023 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants