mirror of
https://github.com/rails/rails.git
synced 2022-11-09 12:12:34 -05:00
9d0cf52096
There are two main reasons why `assert_called_with` should require
`args` argument:
1) If we want to assert that some method should be called and we don't
need to check with which arguments it should be called then we should use
`assert_called`.
2) `assert_called_with` without `args` argument doesn't assert anything!
```ruby
assert_called_with(@object, :increment) do
@object.decrement
end
```
It causes false assertions in tests that could cause regressions in the project.
I found this bug by working on
[minitest-mock_expectations](https://github.com/bogdanvlviv/minitest-mock_expectations) gem.
This gem is an extension for minitest that provides almost the same method call
assertions.
I was wondering whether you would consider adding "minitest-mock_expectations"
to `rails/rails` instead of private `ActiveSupport::Testing::MethodCallAssertions` module.
If yes, I'll send a patch -
|
||
|---|---|---|
| .. | ||
| assertions.rb | ||
| autorun.rb | ||
| constant_lookup.rb | ||
| declarative.rb | ||
| deprecation.rb | ||
| file_fixtures.rb | ||
| isolation.rb | ||
| method_call_assertions.rb | ||
| parallelization.rb | ||
| setup_and_teardown.rb | ||
| stream.rb | ||
| tagged_logging.rb | ||
| time_helpers.rb | ||