Skip to content

Commit 32ef48f

Browse files
committed
Fix Mocha/Minitest compatibility
It turns out we were relying on the very old [1] MiniTest module name rather than the newer Minitest module name. While there are other places in the code that use the MiniTest form, most (all?) of those are internal to Mocha. Anyway making this one change seems to fix the problems people are having. Ideally I'd like to add a failing test to force me to make this change, but I don't have time right now and I want to fix the problem for people as soon as possible. I've run this fix against a large test suite and the build passes [2], so it doesn't feel very risky releasing this. It would also be good to do a general clean up of all the uses of MiniTest to make sure I haven't missed anything. Fixes #614. [1]: https://github.com/minitest/minitest/blob/master/History.rdoc#505--2013-06-20- [2]: alphagov/whitehall#8015
1 parent d6470af commit 32ef48f

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

lib/mocha/integration/mini_test/adapter.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ def self.description
2323

2424
# @private
2525
def self.included(_mod)
26-
Mocha::ExpectationErrorFactory.exception_class = ::MiniTest::Assertion
26+
Mocha::ExpectationErrorFactory.exception_class = ::Minitest::Assertion
2727
end
2828

2929
# @private

0 commit comments

Comments
 (0)