[openstreetmap/openstreetmap-website] More controller test refactoring (#2613)

Andy Allan notifications at github.com
Wed May 6 14:53:39 UTC 2020


There's definitely some future work in renaming the routes. In this PR I've added various names, which either a) match similar examples that already existed for other elements (like nodes when naming a relation route) or b) matched what the "resourceful routing" would give, when we eventually refactor the routes to be resourceful.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * More controller test refactoring

-- File Changes --

    M config/routes.rb (32)
    M test/controllers/api/nodes_controller_test.rb (146)
    M test/controllers/api/old_nodes_controller_test.rb (92)
    M test/controllers/api/old_relations_controller_test.rb (76)
    M test/controllers/api/old_ways_controller_test.rb (86)
    M test/controllers/api/relations_controller_test.rb (204)
    M test/controllers/api/ways_controller_test.rb (158)

-- Patch Links --

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

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/2613
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20200506/baf491f2/attachment.htm>


More information about the rails-dev mailing list