diff options
| author | Kasper Timm Hansen <kaspth@gmail.com> | 2018-02-18 21:35:30 +0100 |
|---|---|---|
| committer | Kasper Timm Hansen <kaspth@gmail.com> | 2018-02-18 21:35:30 +0100 |
| commit | 6cc000c34cc6ecea9262033c46ad795f9bc56f07 (patch) | |
| tree | 4696e3f12e647ed129be09d66ae994d9fb9f4aa7 /railties/test/generators | |
| parent | 86f7c269073a3a9e6ddec9b957deaa2716f2627d (diff) | |
| download | rails-6cc000c34cc6ecea9262033c46ad795f9bc56f07.tar.gz rails-6cc000c34cc6ecea9262033c46ad795f9bc56f07.tar.bz2 rails-6cc000c34cc6ecea9262033c46ad795f9bc56f07.zip | |
Clean up reporter replacement a bit.
* Don't use :: for class methods, we don't do that elsewhere.
* Don't install a needless method on minitest. Prefer assigning the
reporter anyway as that's what minitest does internally.
* Don't bother opting out when the reporter ain't a Minitest::CompositeReporter.
It's hardcoded: https://github.com/seattlerb/minitest/blob/005a3ba42c07d04797e2d00ac2c53e3be127c12f/lib/minitest.rb#L125
And overrides have to create delegate reporters:
https://github.com/kern/minitest-reporters/blob/1018b1b42f34b01d4de179c8aad2fa06771fe9b0/lib/minitest/minitest_reporter_plugin.rb#L72
Diffstat (limited to 'railties/test/generators')
0 files changed, 0 insertions, 0 deletions