test/controllers/api/changesets/uploads_controller_test.rb

   1 require "test_helper"
   2
   3 module Api
   4   module Changesets
   5     class UploadsControllerTest < ActionDispatch::IntegrationTest
   6       ##
   7       # test all routes which lead to this controller
   8       def test_routes
   9         assert_routing(
  10           { :path => "/api/0.6/changeset/1/upload", :method => :post },
  11           { :controller => "api/changesets/uploads", :action => "create", :changeset_id => "1" }
  12         )
  13       end
  14
  15       def test_upload_when_unauthorized
  16         changeset = create(:changeset)
  17         node = create(:node, :latitude => 0, :longitude => 0)
  18
  19         diff = <<~CHANGESET
  20           <osmChange>
  21             <modify>
  22               <node id='#{node.id}' lon='1' lat='2' changeset='#{changeset.id}' version='1'/>
  23             </modify>
  24           </osmChange>
  25         CHANGESET
  26
  27         post api_changeset_upload_path(changeset), :params => diff
  28
  29         assert_response :unauthorized
  30
  31         changeset.reload
  32         assert_equal 0, changeset.num_changes
  33         node.reload
  34         assert_equal 1, node.version
  35         assert_equal 0, node.latitude
  36         assert_equal 0, node.longitude
  37       end
  38
  39       def test_upload_by_private_user
  40         user = create(:user, :data_public => false)
  41         changeset = create(:changeset, :user => user)
  42         node = create(:node, :latitude => 0, :longitude => 0)
  43
  44         diff = <<~CHANGESET
  45           <osmChange>
  46             <modify>
  47               <node id='#{node.id}' lon='1' lat='2' changeset='#{changeset.id}' version='1'/>
  48             </modify>
  49           </osmChange>
  50         CHANGESET
  51
  52         auth_header = bearer_authorization_header user
  53
  54         post api_changeset_upload_path(changeset), :params => diff, :headers => auth_header
  55
  56         assert_response :forbidden
  57
  58         changeset.reload
  59         assert_equal 0, changeset.num_changes
  60         node.reload
  61         assert_equal 1, node.version
  62         assert_equal 0, node.latitude
  63         assert_equal 0, node.longitude
  64       end
  65
  66       def test_upload_without_required_scope
  67         user = create(:user)
  68         changeset = create(:changeset, :user => user)
  69         node = create(:node, :latitude => 0, :longitude => 0)
  70
  71         diff = <<~CHANGESET
  72           <osmChange>
  73             <modify>
  74               <node id='#{node.id}' lon='1' lat='2' changeset='#{changeset.id}' version='1'/>
  75             </modify>
  76           </osmChange>
  77         CHANGESET
  78
  79         auth_header = bearer_authorization_header user, :scopes => %w[read_prefs]
  80
  81         post api_changeset_upload_path(changeset), :params => diff, :headers => auth_header
  82
  83         assert_response :forbidden
  84
  85         changeset.reload
  86         assert_equal 0, changeset.num_changes
  87         node.reload
  88         assert_equal 1, node.version
  89         assert_equal 0, node.latitude
  90         assert_equal 0, node.longitude
  91       end
  92
  93       def test_upload_with_required_scope
  94         user = create(:user)
  95         changeset = create(:changeset, :user => user)
  96         node = create(:node, :latitude => 0, :longitude => 0)
  97
  98         diff = <<~CHANGESET
  99           <osmChange>
 100             <modify>
 101               <node id='#{node.id}' lon='1' lat='2' changeset='#{changeset.id}' version='1'/>
 102             </modify>
 103           </osmChange>
 104         CHANGESET
 105
 106         auth_header = bearer_authorization_header user, :scopes => %w[write_api]
 107
 108         post api_changeset_upload_path(changeset), :params => diff, :headers => auth_header
 109
 110         assert_response :success
 111
 112         assert_dom "diffResult[version='#{Settings.api_version}'][generator='#{Settings.generator}']", 1 do
 113           assert_dom "> node", 1 do
 114             assert_dom "> @old_id", node.id.to_s
 115             assert_dom "> @new_id", node.id.to_s
 116             assert_dom "> @new_version", "2"
 117           end
 118         end
 119
 120         changeset.reload
 121         assert_equal 1, changeset.num_changes
 122         node.reload
 123         assert_equal 2, node.version
 124         assert_equal 2 * GeoRecord::SCALE, node.latitude
 125         assert_equal 1 * GeoRecord::SCALE, node.longitude
 126       end
 127
 128       # -------------------------------------
 129       # Test creating elements.
 130       # -------------------------------------
 131
 132       def test_upload_create_elements
 133         user = create(:user)
 134         changeset = create(:changeset, :user => user)
 135         node = create(:node)
 136         way = create(:way_with_nodes, :nodes_count => 2)
 137         relation = create(:relation)
 138
 139         diff = <<~CHANGESET
 140           <osmChange>
 141             <create>
 142               <node id='-1' lon='0' lat='0' changeset='#{changeset.id}'>
 143                 <tag k='foo' v='bar'/>
 144                 <tag k='baz' v='bat'/>
 145               </node>
 146               <way id='-1' changeset='#{changeset.id}'>
 147                 <nd ref='#{node.id}'/>
 148               </way>
 149             </create>
 150             <create>
 151               <relation id='-1' changeset='#{changeset.id}'>
 152                 <member type='way' role='some' ref='#{way.id}'/>
 153                 <member type='node' role='some' ref='#{node.id}'/>
 154                 <member type='relation' role='some' ref='#{relation.id}'/>
 155               </relation>
 156             </create>
 157           </osmChange>
 158         CHANGESET
 159
 160         auth_header = bearer_authorization_header user
 161
 162         post api_changeset_upload_path(changeset), :params => diff, :headers => auth_header
 163
 164         assert_response :success
 165
 166         new_node_id, new_way_id, new_rel_id = nil
 167         assert_dom "diffResult[version='#{Settings.api_version}'][generator='#{Settings.generator}']", 1 do
 168           # inspect the response to find out what the new element IDs are
 169           # check the old IDs are all present and negative one
 170           # check the versions are present and equal one
 171           assert_dom "> node", 1 do |(node_el)|
 172             new_node_id = node_el["new_id"].to_i
 173             assert_dom "> @old_id", "-1"
 174             assert_dom "> @new_version", "1"
 175           end
 176           assert_dom "> way", 1 do |(way_el)|
 177             new_way_id = way_el["new_id"].to_i
 178             assert_dom "> @old_id", "-1"
 179             assert_dom "> @new_version", "1"
 180           end
 181           assert_dom "> relation", 1 do |(rel_el)|
 182             new_rel_id = rel_el["new_id"].to_i
 183             assert_dom "> @old_id", "-1"
 184             assert_dom "> @new_version", "1"
 185           end
 186         end
 187
 188         assert_equal 2, Node.find(new_node_id).tags.size, "new node should have two tags"
 189         assert_equal 0, Way.find(new_way_id).tags.size, "new way should have no tags"
 190         assert_equal 0, Relation.find(new_rel_id).tags.size, "new relation should have no tags"
 191       end
 192
 193       ##
 194       # upload an element with a really long tag value
 195       def test_upload_create_node_with_tag_too_long
 196         changeset = create(:changeset)
 197
 198         diff = <<~CHANGESET
 199           <osmChange>
 200             <create>
 201               <node id='-1' lon='0' lat='0' changeset='#{changeset.id}'>
 202                 <tag k='foo' v='#{'x' * 256}'/>
 203               </node>
 204             </create>
 205           </osmChange>
 206         CHANGESET
 207
 208         auth_header = bearer_authorization_header changeset.user
 209
 210         assert_no_difference "Node.count" do
 211           post api_changeset_upload_path(changeset), :params => diff, :headers => auth_header
 212
 213           assert_response :bad_request
 214         end
 215       end
 216
 217       def test_upload_create_nodes_with_invalid_placeholder_reuse_in_one_action_block
 218         changeset = create(:changeset)
 219
 220         diff = <<~CHANGESET
 221           <osmChange>
 222             <create>
 223               <node id='-1' lon='0' lat='0' changeset='#{changeset.id}' version='1'/>
 224               <node id='-1' lon='1' lat='1' changeset='#{changeset.id}' version='1'/>
 225             </create>
 226           </osmChange>
 227         CHANGESET
 228
 229         auth_header = bearer_authorization_header changeset.user
 230
 231         assert_no_difference "Node.count" do
 232           post api_changeset_upload_path(changeset), :params => diff, :headers => auth_header
 233
 234           assert_response :bad_request
 235         end
 236       end
 237
 238       def test_upload_create_nodes_with_invalid_placeholder_reuse_in_two_action_blocks
 239         changeset = create(:changeset)
 240
 241         diff = <<~CHANGESET
 242           <osmChange>
 243             <create>
 244               <node id='-1' lon='0' lat='0' changeset='#{changeset.id}' version='1'/>
 245             </create>
 246             <create>
 247               <node id='-1' lon='1' lat='1' changeset='#{changeset.id}' version='1'/>
 248             </create>
 249           </osmChange>
 250         CHANGESET
 251
 252         auth_header = bearer_authorization_header changeset.user
 253
 254         assert_no_difference "Node.count" do
 255           post api_changeset_upload_path(changeset), :params => diff, :headers => auth_header
 256
 257           assert_response :bad_request
 258         end
 259       end
 260
 261       def test_upload_create_way_referring_node_placeholder_defined_later
 262         changeset = create(:changeset)
 263
 264         diff = <<~CHANGESET
 265           <osmChange>
 266             <create>
 267               <way id="-1" changeset="#{changeset.id}">
 268                 <nd ref="-1"/>
 269               </way>
 270               <node id="-1" lat="1" lon="2" changeset="#{changeset.id}"/>
 271             </create>
 272           </osmChange>
 273         CHANGESET
 274
 275         auth_header = bearer_authorization_header changeset.user
 276
 277         assert_no_difference "Node.count" do
 278           assert_no_difference "Way.count" do
 279             post api_changeset_upload_path(changeset), :params => diff, :headers => auth_header
 280
 281             assert_response :bad_request
 282           end
 283         end
 284         assert_equal "Placeholder node not found for reference -1 in way -1", @response.body
 285       end
 286
 287       def test_upload_create_way_referring_undefined_node_placeholder
 288         changeset = create(:changeset)
 289
 290         diff = <<~CHANGESET
 291           <osmChange>
 292             <create>
 293               <way id="-1" changeset="#{changeset.id}">
 294                 <nd ref="-1"/>
 295               </way>
 296             </create>
 297           </osmChange>
 298         CHANGESET
 299
 300         auth_header = bearer_authorization_header changeset.user
 301
 302         assert_no_difference "Way.count" do
 303           post api_changeset_upload_path(changeset), :params => diff, :headers => auth_header
 304
 305           assert_response :bad_request
 306         end
 307         assert_equal "Placeholder node not found for reference -1 in way -1", @response.body
 308       end
 309
 310       def test_upload_create_existing_way_referring_undefined_node_placeholder
 311         changeset = create(:changeset)
 312         way = create(:way)
 313
 314         diff = <<~CHANGESET
 315           <osmChange>
 316             <create>
 317               <way id="#{way.id}" changeset="#{changeset.id}" version="1">
 318                 <nd ref="-1"/>
 319               </way>
 320             </create>
 321           </osmChange>
 322         CHANGESET
 323
 324         auth_header = bearer_authorization_header changeset.user
 325
 326         post api_changeset_upload_path(changeset), :params => diff, :headers => auth_header
 327
 328         assert_response :bad_request
 329         assert_equal "Placeholder node not found for reference -1 in way #{way.id}", @response.body
 330
 331         way.reload
 332         assert_equal 1, way.version
 333       end
 334
 335       def test_upload_create_relation_referring_undefined_node_placeholder
 336         changeset = create(:changeset)
 337
 338         diff = <<~CHANGESET
 339           <osmChange>
 340             <create>
 341               <relation id="-1" changeset="#{changeset.id}" version="1">
 342                 <member type="node" role="foo" ref="-1"/>
 343               </relation>
 344             </create>
 345           </osmChange>
 346         CHANGESET
 347
 348         auth_header = bearer_authorization_header changeset.user
 349
 350         assert_no_difference "Relation.count" do
 351           post api_changeset_upload_path(changeset), :params => diff, :headers => auth_header
 352
 353           assert_response :bad_request
 354         end
 355         assert_equal "Placeholder Node not found for reference -1 in relation -1.", @response.body
 356       end
 357
 358       def test_upload_create_existing_relation_referring_undefined_way_placeholder
 359         changeset = create(:changeset)
 360         relation = create(:relation)
 361
 362         diff = <<~CHANGESET
 363           <osmChange>
 364             <create>
 365               <relation id="#{relation.id}" changeset="#{changeset.id}" version="1">
 366                 <member type="way" role="bar" ref="-1"/>
 367               </relation>
 368             </create>
 369           </osmChange>
 370         CHANGESET
 371
 372         auth_header = bearer_authorization_header changeset.user
 373
 374         post api_changeset_upload_path(changeset), :params => diff, :headers => auth_header
 375
 376         assert_response :bad_request
 377         assert_equal "Placeholder Way not found for reference -1 in relation #{relation.id}.", @response.body
 378
 379         relation.reload
 380         assert_equal 1, relation.version
 381       end
 382
 383       def test_upload_create_relations_with_circular_references
 384         changeset = create(:changeset)
 385
 386         diff = <<~CHANGESET
 387           <osmChange version='0.6'>
 388             <create>
 389               <relation id='-2' version='0' changeset='#{changeset.id}'>
 390                 <member type='relation' role='' ref='-4' />
 391                 <tag k='type' v='route' />
 392                 <tag k='name' v='AtoB' />
 393               </relation>
 394               <relation id='-3' version='0' changeset='#{changeset.id}'>
 395                 <tag k='type' v='route' />
 396                 <tag k='name' v='BtoA' />
 397               </relation>
 398               <relation id='-4' version='0' changeset='#{changeset.id}'>
 399                 <member type='relation' role='' ref='-2' />
 400                 <member type='relation' role='' ref='-3' />
 401                 <tag k='type' v='route_master' />
 402                 <tag k='name' v='master' />
 403               </relation>
 404             </create>
 405           </osmChange>
 406         CHANGESET
 407
 408         auth_header = bearer_authorization_header changeset.user
 409
 410         post api_changeset_upload_path(changeset), :params => diff.to_s, :headers => auth_header
 411
 412         assert_response :bad_request
 413         assert_equal "Placeholder Relation not found for reference -4 in relation -2.", @response.body
 414       end
 415
 416       # -------------------------------------
 417       # Test modifying elements.
 418       # -------------------------------------
 419
 420       def test_upload_modify_elements
 421         user = create(:user)
 422         changeset = create(:changeset, :user => user)
 423         node = create(:node, :latitude => 0, :longitude => 0)
 424         way = create(:way)
 425         relation = create(:relation)
 426         other_relation = create(:relation)
 427
 428         # create some tags, since we test that they are removed later
 429         create(:node_tag, :node => node)
 430         create(:way_tag, :way => way)
 431         create(:relation_tag, :relation => relation)
 432
 433         # simple diff to change a node, way and relation by removing their tags
 434         diff = <<~CHANGESET
 435           <osmChange>
 436             <modify>
 437               <node id='#{node.id}' lon='1' lat='2' changeset='#{changeset.id}' version='1'/>
 438               <way id='#{way.id}' changeset='#{changeset.id}' version='1'>
 439                 <nd ref='#{node.id}'/>
 440               </way>
 441             </modify>
 442             <modify>
 443               <relation id='#{relation.id}' changeset='#{changeset.id}' version='1'>
 444                 <member type='way' role='some' ref='#{way.id}'/>
 445                 <member type='node' role='some' ref='#{node.id}'/>
 446                 <member type='relation' role='some' ref='#{other_relation.id}'/>
 447               </relation>
 448             </modify>
 449           </osmChange>
 450         CHANGESET
 451
 452         auth_header = bearer_authorization_header user
 453
 454         post api_changeset_upload_path(changeset), :params => diff, :headers => auth_header
 455
 456         assert_response :success
 457
 458         assert_dom "diffResult[version='#{Settings.api_version}'][generator='#{Settings.generator}']", 1 do
 459           assert_dom "> node", 1 do
 460             assert_dom "> @old_id", node.id.to_s
 461             assert_dom "> @new_id", node.id.to_s
 462             assert_dom "> @new_version", "2"
 463           end
 464           assert_dom "> way", 1 do
 465             assert_dom "> @old_id", way.id.to_s
 466             assert_dom "> @new_id", way.id.to_s
 467             assert_dom "> @new_version", "2"
 468           end
 469           assert_dom "> relation", 1 do
 470             assert_dom "> @old_id", relation.id.to_s
 471             assert_dom "> @new_id", relation.id.to_s
 472             assert_dom "> @new_version", "2"
 473           end
 474         end
 475
 476         changeset.reload
 477         assert_equal 3, changeset.num_changes
 478         node.reload
 479         assert_equal 2, node.version
 480         assert_equal 2 * GeoRecord::SCALE, node.latitude
 481         assert_equal 1 * GeoRecord::SCALE, node.longitude
 482         assert_equal 0, node.tags.size, "node #{node.id} should now have no tags"
 483         way.reload
 484         assert_equal 2, way.version
 485         assert_equal 0, way.tags.size, "way #{way.id} should now have no tags"
 486         assert_equal [node], way.nodes
 487         relation.reload
 488         assert_equal 2, relation.version
 489         assert_equal 0, relation.tags.size, "relation #{relation.id} should now have no tags"
 490         assert_equal [["Way", way.id, "some"], ["Node", node.id, "some"], ["Relation", other_relation.id, "some"]], relation.members
 491       end
 492     end
 493   end
 494 end