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

Error message shows no difference #1493

Closed
jcoyne opened this issue Oct 26, 2022 · 5 comments
Closed

Error message shows no difference #1493

jcoyne opened this issue Oct 26, 2022 · 5 comments

Comments

@jcoyne
Copy link
Contributor

jcoyne commented Oct 26, 2022

This test started failing on 3.11.2

       #<InstanceDouble(Blacklight::Solr::Repository) (anonymous)> received :suggestions with unexpected arguments
         expected: ({:q=>"test"})
              got: ({:q=>"test"})

When I pin rspec-mocks to 3.22.1, the error goes away.

The hash that is passed is in fact a different object, but they both contain the same data.

@jrochkind
Copy link

This test started failing on 3.11.2
When I pin rspec-mocks to 3.22.1, the error goes away.

Do you mean tests started failing on 3.22.2?

@jcoyne
Copy link
Contributor Author

jcoyne commented Oct 26, 2022

Do you mean tests started failing on 3.22.2?

No, I meant when I pin to 3.11.1 it works.

@JonRowe
Copy link
Member

JonRowe commented Oct 26, 2022

👋 This is fixed in the yet to be released 3.12.0, you can use RSpec from main for now or await a release. There was a fix when detecting keyword arguments with with in 3.11.2 but the change to improve the diff requires a different release cycle, given that this may occur for a few people I'm working on releasing 3.12.0 either tonight or tomorrow.

@JonRowe JonRowe closed this as completed Oct 26, 2022
@JonRowe
Copy link
Member

JonRowe commented Oct 26, 2022

3.12.0 is releases which should highligh the difference (that one is a keyword argument and the other a hash), feel free to provide a reproduction if you think thats in error (there is one unresolved issue related to this in #1492 )

@jcoyne
Copy link
Contributor Author

jcoyne commented Oct 26, 2022

Yes, loads better. Thanks!

jcoyne added a commit to projectblacklight/blacklight that referenced this issue Oct 26, 2022
cbeer pushed a commit to projectblacklight/blacklight that referenced this issue Nov 8, 2022
cbeer pushed a commit to projectblacklight/blacklight that referenced this issue Nov 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants