Skip to content

Conversation

@juliusknorr
Copy link
Member

@juliusknorr juliusknorr commented Apr 29, 2024

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

@juliusknorr juliusknorr force-pushed the tests/cypress-24.04 branch from 17ff180 to 059a668 Compare April 29, 2024 13:13
@juliusknorr juliusknorr force-pushed the tests/cypress-24.04 branch from 059a668 to c563248 Compare April 29, 2024 14:17
@juliusknorr juliusknorr added 3. to review Ready to be reviewed tests labels Apr 30, 2024
@cypress
Copy link

cypress bot commented Apr 30, 2024

Passing run #842 ↗︎

0 8 5 0 Flakiness 0

Details:

test(cypress): Adapt selectors to Collabora 24.04
Project: Richdocuments Commit: 02cec464be
Status: Passed Duration: 00:58 💡
Started: Apr 30, 2024 6:29 AM Ended: Apr 30, 2024 6:30 AM

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>
@juliusknorr juliusknorr force-pushed the tests/cypress-24.04 branch from 02cec46 to 99b330b Compare April 30, 2024 06:30
cy.waitForPostMessage('App_LoadingStatus', { Status: 'Document_Loaded' })

// Share action
cy.wait(2000)
Copy link
Member Author

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

@juliusknorr juliusknorr marked this pull request as ready for review April 30, 2024 06:37
@juliusknorr juliusknorr merged commit ae5178d into main Apr 30, 2024
@delete-merged-branch delete-merged-branch bot deleted the tests/cypress-24.04 branch April 30, 2024 14:54
@juliusknorr
Copy link
Member Author

/backport to stable29

@juliusknorr
Copy link
Member Author

/backport to stable28

@juliusknorr
Copy link
Member Author

/backport to stable27

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Ready to be reviewed tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants