-
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
Add filter tests to URL field type #3375
Conversation
🦋 Changeset is good to goLatest commit: bfa3c86 We got this. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
It looks like we could use some more details. Good titles and descriptions help us fix things faster. If you can provide more information please update this issue. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are testing this since it uses the Text
implementation?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We will want to test the case-insensitive filter options as well.
5df1bd8
to
67c0e0f
Compare
This is about black-box testing the API to ensure that it behaves in the way we expect, independent of what we know about the implementation details. This is important for verifying the correctness of the prisma adapter in #3298 . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good 👍
67c0e0f
to
4c60a0f
Compare
4c60a0f
to
bfa3c86
Compare
No description provided.