[openstreetmap/openstreetmap-website] Use assert_not_predicate instead of assert_not next to assert_predicate (PR #5017)

Anton Khorev notifications at github.com
Thu Jul 25 02:01:08 UTC 2024


Rubocop would complain about something like `assert issue.open?` and it gets changed to `assert_predicate`, but `assert_not issue.open?` is left alone for some reason.
You can view, comment on, or merge this pull request online at:

  https://github.com/openstreetmap/openstreetmap-website/pull/5017

-- Commit Summary --

  * Use assert_not_predicate instead of assert_not next to assert_predicate

-- File Changes --

    M test/controllers/issues_controller_test.rb (12)
    M test/models/user_test.rb (6)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/5017.patch
https://github.com/openstreetmap/openstreetmap-website/pull/5017.diff

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5017
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/5017 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240724/215e2fc6/attachment.htm>


More information about the rails-dev mailing list