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

chore(cypress): Migrate access-level tests from Behat to Cypress #41137

Merged
merged 3 commits into from Nov 2, 2023

Conversation

susnux
Copy link
Contributor

@susnux susnux commented Oct 26, 2023

Summary

Port some more acceptance tests from Behat to Cypress.
They also belong somewhat to the already existing Cypress tests for the settings.

Checklist

@susnux susnux added 3. to review Waiting for reviews tests Related to tests CI labels Oct 26, 2023
@susnux susnux force-pushed the chore/migrate-access-level-cypress branch 2 times, most recently from 40b265d to 4c12e19 Compare October 26, 2023 14:57
@susnux
Copy link
Contributor Author

susnux commented Oct 26, 2023

I noticed the header acceptance tests are basically the same ...
So I also migrated the duplicated tests

@solracsf solracsf added this to the Nextcloud 28 milestone Oct 27, 2023
@susnux susnux mentioned this pull request Oct 29, 2023
13 tasks
…his are UI tests

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
…to Cypress

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux susnux force-pushed the chore/migrate-access-level-cypress branch from d1746b0 to 8e2824e Compare October 30, 2023 00:05
@skjnldsv skjnldsv mentioned this pull request Nov 1, 2023
Copy link
Member

@AndyScherzinger AndyScherzinger left a comment

Choose a reason for hiding this comment

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

🐘

@susnux susnux merged commit 84421f4 into master Nov 2, 2023
51 checks passed
@susnux susnux deleted the chore/migrate-access-level-cypress branch November 2, 2023 16:44
@blizzz blizzz mentioned this pull request Nov 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews CI tests Related to tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants