diff options
| author | Kasper Timm Hansen <kaspth@gmail.com> | 2016-05-28 22:26:45 +0200 |
|---|---|---|
| committer | Kasper Timm Hansen <kaspth@gmail.com> | 2016-05-28 22:26:45 +0200 |
| commit | e45f388eb439448f4f87eabcef1bc01a14faa4a5 (patch) | |
| tree | 72901b9cb4d2df067205e65fdf4907cf3ec1345c /activesupport | |
| parent | af245aaf3a1c447752a1b5895adddc66e5f73c00 (diff) | |
| download | rails-e45f388eb439448f4f87eabcef1bc01a14faa4a5.tar.gz rails-e45f388eb439448f4f87eabcef1bc01a14faa4a5.tar.bz2 rails-e45f388eb439448f4f87eabcef1bc01a14faa4a5.zip | |
Fix assertion count after 69e1bb52.
When changing the generated integration tests to assert redirects with
an absolute path this redirection check in `assert_redirected_to` against
the absolute response location would now pass:
https://github.com/rails/rails/blob/af245aaf3a1c447752a1b5895adddc66e5f73c00/actionpack/lib/action_dispatch/testing/assertions/response.rb#L55
Thus we'd break early instead of hitting `assert_operator` and have
2 fewer assertions per `assert_redirected_to` as `assert_operator`
is composed of 2 assertions internally:
https://github.com/seattlerb/minitest/blob/4e146b1515b19a5c474e39c174b037510f6dbc6e/lib/minitest/assertions.rb#L254-L258
Deduct 2 assertions for 2 redirects taking the expected count down by 4
in total for the two failing tests.
Diffstat (limited to 'activesupport')
0 files changed, 0 insertions, 0 deletions