Skip to content

Conversation

@max-nextcloud
Copy link
Collaborator

Tests were failing because cypress proceeded
with adding files to a folder that had not been created yet:
https://github.com/nextcloud/text/runs/6780248454?check_suite_focus=true

This also happened on stable24: https://github.com/nextcloud/text/runs/7165024139?check_suite_focus=true

Signed-off-by: Max [email protected]

  • Target version: stable24

Tests were failing because cypress proceeded
with adding files to a folder that had not been created yet:
https://github.com/nextcloud/text/runs/6780248454?check_suite_focus=true

Do not create the callout test directory twice.
Nested cypress blocks will execute `beforeEach` on all levels.

Signed-off-by: Max <[email protected]>
@max-nextcloud max-nextcloud changed the title fix: return promise in cypress command to wait for it [stable24] fix: return promise in cypress command to wait for it Jul 4, 2022
@juliusknorr juliusknorr added tests If you write them we ♥ you 3. to review labels Jul 4, 2022
@mejo- mejo- merged commit e1f374f into stable24 Jul 6, 2022
@delete-merged-branch delete-merged-branch bot deleted the test/wait-for-commands branch July 6, 2022 10:28
@blizzz blizzz mentioned this pull request Jul 6, 2022
18 tasks
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.

4 participants