test/controllers/api/relations_controller_test.rb

   1 require "test_helper"
   2 require_relative "elements_test_helper"
   3
   4 module Api
   5   class RelationsControllerTest < ActionDispatch::IntegrationTest
   6     include ElementsTestHelper
   7
   8     ##
   9     # test all routes which lead to this controller
  10     def test_routes
  11       assert_routing(
  12         { :path => "/api/0.6/relations", :method => :get },
  13         { :controller => "api/relations", :action => "index" }
  14       )
  15       assert_routing(
  16         { :path => "/api/0.6/relations.json", :method => :get },
  17         { :controller => "api/relations", :action => "index", :format => "json" }
  18       )
  19       assert_routing(
  20         { :path => "/api/0.6/relations", :method => :post },
  21         { :controller => "api/relations", :action => "create" }
  22       )
  23       assert_routing(
  24         { :path => "/api/0.6/relation/1", :method => :get },
  25         { :controller => "api/relations", :action => "show", :id => "1" }
  26       )
  27       assert_routing(
  28         { :path => "/api/0.6/relation/1.json", :method => :get },
  29         { :controller => "api/relations", :action => "show", :id => "1", :format => "json" }
  30       )
  31       assert_routing(
  32         { :path => "/api/0.6/relation/1/full", :method => :get },
  33         { :controller => "api/relations", :action => "show", :full => true, :id => "1" }
  34       )
  35       assert_routing(
  36         { :path => "/api/0.6/relation/1/full.json", :method => :get },
  37         { :controller => "api/relations", :action => "show", :full => true, :id => "1", :format => "json" }
  38       )
  39       assert_routing(
  40         { :path => "/api/0.6/relation/1", :method => :put },
  41         { :controller => "api/relations", :action => "update", :id => "1" }
  42       )
  43       assert_routing(
  44         { :path => "/api/0.6/relation/1", :method => :delete },
  45         { :controller => "api/relations", :action => "destroy", :id => "1" }
  46       )
  47
  48       assert_recognizes(
  49         { :controller => "api/relations", :action => "create" },
  50         { :path => "/api/0.6/relation/create", :method => :put }
  51       )
  52     end
  53
  54     ##
  55     # test fetching multiple relations
  56     def test_index
  57       relation1 = create(:relation)
  58       relation2 = create(:relation, :deleted)
  59       relation3 = create(:relation, :with_history, :version => 2)
  60       relation4 = create(:relation, :with_history, :version => 2)
  61       relation4.old_relations.find_by(:version => 1).redact!(create(:redaction))
  62
  63       # check error when no parameter provided
  64       get api_relations_path
  65       assert_response :bad_request
  66
  67       # check error when no parameter value provided
  68       get api_relations_path(:relations => "")
  69       assert_response :bad_request
  70
  71       # test a working call
  72       get api_relations_path(:relations => "#{relation1.id},#{relation2.id},#{relation3.id},#{relation4.id}")
  73       assert_response :success
  74       assert_select "osm" do
  75         assert_select "relation", :count => 4
  76         assert_select "relation[id='#{relation1.id}'][visible='true']", :count => 1
  77         assert_select "relation[id='#{relation2.id}'][visible='false']", :count => 1
  78         assert_select "relation[id='#{relation3.id}'][visible='true']", :count => 1
  79         assert_select "relation[id='#{relation4.id}'][visible='true']", :count => 1
  80       end
  81
  82       # test a working call with json format
  83       get api_relations_path(:relations => "#{relation1.id},#{relation2.id},#{relation3.id},#{relation4.id}", :format => "json")
  84
  85       js = ActiveSupport::JSON.decode(@response.body)
  86       assert_not_nil js
  87       assert_equal 4, js["elements"].count
  88       assert_equal 4, (js["elements"].count { |a| a["type"] == "relation" })
  89       assert_equal 1, (js["elements"].count { |a| a["id"] == relation1.id && a["visible"].nil? })
  90       assert_equal 1, (js["elements"].count { |a| a["id"] == relation2.id && a["visible"] == false })
  91       assert_equal 1, (js["elements"].count { |a| a["id"] == relation3.id && a["visible"].nil? })
  92       assert_equal 1, (js["elements"].count { |a| a["id"] == relation4.id && a["visible"].nil? })
  93
  94       # check error when a non-existent relation is included
  95       get api_relations_path(:relations => "#{relation1.id},#{relation2.id},#{relation3.id},#{relation4.id},0")
  96       assert_response :not_found
  97     end
  98
  99     # -------------------------------------
 100     # Test showing relations.
 101     # -------------------------------------
 102
 103     def test_show_not_found
 104       get api_relation_path(0)
 105       assert_response :not_found
 106     end
 107
 108     def test_show_deleted
 109       get api_relation_path(create(:relation, :deleted))
 110       assert_response :gone
 111     end
 112
 113     def test_show
 114       relation = create(:relation, :timestamp => "2021-02-03T00:00:00Z")
 115       node = create(:node, :timestamp => "2021-04-05T00:00:00Z")
 116       create(:relation_member, :relation => relation, :member => node)
 117
 118       get api_relation_path(relation)
 119
 120       assert_response :success
 121       assert_not_nil @response.header["Last-Modified"]
 122       assert_equal "2021-02-03T00:00:00Z", Time.parse(@response.header["Last-Modified"]).utc.xmlschema
 123       assert_dom "node", :count => 0
 124       assert_dom "relation", :count => 1 do
 125         assert_dom "> @id", :text => relation.id.to_s
 126       end
 127     end
 128
 129     def test_full_not_found
 130       get api_relation_path(999999, :full => true)
 131       assert_response :not_found
 132     end
 133
 134     def test_full_deleted
 135       get api_relation_path(create(:relation, :deleted), :full => true)
 136       assert_response :gone
 137     end
 138
 139     def test_full_empty
 140       relation = create(:relation)
 141
 142       get api_relation_path(relation, :full => true)
 143
 144       assert_response :success
 145       assert_dom "relation", :count => 1 do
 146         assert_dom "> @id", :text => relation.id.to_s
 147       end
 148     end
 149
 150     def test_full_with_node_member
 151       relation = create(:relation)
 152       node = create(:node)
 153       create(:relation_member, :relation => relation, :member => node)
 154
 155       get api_relation_path(relation, :full => true)
 156
 157       assert_response :success
 158       assert_dom "node", :count => 1 do
 159         assert_dom "> @id", :text => node.id.to_s
 160       end
 161       assert_dom "relation", :count => 1 do
 162         assert_dom "> @id", :text => relation.id.to_s
 163       end
 164     end
 165
 166     def test_full_with_way_member
 167       relation = create(:relation)
 168       way = create(:way_with_nodes)
 169       create(:relation_member, :relation => relation, :member => way)
 170
 171       get api_relation_path(relation, :full => true)
 172
 173       assert_response :success
 174       assert_dom "node", :count => 1 do
 175         assert_dom "> @id", :text => way.nodes[0].id.to_s
 176       end
 177       assert_dom "way", :count => 1 do
 178         assert_dom "> @id", :text => way.id.to_s
 179       end
 180       assert_dom "relation", :count => 1 do
 181         assert_dom "> @id", :text => relation.id.to_s
 182       end
 183     end
 184
 185     def test_full_with_node_member_json
 186       relation = create(:relation)
 187       node = create(:node)
 188       create(:relation_member, :relation => relation, :member => node)
 189
 190       get api_relation_path(relation, :full => true, :format => "json")
 191
 192       assert_response :success
 193       js = ActiveSupport::JSON.decode(@response.body)
 194       assert_not_nil js
 195       assert_equal 2, js["elements"].count
 196
 197       js_relations = js["elements"].filter { |e| e["type"] == "relation" }
 198       assert_equal 1, js_relations.count
 199       assert_equal relation.id, js_relations[0]["id"]
 200       assert_equal 1, js_relations[0]["members"].count
 201       assert_equal "node", js_relations[0]["members"][0]["type"]
 202       assert_equal node.id, js_relations[0]["members"][0]["ref"]
 203
 204       js_nodes = js["elements"].filter { |e| e["type"] == "node" }
 205       assert_equal 1, js_nodes.count
 206       assert_equal node.id, js_nodes[0]["id"]
 207     end
 208
 209     # -------------------------------------
 210     # Test creating relations.
 211     # -------------------------------------
 212
 213     def test_create_without_members_by_private_user
 214       with_unchanging_request([:data_public => false]) do |headers, changeset|
 215         osm = <<~OSM
 216           <osm>
 217             <relation changeset='#{changeset.id}'>
 218               <tag k='test' v='yes' />
 219             </relation>
 220           </osm>
 221         OSM
 222
 223         post api_relations_path, :params => osm, :headers => headers
 224
 225         assert_response :forbidden, "relation upload should have failed with forbidden"
 226       end
 227     end
 228
 229     def test_create_with_node_member_with_role_by_private_user
 230       node = create(:node)
 231
 232       with_unchanging_request([:data_public => false]) do |headers, changeset|
 233         osm = <<~OSM
 234           <osm>
 235             <relation changeset='#{changeset.id}'>
 236               <member ref='#{node.id}' type='node' role='some'/>
 237               <tag k='test' v='yes' />
 238             </relation>
 239           </osm>
 240         OSM
 241
 242         post api_relations_path, :params => osm, :headers => headers
 243
 244         assert_response :forbidden, "relation upload did not return forbidden status"
 245       end
 246     end
 247
 248     def test_create_with_node_member_without_role_by_private_user
 249       node = create(:node)
 250
 251       with_unchanging_request([:data_public => false]) do |headers, changeset|
 252         osm = <<~OSM
 253           <osm>
 254             <relation changeset='#{changeset.id}'>
 255               <member ref='#{node.id}' type='node'/>
 256               <tag k='test' v='yes' />
 257             </relation>
 258           </osm>
 259         OSM
 260
 261         post api_relations_path, :params => osm, :headers => headers
 262
 263         assert_response :forbidden, "relation upload did not return forbidden status"
 264       end
 265     end
 266
 267     def test_create_with_node_and_way_members_by_private_user
 268       node = create(:node)
 269       way = create(:way_with_nodes, :nodes_count => 2)
 270
 271       with_unchanging_request([:data_public => false]) do |headers, changeset|
 272         osm = <<~OSM
 273           <osm>
 274             <relation changeset='#{changeset.id}'>
 275               <member type='node' ref='#{node.id}' role='some'/>
 276               <member type='way' ref='#{way.id}' role='other'/>
 277               <tag k='test' v='yes' />
 278             </relation>
 279           </osm>
 280         OSM
 281
 282         post api_relations_path, :params => osm, :headers => headers
 283
 284         assert_response :forbidden, "relation upload did not return success status"
 285       end
 286     end
 287
 288     def test_create_without_members
 289       with_request do |headers, changeset|
 290         assert_difference "Relation.count" => 1,
 291                           "RelationMember.count" => 0 do
 292           osm = <<~OSM
 293             <osm>
 294               <relation changeset='#{changeset.id}'>
 295                 <tag k='test' v='yes' />
 296               </relation>
 297             </osm>
 298           OSM
 299
 300           post api_relations_path, :params => osm, :headers => headers
 301
 302           assert_response :success, "relation upload did not return success status"
 303         end
 304
 305         created_relation_id = @response.body
 306         relation = Relation.find(created_relation_id)
 307         assert_empty relation.members
 308         assert_equal({ "test" => "yes" }, relation.tags)
 309         assert_equal changeset.id, relation.changeset_id, "saved relation does not belong in the changeset it was assigned to"
 310         assert relation.visible, "saved relation is not visible"
 311       end
 312     end
 313
 314     def test_create_with_node_member_with_role
 315       node = create(:node)
 316
 317       with_request do |headers, changeset|
 318         assert_difference "Relation.count" => 1,
 319                           "RelationMember.count" => 1 do
 320           osm = <<~OSM
 321             <osm>
 322               <relation changeset='#{changeset.id}'>
 323                 <member ref='#{node.id}' type='node' role='some'/>
 324                 <tag k='test' v='yes' />
 325               </relation>
 326             </osm>
 327           OSM
 328
 329           post api_relations_path, :params => osm, :headers => headers
 330
 331           assert_response :success, "relation upload did not return success status"
 332         end
 333
 334         created_relation_id = @response.body
 335         relation = Relation.find(created_relation_id)
 336         assert_equal [["Node", node.id, "some"]], relation.members
 337         assert_equal({ "test" => "yes" }, relation.tags)
 338         assert_equal changeset.id, relation.changeset_id, "saved relation does not belong in the changeset it was assigned to"
 339         assert relation.visible, "saved relation is not visible"
 340       end
 341     end
 342
 343     def test_create_with_node_member_without_role
 344       node = create(:node)
 345
 346       with_request do |headers, changeset|
 347         assert_difference "Relation.count" => 1,
 348                           "RelationMember.count" => 1 do
 349           osm = <<~OSM
 350             <osm>
 351               <relation changeset='#{changeset.id}'>
 352                 <member ref='#{node.id}' type='node'/>
 353                 <tag k='test' v='yes' />
 354               </relation>
 355             </osm>
 356           OSM
 357
 358           post api_relations_path, :params => osm, :headers => headers
 359
 360           assert_response :success, "relation upload did not return success status"
 361         end
 362
 363         created_relation_id = @response.body
 364         relation = Relation.find(created_relation_id)
 365         assert_equal [["Node", node.id, ""]], relation.members
 366         assert_equal({ "test" => "yes" }, relation.tags)
 367         assert_equal changeset.id, relation.changeset_id, "saved relation does not belong in the changeset it was assigned to"
 368         assert relation.visible, "saved relation is not visible"
 369       end
 370     end
 371
 372     def test_create_with_node_and_way_members
 373       node = create(:node)
 374       way = create(:way_with_nodes, :nodes_count => 2)
 375
 376       with_request do |headers, changeset|
 377         assert_difference "Relation.count" => 1,
 378                           "RelationMember.count" => 2 do
 379           osm = <<~OSM
 380             <osm>
 381               <relation changeset='#{changeset.id}'>
 382                 <member type='node' ref='#{node.id}' role='some'/>
 383                 <member type='way' ref='#{way.id}' role='other'/>
 384                 <tag k='test' v='yes' />
 385               </relation>
 386             </osm>
 387           OSM
 388
 389           post api_relations_path, :params => osm, :headers => headers
 390
 391           assert_response :success, "relation upload did not return success status"
 392         end
 393
 394         created_relation_id = @response.body
 395         relation = Relation.find(created_relation_id)
 396         assert_equal [["Node", node.id, "some"],
 397                       ["Way", way.id, "other"]], relation.members
 398         assert_equal({ "test" => "yes" }, relation.tags)
 399         assert_equal changeset.id, relation.changeset_id, "saved relation does not belong in the changeset it was assigned to"
 400         assert relation.visible, "saved relation is not visible"
 401       end
 402     end
 403
 404     def test_create_with_missing_node_member
 405       with_unchanging_request do |headers, changeset|
 406         osm = <<~OSM
 407           <osm>
 408             <relation changeset='#{changeset.id}'>
 409               <member type='node' ref='0'/>
 410             </relation>
 411           </osm>
 412         OSM
 413
 414         post api_relations_path, :params => osm, :headers => headers
 415
 416         assert_response :precondition_failed, "relation upload with invalid node did not return 'precondition failed'"
 417         assert_equal "Precondition failed: Relation with id  cannot be saved due to Node with id 0", @response.body
 418       end
 419     end
 420
 421     def test_create_and_show
 422       user = create(:user)
 423       changeset = create(:changeset, :user => user)
 424
 425       osm = <<~OSM
 426         <osm>
 427           <relation changeset='#{changeset.id}'/>
 428         </osm>
 429       OSM
 430
 431       post api_relations_path, :params => osm, :headers => bearer_authorization_header(user)
 432
 433       assert_response :success, "relation upload did not return success status"
 434
 435       created_relation_id = @response.body
 436
 437       get api_relation_path(created_relation_id)
 438
 439       assert_response :success
 440     end
 441
 442     # ------------------------------------
 443     # Test updating relations
 444     # ------------------------------------
 445
 446     def test_update_wrong_id
 447       user = create(:user)
 448       changeset = create(:changeset, :user => user)
 449       relation = create(:relation)
 450       other_relation = create(:relation)
 451
 452       auth_header = bearer_authorization_header user
 453       get api_relation_path(relation)
 454       assert_response :success
 455       rel = XML::Parser.string(@response.body).parse
 456
 457       update_changeset(rel, changeset.id)
 458       put api_relation_path(other_relation), :params => rel.to_s, :headers => auth_header
 459       assert_response :bad_request
 460     end
 461
 462     # -------------------------------------
 463     # Test creating a relation, with some invalid XML
 464     # -------------------------------------
 465     def test_create_invalid_xml
 466       user = create(:user)
 467       changeset = create(:changeset, :user => user)
 468       node = create(:node)
 469
 470       auth_header = bearer_authorization_header user
 471
 472       # create some xml that should return an error
 473       xml = <<~OSM
 474         <osm>
 475           <relation changeset='#{changeset.id}'>
 476             <member type='type' ref='#{node.id}' role=''/>
 477           </relation>
 478         </osm>
 479       OSM
 480       post api_relations_path, :params => xml, :headers => auth_header
 481       # expect failure
 482       assert_response :bad_request
 483       assert_match(/Cannot parse valid relation from xml string/, @response.body)
 484       assert_match(/The type is not allowed only, /, @response.body)
 485     end
 486
 487     # -------------------------------------
 488     # Test deleting relations.
 489     # -------------------------------------
 490
 491     def test_destroy
 492       private_user = create(:user, :data_public => false)
 493       private_user_closed_changeset = create(:changeset, :closed, :user => private_user)
 494       user = create(:user)
 495       closed_changeset = create(:changeset, :closed, :user => user)
 496       changeset = create(:changeset, :user => user)
 497       relation = create(:relation)
 498       used_relation = create(:relation)
 499       super_relation = create(:relation_member, :member => used_relation).relation
 500       deleted_relation = create(:relation, :deleted)
 501       multi_tag_relation = create(:relation)
 502       create_list(:relation_tag, 4, :relation => multi_tag_relation)
 503
 504       ## First try to delete relation without auth
 505       delete api_relation_path(relation)
 506       assert_response :unauthorized
 507
 508       ## Then try with the private user, to make sure that you get a forbidden
 509       auth_header = bearer_authorization_header private_user
 510
 511       # this shouldn't work, as we should need the payload...
 512       delete api_relation_path(relation), :headers => auth_header
 513       assert_response :forbidden
 514
 515       # try to delete without specifying a changeset
 516       xml = "<osm><relation id='#{relation.id}'/></osm>"
 517       delete api_relation_path(relation), :params => xml.to_s, :headers => auth_header
 518       assert_response :forbidden
 519
 520       # try to delete with an invalid (closed) changeset
 521       xml = update_changeset(xml_for_relation(relation),
 522                              private_user_closed_changeset.id)
 523       delete api_relation_path(relation), :params => xml.to_s, :headers => auth_header
 524       assert_response :forbidden
 525
 526       # try to delete with an invalid (non-existent) changeset
 527       xml = update_changeset(xml_for_relation(relation), 0)
 528       delete api_relation_path(relation), :params => xml.to_s, :headers => auth_header
 529       assert_response :forbidden
 530
 531       # this won't work because the relation is in-use by another relation
 532       xml = xml_for_relation(used_relation)
 533       delete api_relation_path(used_relation), :params => xml.to_s, :headers => auth_header
 534       assert_response :forbidden
 535
 536       # this should work when we provide the appropriate payload...
 537       xml = xml_for_relation(relation)
 538       delete api_relation_path(relation), :params => xml.to_s, :headers => auth_header
 539       assert_response :forbidden
 540
 541       # this won't work since the relation is already deleted
 542       xml = xml_for_relation(deleted_relation)
 543       delete api_relation_path(deleted_relation), :params => xml.to_s, :headers => auth_header
 544       assert_response :forbidden
 545
 546       # this won't work since the relation never existed
 547       delete api_relation_path(0), :headers => auth_header
 548       assert_response :forbidden
 549
 550       ## now set auth for the public user
 551       auth_header = bearer_authorization_header user
 552
 553       # this shouldn't work, as we should need the payload...
 554       delete api_relation_path(relation), :headers => auth_header
 555       assert_response :bad_request
 556
 557       # try to delete without specifying a changeset
 558       xml = "<osm><relation id='#{relation.id}' version='#{relation.version}' /></osm>"
 559       delete api_relation_path(relation), :params => xml.to_s, :headers => auth_header
 560       assert_response :bad_request
 561       assert_match(/Changeset id is missing/, @response.body)
 562
 563       # try to delete with an invalid (closed) changeset
 564       xml = update_changeset(xml_for_relation(relation),
 565                              closed_changeset.id)
 566       delete api_relation_path(relation), :params => xml.to_s, :headers => auth_header
 567       assert_response :conflict
 568
 569       # try to delete with an invalid (non-existent) changeset
 570       xml = update_changeset(xml_for_relation(relation), 0)
 571       delete api_relation_path(relation), :params => xml.to_s, :headers => auth_header
 572       assert_response :conflict
 573
 574       # this won't work because the relation is in a changeset owned by someone else
 575       xml = update_changeset(xml_for_relation(relation), create(:changeset).id)
 576       delete api_relation_path(relation), :params => xml.to_s, :headers => auth_header
 577       assert_response :conflict,
 578                       "shouldn't be able to delete a relation in a changeset owned by someone else (#{@response.body})"
 579
 580       # this won't work because the relation in the payload is different to that passed
 581       xml = update_changeset(xml_for_relation(relation), changeset.id)
 582       delete api_relation_path(create(:relation)), :params => xml.to_s, :headers => auth_header
 583       assert_response :bad_request, "shouldn't be able to delete a relation when payload is different to the url"
 584
 585       # this won't work because the relation is in-use by another relation
 586       xml = update_changeset(xml_for_relation(used_relation), changeset.id)
 587       delete api_relation_path(used_relation), :params => xml.to_s, :headers => auth_header
 588       assert_response :precondition_failed,
 589                       "shouldn't be able to delete a relation used in a relation (#{@response.body})"
 590       assert_equal "Precondition failed: The relation #{used_relation.id} is used in relation #{super_relation.id}.", @response.body
 591
 592       # this should work when we provide the appropriate payload...
 593       xml = update_changeset(xml_for_relation(multi_tag_relation), changeset.id)
 594       delete api_relation_path(multi_tag_relation), :params => xml.to_s, :headers => auth_header
 595       assert_response :success
 596
 597       # valid delete should return the new version number, which should
 598       # be greater than the old version number
 599       assert_operator @response.body.to_i, :>, multi_tag_relation.version, "delete request should return a new version number for relation"
 600
 601       # this won't work since the relation is already deleted
 602       xml = update_changeset(xml_for_relation(deleted_relation), changeset.id)
 603       delete api_relation_path(deleted_relation), :params => xml.to_s, :headers => auth_header
 604       assert_response :gone
 605
 606       # Public visible relation needs to be deleted
 607       xml = update_changeset(xml_for_relation(super_relation), changeset.id)
 608       delete api_relation_path(super_relation), :params => xml.to_s, :headers => auth_header
 609       assert_response :success
 610
 611       # this works now because the relation which was using this one
 612       # has been deleted.
 613       xml = update_changeset(xml_for_relation(used_relation), changeset.id)
 614       delete api_relation_path(used_relation), :params => xml.to_s, :headers => auth_header
 615       assert_response :success,
 616                       "should be able to delete a relation used in an old relation (#{@response.body})"
 617
 618       # this won't work since the relation never existed
 619       delete api_relation_path(0), :headers => auth_header
 620       assert_response :not_found
 621     end
 622
 623     ##
 624     # test initial rate limit
 625     def test_initial_rate_limit
 626       # create a user
 627       user = create(:user)
 628
 629       # create some nodes
 630       node1 = create(:node)
 631       node2 = create(:node)
 632
 633       # create a changeset that puts us near the initial rate limit
 634       changeset = create(:changeset, :user => user,
 635                                      :created_at => Time.now.utc - 5.minutes,
 636                                      :num_changes => Settings.initial_changes_per_hour - 1)
 637
 638       # create authentication header
 639       auth_header = bearer_authorization_header user
 640
 641       # try creating a relation
 642       xml = <<~OSM
 643         <osm>
 644           <relation changeset='#{changeset.id}'>
 645             <member ref='#{node1.id}' type='node' role='some'/>
 646             <member ref='#{node2.id}' type='node' role='some'/>
 647           </relation>
 648         </osm>
 649       OSM
 650       post api_relations_path, :params => xml, :headers => auth_header
 651       assert_response :success, "relation create did not return success status"
 652
 653       # get the id of the relation we created
 654       relationid = @response.body
 655
 656       # try updating the relation, which should be rate limited
 657       xml = <<~OSM
 658         <osm>
 659           <relation id='#{relationid}' version='1' changeset='#{changeset.id}'>
 660             <member ref='#{node2.id}' type='node' role='some'/>
 661             <member ref='#{node1.id}' type='node' role='some'/>
 662           </relation>
 663         </osm>
 664       OSM
 665       put api_relation_path(relationid), :params => xml, :headers => auth_header
 666       assert_response :too_many_requests, "relation update did not hit rate limit"
 667
 668       # try deleting the relation, which should be rate limited
 669       xml = "<osm><relation id='#{relationid}' version='2' changeset='#{changeset.id}'/></osm>"
 670       delete api_relation_path(relationid), :params => xml, :headers => auth_header
 671       assert_response :too_many_requests, "relation delete did not hit rate limit"
 672
 673       # try creating a relation, which should be rate limited
 674       xml = <<~OSM
 675         <osm>
 676           <relation changeset='#{changeset.id}'>
 677             <member ref='#{node1.id}' type='node' role='some'/>
 678             <member ref='#{node2.id}' type='node' role='some'/>
 679           </relation>
 680         </osm>
 681       OSM
 682       post api_relations_path, :params => xml, :headers => auth_header
 683       assert_response :too_many_requests, "relation create did not hit rate limit"
 684     end
 685
 686     ##
 687     # test maximum rate limit
 688     def test_maximum_rate_limit
 689       # create a user
 690       user = create(:user)
 691
 692       # create some nodes
 693       node1 = create(:node)
 694       node2 = create(:node)
 695
 696       # create a changeset to establish our initial edit time
 697       changeset = create(:changeset, :user => user,
 698                                      :created_at => Time.now.utc - 28.days)
 699
 700       # create changeset to put us near the maximum rate limit
 701       total_changes = Settings.max_changes_per_hour - 1
 702       while total_changes.positive?
 703         changes = [total_changes, Changeset::MAX_ELEMENTS].min
 704         changeset = create(:changeset, :user => user,
 705                                        :created_at => Time.now.utc - 5.minutes,
 706                                        :num_changes => changes)
 707         total_changes -= changes
 708       end
 709
 710       # create authentication header
 711       auth_header = bearer_authorization_header user
 712
 713       # try creating a relation
 714       xml = <<~OSM
 715         <osm>
 716           <relation changeset='#{changeset.id}'>
 717             <member ref='#{node1.id}' type='node' role='some'/>
 718             <member ref='#{node2.id}' type='node' role='some'/>
 719           </relation>
 720         </osm>
 721       OSM
 722       post api_relations_path, :params => xml, :headers => auth_header
 723       assert_response :success, "relation create did not return success status"
 724
 725       # get the id of the relation we created
 726       relationid = @response.body
 727
 728       # try updating the relation, which should be rate limited
 729       xml = <<~OSM
 730         <osm>
 731           <relation id='#{relationid}' version='1' changeset='#{changeset.id}'>
 732             <member ref='#{node2.id}' type='node' role='some'/>
 733             <member ref='#{node1.id}' type='node' role='some'/>
 734           </relation>
 735         </osm>
 736       OSM
 737       put api_relation_path(relationid), :params => xml, :headers => auth_header
 738       assert_response :too_many_requests, "relation update did not hit rate limit"
 739
 740       # try deleting the relation, which should be rate limited
 741       xml = "<osm><relation id='#{relationid}' version='2' changeset='#{changeset.id}'/></osm>"
 742       delete api_relation_path(relationid), :params => xml, :headers => auth_header
 743       assert_response :too_many_requests, "relation delete did not hit rate limit"
 744
 745       # try creating a relation, which should be rate limited
 746       xml = <<~OSM
 747         <osm>
 748           <relation changeset='#{changeset.id}'>
 749             <member ref='#{node1.id}' type='node' role='some'/>
 750             <member ref='#{node2.id}' type='node' role='some'/>
 751           </relation>
 752         </osm>
 753       OSM
 754       post api_relations_path, :params => xml, :headers => auth_header
 755       assert_response :too_many_requests, "relation create did not hit rate limit"
 756     end
 757
 758     private
 759
 760     def affected_models
 761       [Relation, RelationTag, RelationMember,
 762        OldRelation, OldRelationTag, OldRelationMember]
 763     end
 764
 765     ##
 766     # update an attribute in the node element
 767     def xml_attr_rewrite(xml, name, value)
 768       xml.find("//osm/relation").first[name] = value.to_s
 769       xml
 770     end
 771   end
 772 end