X-Git-Url: https://git.openstreetmap.org/rails.git/blobdiff_plain/dc2a2c8ebd1a11e4a64555fda22c6859a51defff..4b5cb04d41e79db06707797c5a4b60e6aba27702:/test/controllers/old_relation_controller_test.rb diff --git a/test/controllers/old_relation_controller_test.rb b/test/controllers/old_relation_controller_test.rb index 1ce87a252..622229661 100644 --- a/test/controllers/old_relation_controller_test.rb +++ b/test/controllers/old_relation_controller_test.rb @@ -145,6 +145,61 @@ class OldRelationControllerTest < ActionController::TestCase assert_select "osm relation[id='#{relation.relation_id}'][version='#{relation.version}']", 0, "redacted relation #{relation.relation_id} version #{relation.version} shouldn't be present in the history." end + ## + # test the unredaction of an old version of a relation, while not being + # authorised. + def test_unredact_relation_unauthorised + relation = relations(:relation_with_redacted_versions_v3) + + post :redact, :id => relation.relation_id, :version => relation.version + assert_response :unauthorized, "should need to be authenticated to unredact." + end + + ## + # test the unredaction of an old version of a relation, while being + # authorised as a normal user. + def test_unredact_relation_normal_user + relation = relations(:relation_with_redacted_versions_v3) + basic_authorization(users(:public_user).email, "test") + + post :redact, :id => relation.relation_id, :version => relation.version + assert_response :forbidden, "should need to be moderator to unredact." + end + + ## + # test the unredaction of an old version of a relation, while being + # authorised as a moderator. + def test_unredact_relation_moderator + relation = relations(:relation_with_redacted_versions_v3) + basic_authorization(users(:moderator_user).email, "test") + + post :redact, :id => relation.relation_id, :version => relation.version + assert_response :success, "should be OK to unredact old version as moderator." + + # check moderator can still see the redacted data, without passing + # the appropriate flag + get :version, :id => relation.relation_id, :version => relation.version + assert_response :success, "After redaction, node should not be gone for moderator, when flag passed." + + # and when accessed via history + get :history, :id => relation.relation_id + assert_response :success, "Redaction shouldn't have stopped history working." + assert_select "osm relation[id='#{relation.relation_id}'][version='#{relation.version}']", 1, "relation #{relation.relation_id} version #{relation.version} should still be present in the history for moderators when passing flag." + + basic_authorization(users(:normal_user).email, "test") + + # check normal user can now see the redacted data + get :version, :id => relation.relation_id, :version => relation.version + assert_response :success, "After redaction, node should not be gone for moderator, when flag passed." + + # and when accessed via history + get :history, :id => relation.relation_id + assert_response :success, "Redaction shouldn't have stopped history working." + assert_select "osm relation[id='#{relation.relation_id}'][version='#{relation.version}']", 1, "relation #{relation.relation_id} version #{relation.version} should still be present in the history for moderators when passing flag." + end + + private + ## # check that the current version of a relation is equivalent to the # version which we're getting from the versions call.