-
Notifications
You must be signed in to change notification settings - Fork 288
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
Use native Jest matchers #112
Comments
Perhaps allow matchers to receive element refs instead of selectors? That way they're simply doing assertions. // Edited because the original idea was silly. |
Thanks for your suggestion but I am sorry didn't get it, can you show me an example of code? |
👍 on the idea of allowing matchers to receive element refs instead of selectors. It would be nice to be able to do:
Where $btn is already an element handle. |
@gregberge so is this essentially a wontfix? |
Sorry it was a mistake. |
Jest supports asynchronous matchers and asymmetric matchers, two required things to port expect-puppeteer matchers into Jest matchers. Some problems still persist:
Some matchers are returning an element like
toMatchElement()
A matcher should not return anything, yes. But it is very convenient to assert that something is in the page and get the element matched in the page. I think we probably need to rethink this part. I am open to propositions.
Some matchers are already used by Jest like
toMatch
I think it is not a big problem, we can change the name. Also open to propositions.
The text was updated successfully, but these errors were encountered: