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): More flexible selectors for Nextcloud 25-28 #869

Merged
merged 3 commits into from
Sep 11, 2023

Conversation

mejo-
Copy link
Member

@mejo- mejo- commented Sep 11, 2023

No description provided.

@cypress
Copy link

cypress bot commented Sep 11, 2023

3 failed and 5 flaky tests on run #1122 ↗︎

3 233 0 0 Flakiness 5
⚠️ You've recorded test results over your free plan limit.
Upgrade your plan to view test results.

Details:

test(cypress): More flexible selectors for Nextcloud 25-28
Project: Collectives Commit: de935164f3
Status: Failed Duration: 33:38 💡
Started: Sep 11, 2023 12:00 PM Ended: Sep 11, 2023 12:33 PM

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

@mejo- mejo- changed the title test(cypress): Use flexible selectors in settings.spec.js for Nextcloud 25-28 test(cypress): More flexible selectors for Nextcloud 25-28 Sep 11, 2023
@mejo- mejo- force-pushed the fix/cypress_selectors branch 3 times, most recently from 41efc66 to 6dbf079 Compare September 11, 2023 10:08
Apparently required to prevent a race condition where the sidebar is
opened and then hidden again.

Signed-off-by: Jonas <[email protected]>
Copy link

@datenangebot datenangebot left a comment

Choose a reason for hiding this comment

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

looks good

@mejo- mejo- merged commit 27fa078 into main Sep 11, 2023
39 of 46 checks passed
@delete-merged-branch delete-merged-branch bot deleted the fix/cypress_selectors branch September 11, 2023 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants