Skip to content

test(cy): use more robust selectors in file picker - #5349

Merged
juliusknorr merged 1 commit into
mainfrom
test/improve-cy-selectors
Feb 4, 2024
Merged

test(cy): use more robust selectors in file picker#5349
juliusknorr merged 1 commit into
mainfrom
test/improve-cy-selectors

Conversation

@max-nextcloud

Copy link
Copy Markdown
Collaborator

📝 Summary

  • Use the selectors used in server to test the files app.
  • Rely on semantics (nav, [aria-label=...]).

🏁 Checklist

  • Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
  • Sign-off message is added to all commits
  • Tests (unit, integration and/or end-to-end) passing and the changes are covered with tests
  • Documentation is not required

@max-nextcloud

Copy link
Copy Markdown
CollaboratorAuthor

/backport to stable28

@max-nextcloud

Copy link
Copy Markdown
CollaboratorAuthor

Requesting backport for sake of consistency.

* Use the selectors used in server to test the files app.
* Rely on semantics (`nav`, `[aria-label=...]`).
Signed-off-by: Max <max@nextcloud.com>
@max-nextcloud
max-nextcloudforce-pushed the test/improve-cy-selectors branch from d2f20b9 to 39b2862CompareFebruary 4, 2024 07:12
@juliusknorr
juliusknorr merged commit 781129b into mainFeb 4, 2024
@juliusknorr
juliusknorr deleted the test/improve-cy-selectors branch February 4, 2024 07:36
@blizzzblizzz mentioned this pull request Mar 5, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@max-nextcloud@juliusknorr