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

shoulda-context is now incompatible with shoulda-matchers 3.0 #53

Closed
mcmire opened this issue Jun 14, 2016 · 3 comments
Closed

shoulda-context is now incompatible with shoulda-matchers 3.0 #53

mcmire opened this issue Jun 14, 2016 · 3 comments

Comments

@mcmire
Copy link
Collaborator

mcmire commented Jun 14, 2016

I'll have a PR forthcoming, but for now:

shoulda-matchers 3.0 dropped support for RSpec 2 and removed negative_failure_message from all matcher classes in favor of failure_message_when_negated. Unfortunately, shoulda-context thinks that negative_failure_message still exists.

There are two ways to fix this:

a) Try to call failure_message_when_negated instead of negative_failure_message (would break backward compatibility with previous versions of shoulda-matchers)
b) Try to call failure_message_when_negated first, then negative_failure_message

Anyone have any opinions on this?

(Original issue: thoughtbot/shoulda-matchers#906)

@mcmire
Copy link
Collaborator Author

mcmire commented Mar 6, 2017

Just to update folks: To fully finish this I need to update Shoulda and add more integration tests for RSpec 3 (specifically around negative assertions). I just recently converted the Cucumber tests in Shoulda to Minitest tests (borrowing some code from shoulda-matchers) so this should be a lot easier than before to do. I'm trying to work on this when I have time although help would certainly be appreciated.

@mcmire
Copy link
Collaborator Author

mcmire commented Feb 2, 2018

For anyone checking in on this, this is the number one priority for this gem right now, but it depends on finishing the (known) Rails 5 issues for shoulda-matchers. Once that's done:

  • I'll release shoulda-matchers 4.0.0.rc1
  • Fix this gem so it's compatible with the RSpec 3 API change which shoulda-matchers abides by
  • Release shoulda-context 2.0.0.rc1
  • Release shoulda 4.0.0.rc1 that points to shoulda-context 2.0.0.rc1 and shoulda-matchers 4.0.0.rc1

@mcmire
Copy link
Collaborator Author

mcmire commented Apr 21, 2019

This is fixed starting with shoulda-context 2.0.0.rc1, so I'm going to go ahead and close this.

@mcmire mcmire closed this as completed Apr 21, 2019
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

1 participant