test/controllers/old_way_controller_test.rb

   1 require "test_helper"
   2 require "old_way_controller"
   3
   4 class OldWayControllerTest < ActionController::TestCase
   5   api_fixtures
   6
   7   ##
   8   # test all routes which lead to this controller
   9   def test_routes
  10     assert_routing(
  11       { :path => "/api/0.6/way/1/history", :method => :get },
  12       { :controller => "old_way", :action => "history", :id => "1" }
  13     )
  14     assert_routing(
  15       { :path => "/api/0.6/way/1/2", :method => :get },
  16       { :controller => "old_way", :action => "version", :id => "1", :version => "2" }
  17     )
  18     assert_routing(
  19       { :path => "/api/0.6/way/1/2/redact", :method => :post },
  20       { :controller => "old_way", :action => "redact", :id => "1", :version => "2" }
  21     )
  22   end
  23
  24   # -------------------------------------
  25   # Test reading old ways.
  26   # -------------------------------------
  27
  28   def test_history_visible
  29     # check that a visible way is returned properly
  30     get :history, :id => ways(:visible_way).way_id
  31     assert_response :success
  32   end
  33
  34   def test_history_invisible
  35     # check that an invisible way's history is returned properly
  36     get :history, :id => ways(:invisible_way).way_id
  37     assert_response :success
  38   end
  39
  40   def test_history_invalid
  41     # check chat a non-existent way is not returned
  42     get :history, :id => 0
  43     assert_response :not_found
  44   end
  45
  46   ##
  47   # check that we can retrieve versions of a way
  48   def test_version
  49     create(:way_tag, :way => current_ways(:visible_way))
  50     create(:way_tag, :way => current_ways(:used_way))
  51     create(:way_tag, :way => current_ways(:way_with_versions))
  52     propagate_tags(current_ways(:visible_way), ways(:visible_way))
  53     propagate_tags(current_ways(:used_way), ways(:used_way))
  54     propagate_tags(current_ways(:way_with_versions), ways(:way_with_versions_v4))
  55
  56     check_current_version(current_ways(:visible_way).id)
  57     check_current_version(current_ways(:used_way).id)
  58     check_current_version(current_ways(:way_with_versions).id)
  59   end
  60
  61   ##
  62   # check that returned history is the same as getting all
  63   # versions of a way from the api.
  64   def test_history_equals_versions
  65     check_history_equals_versions(current_ways(:visible_way).id)
  66     check_history_equals_versions(current_ways(:used_way).id)
  67     check_history_equals_versions(current_ways(:way_with_versions).id)
  68   end
  69
  70   ##
  71   # test the redaction of an old version of a way, while not being
  72   # authorised.
  73   def test_redact_way_unauthorised
  74     do_redact_way(ways(:way_with_versions_v3),
  75                   redactions(:example))
  76     assert_response :unauthorized, "should need to be authenticated to redact."
  77   end
  78
  79   ##
  80   # test the redaction of an old version of a way, while being
  81   # authorised as a normal user.
  82   def test_redact_way_normal_user
  83     basic_authorization(users(:public_user).email, "test")
  84
  85     do_redact_way(ways(:way_with_versions_v3),
  86                   redactions(:example))
  87     assert_response :forbidden, "should need to be moderator to redact."
  88   end
  89
  90   ##
  91   # test that, even as moderator, the current version of a way
  92   # can't be redacted.
  93   def test_redact_way_current_version
  94     basic_authorization(users(:moderator_user).email, "test")
  95
  96     do_redact_way(ways(:way_with_versions_v4),
  97                   redactions(:example))
  98     assert_response :bad_request, "shouldn't be OK to redact current version as moderator."
  99   end
 100
 101   ##
 102   # test that redacted ways aren't visible, regardless of
 103   # authorisation except as moderator...
 104   def test_version_redacted
 105     way = ways(:way_with_redacted_versions_v2)
 106
 107     get :version, :id => way.way_id, :version => way.version
 108     assert_response :forbidden, "Redacted node shouldn't be visible via the version API."
 109
 110     # not even to a logged-in user
 111     basic_authorization(users(:public_user).email, "test")
 112     get :version, :id => way.way_id, :version => way.version
 113     assert_response :forbidden, "Redacted node shouldn't be visible via the version API, even when logged in."
 114   end
 115
 116   ##
 117   # test that redacted nodes aren't visible in the history
 118   def test_history_redacted
 119     way = ways(:way_with_redacted_versions_v2)
 120
 121     get :history, :id => way.way_id
 122     assert_response :success, "Redaction shouldn't have stopped history working."
 123     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."
 124
 125     # not even to a logged-in user
 126     basic_authorization(users(:public_user).email, "test")
 127     get :version, :id => way.way_id, :version => way.version
 128     get :history, :id => way.way_id
 129     assert_response :success, "Redaction shouldn't have stopped history working."
 130     assert_select "osm way[id='#{way.way_id}'][version='#{way.version}']", 0, "redacted node #{way.way_id} version #{way.version} shouldn't be present in the history, even when logged in."
 131   end
 132
 133   ##
 134   # test the redaction of an old version of a way, while being
 135   # authorised as a moderator.
 136   def test_redact_way_moderator
 137     way = ways(:way_with_versions_v3)
 138     basic_authorization(users(:moderator_user).email, "test")
 139
 140     do_redact_way(way, redactions(:example))
 141     assert_response :success, "should be OK to redact old version as moderator."
 142
 143     # check moderator can still see the redacted data, when passing
 144     # the appropriate flag
 145     get :version, :id => way.way_id, :version => way.version
 146     assert_response :forbidden, "After redaction, node should be gone for moderator, when flag not passed."
 147     get :version, :id => way.way_id, :version => way.version, :show_redactions => "true"
 148     assert_response :success, "After redaction, node should not be gone for moderator, when flag passed."
 149
 150     # and when accessed via history
 151     get :history, :id => way.way_id
 152     assert_response :success, "Redaction shouldn't have stopped history working."
 153     assert_select "osm way[id='#{way.way_id}'][version='#{way.version}']", 0, "way #{way.way_id} version #{way.version} should not be present in the history for moderators when not passing flag."
 154     get :history, :id => way.way_id, :show_redactions => "true"
 155     assert_response :success, "Redaction shouldn't have stopped history working."
 156     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."
 157   end
 158
 159   # testing that if the moderator drops auth, he can't see the
 160   # redacted stuff any more.
 161   def test_redact_way_is_redacted
 162     way = ways(:way_with_versions_v3)
 163     basic_authorization(users(:moderator_user).email, "test")
 164
 165     do_redact_way(way, redactions(:example))
 166     assert_response :success, "should be OK to redact old version as moderator."
 167
 168     # re-auth as non-moderator
 169     basic_authorization(users(:public_user).email, "test")
 170
 171     # check can't see the redacted data
 172     get :version, :id => way.way_id, :version => way.version
 173     assert_response :forbidden, "Redacted node shouldn't be visible via the version API."
 174
 175     # and when accessed via history
 176     get :history, :id => way.way_id
 177     assert_response :success, "Redaction shouldn't have stopped history working."
 178     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."
 179   end
 180
 181   ##
 182   # test the unredaction of an old version of a way, while not being
 183   # authorised.
 184   def test_unredact_way_unauthorised
 185     way = ways(:way_with_redacted_versions_v3)
 186
 187     post :redact, :id => way.way_id, :version => way.version
 188     assert_response :unauthorized, "should need to be authenticated to unredact."
 189   end
 190
 191   ##
 192   # test the unredaction of an old version of a way, while being
 193   # authorised as a normal user.
 194   def test_unredact_way_normal_user
 195     way = ways(:way_with_redacted_versions_v3)
 196     basic_authorization(users(:public_user).email, "test")
 197
 198     post :redact, :id => way.way_id, :version => way.version
 199     assert_response :forbidden, "should need to be moderator to unredact."
 200   end
 201
 202   ##
 203   # test the unredaction of an old version of a way, while being
 204   # authorised as a moderator.
 205   def test_unredact_way_moderator
 206     way = ways(:way_with_redacted_versions_v3)
 207     basic_authorization(users(:moderator_user).email, "test")
 208
 209     post :redact, :id => way.way_id, :version => way.version
 210     assert_response :success, "should be OK to unredact old version as moderator."
 211
 212     # check moderator can still see the redacted data, without passing
 213     # the appropriate flag
 214     get :version, :id => way.way_id, :version => way.version
 215     assert_response :success, "After redaction, node should not be gone for moderator, when flag passed."
 216
 217     # and when accessed via history
 218     get :history, :id => way.way_id
 219     assert_response :success, "Redaction shouldn't have stopped history working."
 220     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."
 221
 222     basic_authorization(users(:normal_user).email, "test")
 223
 224     # check normal user can now see the redacted data
 225     get :version, :id => way.way_id, :version => way.version
 226     assert_response :success, "After redaction, node should not be gone for moderator, when flag passed."
 227
 228     # and when accessed via history
 229     get :history, :id => way.way_id
 230     assert_response :success, "Redaction shouldn't have stopped history working."
 231     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."
 232   end
 233
 234   private
 235
 236   ##
 237   # check that the current version of a way is equivalent to the
 238   # version which we're getting from the versions call.
 239   def check_current_version(way_id)
 240     # get the current version
 241     current_way = with_controller(WayController.new) do
 242       get :read, :id => way_id
 243       assert_response :success, "can't get current way #{way_id}"
 244       Way.from_xml(@response.body)
 245     end
 246     assert_not_nil current_way, "getting way #{way_id} returned nil"
 247
 248     # get the "old" version of the way from the version method
 249     get :version, :id => way_id, :version => current_way.version
 250     assert_response :success, "can't get old way #{way_id}, v#{current_way.version}"
 251     old_way = Way.from_xml(@response.body)
 252
 253     # check that the ways are identical
 254     assert_ways_are_equal current_way, old_way
 255   end
 256
 257   ##
 258   # look at all the versions of the way in the history and get each version from
 259   # the versions call. check that they're the same.
 260   def check_history_equals_versions(way_id)
 261     get :history, :id => way_id
 262     assert_response :success, "can't get way #{way_id} from API"
 263     history_doc = XML::Parser.string(@response.body).parse
 264     assert_not_nil history_doc, "parsing way #{way_id} history failed"
 265
 266     history_doc.find("//osm/way").each do |way_doc|
 267       history_way = Way.from_xml_node(way_doc)
 268       assert_not_nil history_way, "parsing way #{way_id} version failed"
 269
 270       get :version, :id => way_id, :version => history_way.version
 271       assert_response :success, "couldn't get way #{way_id}, v#{history_way.version}"
 272       version_way = Way.from_xml(@response.body)
 273       assert_not_nil version_way, "failed to parse #{way_id}, v#{history_way.version}"
 274
 275       assert_ways_are_equal history_way, version_way
 276     end
 277   end
 278
 279   def do_redact_way(way, redaction)
 280     get :version, :id => way.way_id, :version => way.version
 281     assert_response :success, "should be able to get version #{way.version} of way #{way.way_id}."
 282
 283     # now redact it
 284     post :redact, :id => way.way_id, :version => way.version, :redaction => redaction.id
 285   end
 286
 287   def propagate_tags(way, old_way)
 288     way.tags.each do |k, v|
 289       create(:old_way_tag, :old_way => old_way, :k => k, :v => v)
 290     end
 291   end
 292 end