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

[stable26] Flaky CI fixes #4392

Merged
merged 7 commits into from Jun 27, 2023
Merged

[stable26] Flaky CI fixes #4392

merged 7 commits into from Jun 27, 2023

Conversation

juliushaertl
Copy link
Member

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 and 1 flaky tests on run #10702 ↗︎

1 144 1 0 Flakiness 1

Details:

[stable26] Flaky CI fixes
Project: Text Commit: b11099f59c
Status: Failed Duration: 04:03 💡
Started: Jun 27, 2023 3:33 PM Ended: Jun 27, 2023 3:37 PM
Failed  cypress/e2e/sync.spec.js • 1 failed test

View Output Video

Test Artifacts
Sync > recovers from a lost connection Output Screenshots
Flakiness  cypress/e2e/share.spec.js • 1 flaky test

View Output Video

Test Artifacts
Open test.md in viewer > Share a file with download disabled shows an error 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-
Copy link
Member

mejo- commented Jun 27, 2023

/compile

@mejo- mejo- merged commit 22d2c19 into stable26 Jun 27, 2023
39 of 42 checks passed
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