test/functional/changeset_controller_test.rb

   1 require File.dirname(__FILE__) + '/../test_helper'
   2 require 'changeset_controller'
   3
   4 # Re-raise errors caught by the controller.
   5 class ChangesetController; def rescue_action(e) raise e end; end
   6
   7   class ChangesetControllerTest < Test::Unit::TestCase
   8   api_fixtures
   9
  10
  11
  12   def setup
  13     @controller = ChangesetController.new
  14     @request    = ActionController::TestRequest.new
  15     @response   = ActionController::TestResponse.new
  16   end
  17
  18   def basic_authorization(user, pass)
  19     @request.env["HTTP_AUTHORIZATION"] = "Basic %s" % Base64.encode64("#{user}:#{pass}")
  20   end
  21
  22   def content(c)
  23     @request.env["RAW_POST_DATA"] = c.to_s
  24   end
  25
  26   # -----------------------
  27   # Test simple changeset creation
  28   # -----------------------
  29
  30   def test_create
  31     basic_authorization "test@openstreetmap.org", "test"
  32
  33     # Create the first user's changeset
  34     content "<osm><changeset>" +
  35       "<tag k='created_by' v='osm test suite checking changesets'/>" +
  36       "</changeset></osm>"
  37     put :create
  38
  39     assert_response :success, "Creation of changeset did not return sucess status"
  40     newid = @response.body
  41   end
  42
  43   def test_create_invalid
  44     basic_authorization "test@openstreetmap.org", "test"
  45     content "<osm><changeset></osm>"
  46     put :create
  47     assert_response :bad_request, "creating a invalid changeset should fail"
  48   end
  49
  50   def test_read
  51
  52   end
  53
  54   def test_close
  55
  56   end
  57
  58   ##
  59   # upload something simple, but valid and check that it can
  60   # be read back ok.
  61   def test_upload_simple_valid
  62     basic_authorization "test@openstreetmap.org", "test"
  63
  64     # simple diff to change a node, way and relation by removing
  65     # their tags
  66     diff = <<EOF
  67 <osmChange>
  68  <modify>
  69   <node id='1' lon='0' lat='0' changeset='1' version='1'/>
  70   <way id='1' changeset='1' version='1'>
  71    <nd ref='3'/>
  72   </way>
  73  </modify>
  74  <modify>
  75   <relation id='1' changeset='1' version='1'>
  76    <member type='way' role='some' ref='3'/>
  77    <member type='node' role='some' ref='5'/>
  78    <member type='relation' role='some' ref='3'/>
  79   </relation>
  80  </modify>
  81 </osmChange>
  82 EOF
  83
  84     # upload it
  85     content diff
  86     post :upload, :id => 1
  87     assert_response :success,
  88       "can't upload a simple valid diff to changeset: #{@response.body}"
  89
  90     # check that the changes made it into the database
  91     assert_equal 0, Node.find(1).tags.size, "node 1 should now have no tags"
  92     assert_equal 0, Way.find(1).tags.size, "way 1 should now have no tags"
  93     assert_equal 0, Relation.find(1).tags.size, "relation 1 should now have no tags"
  94   end
  95
  96   ##
  97   # upload something which creates new objects using placeholders
  98   def test_upload_create_valid
  99     basic_authorization "test@openstreetmap.org", "test"
 100
 101     # simple diff to create a node way and relation using placeholders
 102     diff = <<EOF
 103 <osmChange>
 104  <create>
 105   <node id='-1' lon='0' lat='0' changeset='1'>
 106    <tag k='foo' v='bar'/>
 107    <tag k='baz' v='bat'/>
 108   </node>
 109   <way id='-1' changeset='1'>
 110    <nd ref='3'/>
 111   </way>
 112  </create>
 113  <create>
 114   <relation id='-1' changeset='1'>
 115    <member type='way' role='some' ref='3'/>
 116    <member type='node' role='some' ref='5'/>
 117    <member type='relation' role='some' ref='3'/>
 118   </relation>
 119  </create>
 120 </osmChange>
 121 EOF
 122
 123     # upload it
 124     content diff
 125     post :upload, :id => 1
 126     assert_response :success,
 127       "can't upload a simple valid creation to changeset: #{@response.body}"
 128
 129     # check the returned payload
 130     assert_select "osm[version=#{API_VERSION}][generator=\"OpenStreetMap server\"]", 1
 131     assert_select "osm>node", 1
 132     assert_select "osm>way", 1
 133     assert_select "osm>relation", 1
 134
 135     # inspect the response to find out what the new element IDs are
 136     doc = XML::Parser.string(@response.body).parse
 137     new_node_id = doc.find("//osm/node").first["new_id"].to_i
 138     new_way_id = doc.find("//osm/way").first["new_id"].to_i
 139     new_rel_id = doc.find("//osm/relation").first["new_id"].to_i
 140
 141     # check the old IDs are all present and negative one
 142     assert_equal -1, doc.find("//osm/node").first["old_id"].to_i
 143     assert_equal -1, doc.find("//osm/way").first["old_id"].to_i
 144     assert_equal -1, doc.find("//osm/relation").first["old_id"].to_i
 145
 146     # check the versions are present and equal one
 147     assert_equal 1, doc.find("//osm/node").first["new_version"].to_i
 148     assert_equal 1, doc.find("//osm/way").first["new_version"].to_i
 149     assert_equal 1, doc.find("//osm/relation").first["new_version"].to_i
 150
 151     # check that the changes made it into the database
 152     assert_equal 2, Node.find(new_node_id).tags.size, "new node should have two tags"
 153     assert_equal 0, Way.find(new_way_id).tags.size, "new way should have no tags"
 154     assert_equal 0, Relation.find(new_rel_id).tags.size, "new relation should have no tags"
 155   end
 156
 157   ##
 158   # test a complex delete where we delete elements which rely on eachother
 159   # in the same transaction.
 160   def test_upload_delete
 161     basic_authorization "test@openstreetmap.org", "test"
 162
 163     diff = XML::Document.new
 164     diff.root = XML::Node.new "osmChange"
 165     delete = XML::Node.new "delete"
 166     diff.root << delete
 167     delete << current_relations(:visible_relation).to_xml_node
 168     delete << current_relations(:used_relation).to_xml_node
 169     delete << current_ways(:used_way).to_xml_node
 170     delete << current_nodes(:node_used_by_relationship).to_xml_node
 171
 172     # upload it
 173     content diff
 174     post :upload, :id => 1
 175     assert_response :success,
 176       "can't upload a deletion diff to changeset: #{@response.body}"
 177
 178     # check that everything was deleted
 179     assert_equal false, Node.find(current_nodes(:node_used_by_relationship).id).visible
 180     assert_equal false, Way.find(current_ways(:used_way).id).visible
 181     assert_equal false, Relation.find(current_relations(:visible_relation).id).visible
 182     assert_equal false, Relation.find(current_relations(:used_relation).id).visible
 183   end
 184
 185   ##
 186   # test that deleting stuff in a transaction doesn't bypass the checks
 187   # to ensure that used elements are not deleted.
 188   def test_upload_delete_invalid
 189     basic_authorization "test@openstreetmap.org", "test"
 190
 191     diff = XML::Document.new
 192     diff.root = XML::Node.new "osmChange"
 193     delete = XML::Node.new "delete"
 194     diff.root << delete
 195     delete << current_relations(:visible_relation).to_xml_node
 196     delete << current_ways(:used_way).to_xml_node
 197     delete << current_nodes(:node_used_by_relationship).to_xml_node
 198
 199     # upload it
 200     content diff
 201     post :upload, :id => 1
 202     assert_response :precondition_failed,
 203       "shouldn't be able to upload a invalid deletion diff: #{@response.body}"
 204
 205     # check that nothing was, in fact, deleted
 206     assert_equal true, Node.find(current_nodes(:node_used_by_relationship).id).visible
 207     assert_equal true, Way.find(current_ways(:used_way).id).visible
 208     assert_equal true, Relation.find(current_relations(:visible_relation).id).visible
 209   end
 210
 211   ##
 212   # upload something which creates new objects and inserts them into
 213   # existing containers using placeholders.
 214   def test_upload_complex
 215     basic_authorization "test@openstreetmap.org", "test"
 216
 217     # simple diff to create a node way and relation using placeholders
 218     diff = <<EOF
 219 <osmChange>
 220  <create>
 221   <node id='-1' lon='0' lat='0' changeset='1'>
 222    <tag k='foo' v='bar'/>
 223    <tag k='baz' v='bat'/>
 224   </node>
 225  </create>
 226  <modify>
 227   <way id='1' changeset='1' version='1'>
 228    <nd ref='-1'/>
 229    <nd ref='3'/>
 230   </way>
 231   <relation id='1' changeset='1' version='1'>
 232    <member type='way' role='some' ref='3'/>
 233    <member type='node' role='some' ref='-1'/>
 234    <member type='relation' role='some' ref='3'/>
 235   </relation>
 236  </modify>
 237 </osmChange>
 238 EOF
 239
 240     # upload it
 241     content diff
 242     post :upload, :id => 1
 243     assert_response :success,
 244       "can't upload a complex diff to changeset: #{@response.body}"
 245
 246     # check the returned payload
 247     assert_select "osm[version=#{API_VERSION}][generator=\"OpenStreetMap server\"]", 1
 248     assert_select "osm>node", 1
 249     assert_select "osm>way", 1
 250     assert_select "osm>relation", 1
 251
 252     # inspect the response to find out what the new element IDs are
 253     doc = XML::Parser.string(@response.body).parse
 254     new_node_id = doc.find("//osm/node").first["new_id"].to_i
 255
 256     # check that the changes made it into the database
 257     assert_equal 2, Node.find(new_node_id).tags.size, "new node should have two tags"
 258     assert_equal [new_node_id, 3], Way.find(1).nds, "way nodes should match"
 259     Relation.find(1).members.each do |type,id,role|
 260       if type == 'node'
 261         assert_equal new_node_id, id, "relation should contain new node"
 262       end
 263     end
 264   end
 265
 266   ##
 267   # create a diff which references several changesets, which should cause
 268   # a rollback and none of the diff gets committed
 269   def test_upload_invalid_changesets
 270     basic_authorization "test@openstreetmap.org", "test"
 271
 272     # simple diff to create a node way and relation using placeholders
 273     diff = <<EOF
 274 <osmChange>
 275  <modify>
 276   <node id='1' lon='0' lat='0' changeset='1' version='1'/>
 277   <way id='1' changeset='1' version='1'>
 278    <nd ref='3'/>
 279   </way>
 280  </modify>
 281  <modify>
 282   <relation id='1' changeset='1' version='1'>
 283    <member type='way' role='some' ref='3'/>
 284    <member type='node' role='some' ref='5'/>
 285    <member type='relation' role='some' ref='3'/>
 286   </relation>
 287  </modify>
 288  <create>
 289   <node id='-1' changeset='4'>
 290    <tag k='foo' v='bar'/>
 291    <tag k='baz' v='bat'/>
 292   </node>
 293  </create>
 294 </osmChange>
 295 EOF
 296     # cache the objects before uploading them
 297     node = current_nodes(:visible_node)
 298     way = current_ways(:visible_way)
 299     rel = current_relations(:visible_relation)
 300
 301     # upload it
 302     content diff
 303     post :upload, :id => 1
 304     assert_response :conflict,
 305       "uploading a diff with multiple changsets should have failed"
 306
 307     # check that objects are unmodified
 308     assert_nodes_are_equal(node, Node.find(1))
 309     assert_ways_are_equal(way, Way.find(1))
 310   end
 311
 312   ##
 313   # upload multiple versions of the same element in the same diff.
 314   def test_upload_multiple_valid
 315     basic_authorization "test@openstreetmap.org", "test"
 316
 317     # change the location of a node multiple times, each time referencing
 318     # the last version. doesn't this depend on version numbers being
 319     # sequential?
 320     diff = <<EOF
 321 <osmChange>
 322  <modify>
 323   <node id='1' lon='0' lat='0' changeset='1' version='1'/>
 324   <node id='1' lon='1' lat='0' changeset='1' version='2'/>
 325   <node id='1' lon='1' lat='1' changeset='1' version='3'/>
 326   <node id='1' lon='1' lat='2' changeset='1' version='4'/>
 327   <node id='1' lon='2' lat='2' changeset='1' version='5'/>
 328   <node id='1' lon='3' lat='2' changeset='1' version='6'/>
 329   <node id='1' lon='3' lat='3' changeset='1' version='7'/>
 330   <node id='1' lon='9' lat='9' changeset='1' version='8'/>
 331  </modify>
 332 </osmChange>
 333 EOF
 334
 335     # upload it
 336     content diff
 337     post :upload, :id => 1
 338     assert_response :success,
 339       "can't upload multiple versions of an element in a diff: #{@response.body}"
 340   end
 341
 342   ##
 343   # upload multiple versions of the same element in the same diff, but
 344   # keep the version numbers the same.
 345   def test_upload_multiple_duplicate
 346     basic_authorization "test@openstreetmap.org", "test"
 347
 348     diff = <<EOF
 349 <osmChange>
 350  <modify>
 351   <node id='1' lon='0' lat='0' changeset='1' version='1'/>
 352   <node id='1' lon='1' lat='1' changeset='1' version='1'/>
 353  </modify>
 354 </osmChange>
 355 EOF
 356
 357     # upload it
 358     content diff
 359     post :upload, :id => 1
 360     assert_response :conflict,
 361       "shouldn't be able to upload the same element twice in a diff: #{@response.body}"
 362   end
 363
 364   ##
 365   # try to upload some elements without specifying the version
 366   def test_upload_missing_version
 367     basic_authorization "test@openstreetmap.org", "test"
 368
 369     diff = <<EOF
 370 <osmChange>
 371  <modify>
 372   <node id='1' lon='1' lat='1' changeset='1'/>
 373  </modify>
 374 </osmChange>
 375 EOF
 376
 377     # upload it
 378     content diff
 379     post :upload, :id => 1
 380     assert_response :bad_request,
 381       "shouldn't be able to upload an element without version: #{@response.body}"
 382   end
 383
 384   ##
 385   # when we make some simple changes we get the same changes back from the
 386   # diff download.
 387   def test_diff_download_simple
 388     basic_authorization(users(:normal_user).email, "test")
 389
 390     # create a temporary changeset
 391     content "<osm><changeset>" +
 392       "<tag k='created_by' v='osm test suite checking changesets'/>" +
 393       "</changeset></osm>"
 394     put :create
 395     assert_response :success
 396     changeset_id = @response.body.to_i
 397
 398     # add a diff to it
 399     diff = <<EOF
 400 <osmChange>
 401  <modify>
 402   <node id='1' lon='0' lat='0' changeset='#{changeset_id}' version='1'/>
 403   <node id='1' lon='1' lat='0' changeset='#{changeset_id}' version='2'/>
 404   <node id='1' lon='1' lat='1' changeset='#{changeset_id}' version='3'/>
 405   <node id='1' lon='1' lat='2' changeset='#{changeset_id}' version='4'/>
 406   <node id='1' lon='2' lat='2' changeset='#{changeset_id}' version='5'/>
 407   <node id='1' lon='3' lat='2' changeset='#{changeset_id}' version='6'/>
 408   <node id='1' lon='3' lat='3' changeset='#{changeset_id}' version='7'/>
 409   <node id='1' lon='9' lat='9' changeset='#{changeset_id}' version='8'/>
 410  </modify>
 411 </osmChange>
 412 EOF
 413
 414     # upload it
 415     content diff
 416     post :upload, :id => changeset_id
 417     assert_response :success,
 418       "can't upload multiple versions of an element in a diff: #{@response.body}"
 419
 420     get :download, :id => changeset_id
 421     assert_response :success
 422
 423     assert_select "osmChange", 1
 424     assert_select "osmChange>modify", 8
 425     assert_select "osmChange>modify>node", 8
 426   end
 427
 428   ##
 429   # when we make some complex changes we get the same changes back from the
 430   # diff download.
 431   def test_diff_download_complex
 432     basic_authorization(users(:normal_user).email, "test")
 433
 434     # create a temporary changeset
 435     content "<osm><changeset>" +
 436       "<tag k='created_by' v='osm test suite checking changesets'/>" +
 437       "</changeset></osm>"
 438     put :create
 439     assert_response :success
 440     changeset_id = @response.body.to_i
 441
 442     # add a diff to it
 443     diff = <<EOF
 444 <osmChange>
 445  <delete>
 446   <node id='1' lon='0' lat='0' changeset='#{changeset_id}' version='1'/>
 447  </delete>
 448  <create>
 449   <node id='-1' lon='9' lat='9' changeset='#{changeset_id}' version='0'/>
 450   <node id='-2' lon='8' lat='9' changeset='#{changeset_id}' version='0'/>
 451   <node id='-3' lon='7' lat='9' changeset='#{changeset_id}' version='0'/>
 452  </create>
 453  <modify>
 454   <node id='3' lon='20' lat='15' changeset='#{changeset_id}' version='1'/>
 455   <way id='1' changeset='#{changeset_id}' version='1'>
 456    <nd ref='3'/>
 457    <nd ref='-1'/>
 458    <nd ref='-2'/>
 459    <nd ref='-3'/>
 460   </way>
 461  </modify>
 462 </osmChange>
 463 EOF
 464
 465     # upload it
 466     content diff
 467     post :upload, :id => changeset_id
 468     assert_response :success,
 469       "can't upload multiple versions of an element in a diff: #{@response.body}"
 470
 471     get :download, :id => changeset_id
 472     assert_response :success
 473
 474     assert_select "osmChange", 1
 475     assert_select "osmChange>create", 3
 476     assert_select "osmChange>delete", 1
 477     assert_select "osmChange>modify", 2
 478     assert_select "osmChange>create>node", 3
 479     assert_select "osmChange>delete>node", 1
 480     assert_select "osmChange>modify>node", 1
 481     assert_select "osmChange>modify>way", 1
 482   end
 483
 484 end