This PR changes redact/unredact tests for the reasons stated in #5645. You can view, comment on, or merge this pull request online at:
https://github.com/openstreetmap/openstreetmap-website/pull/5649 -- Commit Summary -- * Split api old element redacted index tests * Split api old element redacted show tests * Add show_redactions checks to api old element index tests * Test redacted element state after redaction action * Test api old element index accessed as moderator * Add show_redactions checks to api old element show tests * Test api old element show accessed as moderator * Remove do_redact_element methods from api old element redaction tests * Use fewer element versions in redaction tests * Check old element state instead of responses in unredaction tests * Rename old element variables in redaction tests * Test states of old elements after unsuccessful redactions * Move current version redaction tests above roles/scopes tests * Make user creation code in unredact test similar to redact tests -- File Changes -- M test/controllers/api/old_nodes_controller_test.rb (293) M test/controllers/api/old_relations_controller_test.rb (280) M test/controllers/api/old_ways_controller_test.rb (287) -- Patch Links -- https://github.com/openstreetmap/openstreetmap-website/pull/5649.patch https://github.com/openstreetmap/openstreetmap-website/pull/5649.diff -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5649 You are receiving this because you are subscribed to this thread. Message ID: <openstreetmap/openstreetmap-website/pull/5...@github.com>
_______________________________________________ rails-dev mailing list rails-dev@openstreetmap.org https://lists.openstreetmap.org/listinfo/rails-dev
