-
Notifications
You must be signed in to change notification settings - Fork 140
test(cypress): Adapt selectors to Collabora 24.04 #3634
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
Conversation
17ff180 to
059a668
Compare
059a668 to
c563248
Compare
Passing run #842 ↗︎Details:
Review all test suite changes for PR #3634 ↗︎ |
|||||||||||||||
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> test(cypress): Wait for menu to be visible (try) Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
…ppears if loading is not fully ready Signed-off-by: Julius Härtl <jus@bitgrid.net>
02cec46 to
99b330b
Compare
| cy.waitForPostMessage('App_LoadingStatus', { Status: 'Document_Loaded' }) | ||
|
|
||
| // Share action | ||
| cy.wait(2000) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like there is some UI reset once the document renders, but I cannot figure out a proper way to assert that it is actually ready. Would go with the wait for now to move on.
Cypress run recording for reference:
open.spec.failure.mp4
FYI @pedropintosilva Nothing critical i think, though wondering why the UI is reset there
|
/backport to stable29 |
|
/backport to stable28 |
|
/backport to stable27 |
Let's get main branch back to green with Collabora 24.04 that was recently released and the docker image used by CI was updated