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

test(cypress): only run in parallel for pull requests #4772

Merged
merged 1 commit into from
Sep 6, 2023

Conversation

max-nextcloud
Copy link
Collaborator

Results from pushes to integration branches are usually not time critical.
Save some resources by avoiding the overhead of parallelization.

https://github.com/nextcloud/text/actions/runs/6089520881 is how pushes to integration branches would look in the future.

Results from pushes are not time critical.
Save some resources by avoiding the overhead of parallelization.

Signed-off-by: Max <max@nextcloud.com>
@cypress
Copy link

cypress bot commented Sep 6, 2023

3 failed and 3 flaky tests on run #12036 ↗︎

3 141 18 0 Flakiness 3

Details:

test(cypress): only run in parallel for pull requests
Project: Text Commit: f30ba20ecd
Status: Failed Duration: 06:56 💡
Started: Sep 6, 2023 6:12 AM Ended: Sep 6, 2023 6:19 AM
Failed  cypress/e2e/attachments.spec.js • 3 failed tests

View Output Video

Test Artifacts
Test all attachment insertion methods > test if attachment folder is moved with the markdown file Output Screenshots
Test all attachment insertion methods > test if attachment folder is copied when copying a markdown file Output Screenshots
Test all attachment insertion methods > [share] check everything behaves correctly on the share target user side Output Screenshots
Flakiness  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
Flakiness  api/SessionApi.spec.js • 1 flaky test

View Output Video

Test Artifacts
The session Api > open the session > returns connection Output Screenshots
Flakiness  api/SyncServiceProvider.spec.js • 1 flaky test

View Output Video

Test Artifacts
Sync service provider > recovers from a dropped message Output Screenshots

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

Copy link
Member

@skjnldsv skjnldsv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure it will save us that much time, but 👍

@juliushaertl juliushaertl merged commit a231529 into main Sep 6, 2023
34 of 37 checks passed
@juliushaertl juliushaertl deleted the test/slow-cypress-on-push branch September 6, 2023 06:42
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.

3 participants