Skip to content

Conversation

@juliusknorr
Copy link
Member

@juliusknorr juliusknorr commented Jun 28, 2023

Yet again, we've ran cypress against the wrong branch 🙈

Copy link
Member

@mejo- mejo- left a comment

Choose a reason for hiding this comment

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

Uppsie. That means we tested against server master so far? 🙈

@juliusknorr
Copy link
Member Author

juliusknorr commented Jun 28, 2023

Yes, I've already wondered why 26 fails the same ways as 27 while my debugging showed that only parts added on 27 caused this,.

Should actually make stable26 tests far more stable again as far as I could see the instability was just started with 27

@cypress
Copy link

cypress bot commented Jun 28, 2023

3 failed and 1 flaky tests on run #10736 ↗︎

3 142 1 0 Flakiness 1

Details:

chore(ci): Run cypress against proper stable branch
Project: Text Commit: 3f7a503057
Status: Failed Duration: 05:37 💡
Started: Jun 28, 2023 1:33 PM Ended: Jun 28, 2023 1:39 PM
Failed  attachments.spec.js • 2 failed tests

View Output Video

Test Artifacts
Test all attachment insertion methods > See test files in the list and display hidden files Output Screenshots
Test all attachment insertion methods > [share] check everything behaves correctly on the share target user side Output Screenshots
Failed  share.spec.js • 1 failed test

View Output Video

Test Artifacts
Open test.md in viewer > Share a file with download disabled shows an error Output Screenshots
Flakiness  cypress/e2e/sync.spec.js • 1 flaky 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.

@mejo- mejo- merged commit ddcfdf1 into stable26 Jun 28, 2023
@delete-merged-branch delete-merged-branch bot deleted the fix/cypress-stable26 branch June 28, 2023 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review tests If you write them we ♥ you

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants