[openstreetmap/openstreetmap-website] Add minitest-focus gem (PR #4510)
Andy Allan
notifications at github.com
Wed Feb 7 18:17:33 UTC 2024
This PR adds the [minitest-focus](https://github.com/minitest/minitest-focus) gem, which I've been using a lot recently. It's particularly helpful if you are examining the test logs, or doing some other work which means the other tests in the file are just a distraction.
There's a small risk with focussing a test, that it might be committed as focussed. If nobody notices during review, that can cause the rest of the test suite to be skipped. To mitigate this (admittedly remote) risk, I've set it up to not be enabled on CI, so any use of the 'focus' keyword there will cause the entire suite to fail.
(My preferred mitigation would be a rubocop rule like https://docs.rubocop.org/rubocop-rspec/cops_rspec.html#rspecfocus which would have a similar effect, but that doesn't yet exist for rubocop-minitest).
You can view, comment on, or merge this pull request online at:
https://github.com/openstreetmap/openstreetmap-website/pull/4510
-- Commit Summary --
* Add minitest-focus gem
-- File Changes --
M Gemfile (1)
M Gemfile.lock (3)
M test/test_helper.rb (1)
-- Patch Links --
https://github.com/openstreetmap/openstreetmap-website/pull/4510.patch
https://github.com/openstreetmap/openstreetmap-website/pull/4510.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4510
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/4510 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240207/67fa5c92/attachment.htm>
More information about the rails-dev
mailing list