-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
feat: Implement Locator.nth() #7137
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for vitest-dev ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
The exception message is 100% accurate now that I am using the I feel a bit bad because I am writing selector notation by hand, using string interpolation, whereas the other methods are importing from ivya I also added |
Description
This adds more parity with Playwright locators as discussed in #7008.
Compared to
elements()[n]
, thenth
modifier promotes easier-to-debug tests by throwing an obvious error message instead of letting a matcher fail onundefined
or allowing the caller to try and call some additional locator method.Please don't delete this checklist! Before submitting the PR, please make sure you do the following:
pnpm-lock.yaml
unless you introduce a new test example.Tests
pnpm test:ci
.Documentation
pnpm run docs
command.Changesets
feat:
,fix:
,perf:
,docs:
, orchore:
.