-
Notifications
You must be signed in to change notification settings - Fork 153
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
getByDisplayValue is missing in TypeScript types #42
Comments
Thanks for the issue. Could you make a pull request to add them? |
The PR should also include updated tests inside https://github.com/testing-library/cypress-testing-library/blob/master/cypress/integration/commands.spec.js and https://github.com/testing-library/cypress-testing-library/blob/master/tests/typescript-types/test.spec.ts if possible. Because these tests are missing the *ByValue or *ByDisplayValue all together the issue didn't occur automatically earlier. PS: For now the PR only contains the fixes but I might get back to adjusting the tests and creating another PR for those. Not enough time right now |
These changes in typings were not done when changing the API from *ByValue to *ByDisplayValue Closes #42
🎉 This issue has been resolved in version 4.0.3 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
cypress-testing-library
version: 3.0.1node
version: 11.10.0yarn
version: 1.14.0Relevant code or config
What you did:
updated to
cypress-testing-library
3.0.1 and because of breaking changes indom-testing-library
as outlined in the release notes, I switched mygetByValue
queries togetByDisplayValue
.What happened:
TS compile fails because
getByDisplayValue
is not defined incypress-testing-library/typings
Suggested solution:
update typings to reflect the changes in the API
The text was updated successfully, but these errors were encountered: