X-Git-Url: https://git.openstreetmap.org./rails.git/blobdiff_plain/dc2a2c8ebd1a11e4a64555fda22c6859a51defff..46761798797638a4ef8c35b3bf8e5b1ee83e263f:/test/controllers/old_way_controller_test.rb?ds=inline diff --git a/test/controllers/old_way_controller_test.rb b/test/controllers/old_way_controller_test.rb index f48b0cb89..612b4a6a4 100644 --- a/test/controllers/old_way_controller_test.rb +++ b/test/controllers/old_way_controller_test.rb @@ -171,6 +171,61 @@ class OldWayControllerTest < ActionController::TestCase assert_select "osm way[id='#{way.way_id}'][version='#{way.version}']", 0, "redacted way #{way.way_id} version #{way.version} shouldn't be present in the history." end + ## + # test the unredaction of an old version of a way, while not being + # authorised. + def test_unredact_way_unauthorised + way = ways(:way_with_redacted_versions_v3) + + post :redact, :id => way.way_id, :version => way.version + assert_response :unauthorized, "should need to be authenticated to unredact." + end + + ## + # test the unredaction of an old version of a way, while being + # authorised as a normal user. + def test_unredact_way_normal_user + way = ways(:way_with_redacted_versions_v3) + basic_authorization(users(:public_user).email, "test") + + post :redact, :id => way.way_id, :version => way.version + assert_response :forbidden, "should need to be moderator to unredact." + end + + ## + # test the unredaction of an old version of a way, while being + # authorised as a moderator. + def test_unredact_way_moderator + way = ways(:way_with_redacted_versions_v3) + basic_authorization(users(:moderator_user).email, "test") + + post :redact, :id => way.way_id, :version => way.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 => way.way_id, :version => way.version + assert_response :success, "After redaction, node should not be gone for moderator, when flag passed." + + # and when accessed via history + get :history, :id => way.way_id + assert_response :success, "Redaction shouldn't have stopped history working." + assert_select "osm way[id='#{way.way_id}'][version='#{way.version}']", 1, "way #{way.way_id} version #{way.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 => way.way_id, :version => way.version + assert_response :success, "After redaction, node should not be gone for moderator, when flag passed." + + # and when accessed via history + get :history, :id => way.way_id + assert_response :success, "Redaction shouldn't have stopped history working." + assert_select "osm way[id='#{way.way_id}'][version='#{way.version}']", 1, "way #{way.way_id} version #{way.version} should still be present in the history for moderators when passing flag." + end + + private + ## # check that the current version of a way is equivalent to the # version which we're getting from the versions call.