update mocha dependency #12

merged 2 commits into from Jun 16, 2012


None yet

3 participants


all tests pass, we need an updated mocha on our project to fix a test-unit bug

thoughtbot, inc. member

I know this is unfortunate, but we haven't found a way to inject bourne's invocation tracking into mocha without copying and modifying this method: https://github.com/thoughtbot/bourne/blob/master/lib/bourne/mock.rb

Any time we update mocha, we need to also update that file and the tests from mocha. Otherwise, we're just running old tests that verify the old behavior, and mocha doesn't actually get updated.

@jferris jferris closed this Jun 4, 2012

can you add a comment that shows what needs to be copied or add instructions to the readme and Ill give it a try, maybe I can automate it somehow...

also please Reopen


Just checked and mocha did not change method_missing between 10.5 and 11.4 so this is save to merge

@jferris jferris reopened this Jun 8, 2012
@mike-burns mike-burns merged commit edebb8b into thoughtbot:master Jun 16, 2012
thoughtbot, inc. member

The tests and method_missing have indeed remain unchanged. Pull this, thanks!

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