Skip to content
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 a disclaimer about include, contain_exactly, and match_array #172

Open
mcmire opened this issue Jan 2, 2023 · 0 comments
Open

Add a disclaimer about include, contain_exactly, and match_array #172

mcmire opened this issue Jan 2, 2023 · 0 comments

Comments

@mcmire
Copy link
Collaborator

mcmire commented Jan 2, 2023

When used with an array, these matchers don't assume that the array is in a particular kind of order. Therefore, the diff that SuperDiff produces for them will probably a bit difficult to grok, particularly if you're using fuzzy matcher objects (an_object_having_attributes et al.). In addition, if you have elision enabled, it's actually going to hide details that you need to debug your test properly. Therefore, we should recommend that people use match or even eq as it will lead to a better developer experience.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant