test/controllers/way_controller_test.rb

   1 require "test_helper"
   2 require "way_controller"
   3
   4 class WayControllerTest < 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/create", :method => :put },
  12       { :controller => "way", :action => "create" }
  13     )
  14     assert_routing(
  15       { :path => "/api/0.6/way/1/full", :method => :get },
  16       { :controller => "way", :action => "full", :id => "1" }
  17     )
  18     assert_routing(
  19       { :path => "/api/0.6/way/1", :method => :get },
  20       { :controller => "way", :action => "read", :id => "1" }
  21     )
  22     assert_routing(
  23       { :path => "/api/0.6/way/1", :method => :put },
  24       { :controller => "way", :action => "update", :id => "1" }
  25     )
  26     assert_routing(
  27       { :path => "/api/0.6/way/1", :method => :delete },
  28       { :controller => "way", :action => "delete", :id => "1" }
  29     )
  30     assert_routing(
  31       { :path => "/api/0.6/ways", :method => :get },
  32       { :controller => "way", :action => "ways" }
  33     )
  34   end
  35
  36   # -------------------------------------
  37   # Test reading ways.
  38   # -------------------------------------
  39
  40   def test_read
  41     # check that a visible way is returned properly
  42     get :read, :id => current_ways(:visible_way).id
  43     assert_response :success
  44
  45     # check that an invisible way is not returned
  46     get :read, :id => current_ways(:invisible_way).id
  47     assert_response :gone
  48
  49     # check chat a non-existent way is not returned
  50     get :read, :id => 0
  51     assert_response :not_found
  52   end
  53
  54   ##
  55   # check the "full" mode
  56   def test_full
  57     Way.all.each do |way|
  58       get :full, :id => way.id
  59
  60       # full call should say "gone" for non-visible ways...
  61       unless way.visible
  62         assert_response :gone
  63         next
  64       end
  65
  66       # otherwise it should say success
  67       assert_response :success
  68
  69       # Check the way is correctly returned
  70       assert_select "osm way[id='#{way.id}'][version='#{way.version}'][visible='#{way.visible}']", 1
  71
  72       # check that each node in the way appears once in the output as a
  73       # reference and as the node element.
  74       way.nodes.each do |n|
  75         count = (way.nodes - (way.nodes - [n])).length
  76         assert_select "osm way nd[ref='#{n.id}']", count
  77         assert_select "osm node[id='#{n.id}'][version='#{n.version}'][lat='#{n.lat}'][lon='#{n.lon}']", 1
  78       end
  79     end
  80   end
  81
  82   ##
  83   # test fetching multiple ways
  84   def test_ways
  85     # check error when no parameter provided
  86     get :ways
  87     assert_response :bad_request
  88
  89     # check error when no parameter value provided
  90     get :ways, :ways => ""
  91     assert_response :bad_request
  92
  93     # test a working call
  94     get :ways, :ways => "1,2,4,6"
  95     assert_response :success
  96     assert_select "osm" do
  97       assert_select "way", :count => 4
  98       assert_select "way[id='1'][visible='true']", :count => 1
  99       assert_select "way[id='2'][visible='false']", :count => 1
 100       assert_select "way[id='4'][visible='true']", :count => 1
 101       assert_select "way[id='6'][visible='true']", :count => 1
 102     end
 103
 104     # check error when a non-existent way is included
 105     get :ways, :ways => "1,2,4,6,400"
 106     assert_response :not_found
 107   end
 108
 109   # -------------------------------------
 110   # Test simple way creation.
 111   # -------------------------------------
 112
 113   def test_create
 114     ## First check that it fails when creating a way using a non-public user
 115     nid1 = current_nodes(:used_node_1).id
 116     nid2 = current_nodes(:used_node_2).id
 117     basic_authorization users(:normal_user).email, "test"
 118
 119     # use the first user's open changeset
 120     changeset_id = changesets(:normal_user_first_change).id
 121
 122     # create a way with pre-existing nodes
 123     content "<osm><way changeset='#{changeset_id}'>" +
 124             "<nd ref='#{nid1}'/><nd ref='#{nid2}'/>" +
 125             "<tag k='test' v='yes' /></way></osm>"
 126     put :create
 127     # hope for failure
 128     assert_response :forbidden,
 129                     "way upload did not return forbidden status"
 130
 131     ## Now use a public user
 132     nid1 = current_nodes(:used_node_1).id
 133     nid2 = current_nodes(:used_node_2).id
 134     basic_authorization users(:public_user).email, "test"
 135
 136     # use the first user's open changeset
 137     changeset_id = changesets(:public_user_first_change).id
 138
 139     # create a way with pre-existing nodes
 140     content "<osm><way changeset='#{changeset_id}'>" +
 141             "<nd ref='#{nid1}'/><nd ref='#{nid2}'/>" +
 142             "<tag k='test' v='yes' /></way></osm>"
 143     put :create
 144     # hope for success
 145     assert_response :success,
 146                     "way upload did not return success status"
 147     # read id of created way and search for it
 148     wayid = @response.body
 149     checkway = Way.find(wayid)
 150     assert_not_nil checkway,
 151                    "uploaded way not found in data base after upload"
 152     # compare values
 153     assert_equal checkway.nds.length, 2,
 154                  "saved way does not contain exactly one node"
 155     assert_equal checkway.nds[0], nid1,
 156                  "saved way does not contain the right node on pos 0"
 157     assert_equal checkway.nds[1], nid2,
 158                  "saved way does not contain the right node on pos 1"
 159     assert_equal checkway.changeset_id, changeset_id,
 160                  "saved way does not belong to the correct changeset"
 161     assert_equal users(:public_user).id, checkway.changeset.user_id,
 162                  "saved way does not belong to user that created it"
 163     assert_equal true, checkway.visible,
 164                  "saved way is not visible"
 165   end
 166
 167   # -------------------------------------
 168   # Test creating some invalid ways.
 169   # -------------------------------------
 170
 171   def test_create_invalid
 172     ## First test with a private user to make sure that they are not authorized
 173     basic_authorization users(:normal_user).email, "test"
 174
 175     # use the first user's open changeset
 176     open_changeset_id = changesets(:normal_user_first_change).id
 177     closed_changeset_id = changesets(:normal_user_closed_change).id
 178     nid1 = current_nodes(:used_node_1).id
 179
 180     # create a way with non-existing node
 181     content "<osm><way changeset='#{open_changeset_id}'>" +
 182             "<nd ref='0'/><tag k='test' v='yes' /></way></osm>"
 183     put :create
 184     # expect failure
 185     assert_response :forbidden,
 186                     "way upload with invalid node using a private user did not return 'forbidden'"
 187
 188     # create a way with no nodes
 189     content "<osm><way changeset='#{open_changeset_id}'>" +
 190             "<tag k='test' v='yes' /></way></osm>"
 191     put :create
 192     # expect failure
 193     assert_response :forbidden,
 194                     "way upload with no node using a private userdid not return 'forbidden'"
 195
 196     # create a way inside a closed changeset
 197     content "<osm><way changeset='#{closed_changeset_id}'>" +
 198             "<nd ref='#{nid1}'/></way></osm>"
 199     put :create
 200     # expect failure
 201     assert_response :forbidden,
 202                     "way upload to closed changeset with a private user did not return 'forbidden'"
 203
 204     ## Now test with a public user
 205     basic_authorization users(:public_user).email, "test"
 206
 207     # use the first user's open changeset
 208     open_changeset_id = changesets(:public_user_first_change).id
 209     closed_changeset_id = changesets(:public_user_closed_change).id
 210     nid1 = current_nodes(:used_node_1).id
 211
 212     # create a way with non-existing node
 213     content "<osm><way changeset='#{open_changeset_id}'>" +
 214             "<nd ref='0'/><tag k='test' v='yes' /></way></osm>"
 215     put :create
 216     # expect failure
 217     assert_response :precondition_failed,
 218                     "way upload with invalid node did not return 'precondition failed'"
 219     assert_equal "Precondition failed: Way  requires the nodes with id in (0), which either do not exist, or are not visible.", @response.body
 220
 221     # create a way with no nodes
 222     content "<osm><way changeset='#{open_changeset_id}'>" +
 223             "<tag k='test' v='yes' /></way></osm>"
 224     put :create
 225     # expect failure
 226     assert_response :precondition_failed,
 227                     "way upload with no node did not return 'precondition failed'"
 228     assert_equal "Precondition failed: Cannot create way: data is invalid.", @response.body
 229
 230     # create a way inside a closed changeset
 231     content "<osm><way changeset='#{closed_changeset_id}'>" +
 232             "<nd ref='#{nid1}'/></way></osm>"
 233     put :create
 234     # expect failure
 235     assert_response :conflict,
 236                     "way upload to closed changeset did not return 'conflict'"
 237
 238     # create a way with a tag which is too long
 239     content "<osm><way changeset='#{open_changeset_id}'>" +
 240             "<nd ref='#{nid1}'/>" +
 241             "<tag k='foo' v='#{'x' * 256}'/>" +
 242             "</way></osm>"
 243     put :create
 244     # expect failure
 245     assert_response :bad_request,
 246                     "way upload to with too long tag did not return 'bad_request'"
 247   end
 248
 249   # -------------------------------------
 250   # Test deleting ways.
 251   # -------------------------------------
 252
 253   def test_delete
 254     # first try to delete way without auth
 255     delete :delete, :id => current_ways(:visible_way).id
 256     assert_response :unauthorized
 257
 258     # now set auth using the private user
 259     basic_authorization(users(:normal_user).email, "test")
 260
 261     # this shouldn't work as with the 0.6 api we need pay load to delete
 262     delete :delete, :id => current_ways(:visible_way).id
 263     assert_response :forbidden
 264
 265     # Now try without having a changeset
 266     content "<osm><way id='#{current_ways(:visible_way).id}'/></osm>"
 267     delete :delete, :id => current_ways(:visible_way).id
 268     assert_response :forbidden
 269
 270     # try to delete with an invalid (closed) changeset
 271     content update_changeset(current_ways(:visible_way).to_xml,
 272                              changesets(:normal_user_closed_change).id)
 273     delete :delete, :id => current_ways(:visible_way).id
 274     assert_response :forbidden
 275
 276     # try to delete with an invalid (non-existent) changeset
 277     content update_changeset(current_ways(:visible_way).to_xml, 0)
 278     delete :delete, :id => current_ways(:visible_way).id
 279     assert_response :forbidden
 280
 281     # Now try with a valid changeset
 282     content current_ways(:visible_way).to_xml
 283     delete :delete, :id => current_ways(:visible_way).id
 284     assert_response :forbidden
 285
 286     # check the returned value - should be the new version number
 287     # valid delete should return the new version number, which should
 288     # be greater than the old version number
 289     # assert @response.body.to_i > current_ways(:visible_way).version,
 290     #   "delete request should return a new version number for way"
 291
 292     # this won't work since the way is already deleted
 293     content current_ways(:invisible_way).to_xml
 294     delete :delete, :id => current_ways(:invisible_way).id
 295     assert_response :forbidden
 296
 297     # this shouldn't work as the way is used in a relation
 298     content current_ways(:used_way).to_xml
 299     delete :delete, :id => current_ways(:used_way).id
 300     assert_response :forbidden,
 301                     "shouldn't be able to delete a way used in a relation (#{@response.body}), when done by a private user"
 302
 303     # this won't work since the way never existed
 304     delete :delete, :id => 0
 305     assert_response :forbidden
 306
 307     ### Now check with a public user
 308     # now set auth
 309     basic_authorization(users(:public_user).email, "test")
 310
 311     # this shouldn't work as with the 0.6 api we need pay load to delete
 312     delete :delete, :id => current_ways(:visible_way).id
 313     assert_response :bad_request
 314
 315     # Now try without having a changeset
 316     content "<osm><way id='#{current_ways(:visible_way).id}'/></osm>"
 317     delete :delete, :id => current_ways(:visible_way).id
 318     assert_response :bad_request
 319
 320     # try to delete with an invalid (closed) changeset
 321     content update_changeset(current_ways(:visible_way).to_xml,
 322                              changesets(:public_user_closed_change).id)
 323     delete :delete, :id => current_ways(:visible_way).id
 324     assert_response :conflict
 325
 326     # try to delete with an invalid (non-existent) changeset
 327     content update_changeset(current_ways(:visible_way).to_xml, 0)
 328     delete :delete, :id => current_ways(:visible_way).id
 329     assert_response :conflict
 330
 331     # Now try with a valid changeset
 332     content current_ways(:visible_way).to_xml
 333     delete :delete, :id => current_ways(:visible_way).id
 334     assert_response :success
 335
 336     # check the returned value - should be the new version number
 337     # valid delete should return the new version number, which should
 338     # be greater than the old version number
 339     assert @response.body.to_i > current_ways(:visible_way).version,
 340            "delete request should return a new version number for way"
 341
 342     # this won't work since the way is already deleted
 343     content current_ways(:invisible_way).to_xml
 344     delete :delete, :id => current_ways(:invisible_way).id
 345     assert_response :gone
 346
 347     # this shouldn't work as the way is used in a relation
 348     content current_ways(:used_way).to_xml
 349     delete :delete, :id => current_ways(:used_way).id
 350     assert_response :precondition_failed,
 351                     "shouldn't be able to delete a way used in a relation (#{@response.body})"
 352     assert_equal "Precondition failed: Way 3 is still used by relations 1.", @response.body
 353
 354     # this won't work since the way never existed
 355     delete :delete, :id => 0
 356     assert_response :not_found
 357   end
 358
 359   ##
 360   # tests whether the API works and prevents incorrect use while trying
 361   # to update ways.
 362   def test_update
 363     ## First test with no user credentials
 364     # try and update a way without authorisation
 365     # first try to delete way without auth
 366     content current_ways(:visible_way).to_xml
 367     put :update, :id => current_ways(:visible_way).id
 368     assert_response :unauthorized
 369
 370     ## Second test with the private user
 371
 372     # setup auth
 373     basic_authorization(users(:normal_user).email, "test")
 374
 375     ## trying to break changesets
 376
 377     # try and update in someone else's changeset
 378     content update_changeset(current_ways(:visible_way).to_xml,
 379                              changesets(:public_user_first_change).id)
 380     put :update, :id => current_ways(:visible_way).id
 381     assert_require_public_data "update with other user's changeset should be forbidden when date isn't public"
 382
 383     # try and update in a closed changeset
 384     content update_changeset(current_ways(:visible_way).to_xml,
 385                              changesets(:normal_user_closed_change).id)
 386     put :update, :id => current_ways(:visible_way).id
 387     assert_require_public_data "update with closed changeset should be forbidden, when data isn't public"
 388
 389     # try and update in a non-existant changeset
 390     content update_changeset(current_ways(:visible_way).to_xml, 0)
 391     put :update, :id => current_ways(:visible_way).id
 392     assert_require_public_data("update with changeset=0 should be forbidden, when data isn't public")
 393
 394     ## try and submit invalid updates
 395     content xml_replace_node(current_ways(:visible_way).to_xml, 3, 9999)
 396     put :update, :id => current_ways(:visible_way).id
 397     assert_require_public_data "way with non-existent node should be forbidden, when data isn't public"
 398
 399     content xml_replace_node(current_ways(:visible_way).to_xml, 3, current_nodes(:invisible_node).id)
 400     put :update, :id => current_ways(:visible_way).id
 401     assert_require_public_data "way with deleted node should be forbidden, when data isn't public"
 402
 403     ## finally, produce a good request which should work
 404     content current_ways(:visible_way).to_xml
 405     put :update, :id => current_ways(:visible_way).id
 406     assert_require_public_data "should have failed with a forbidden when data isn't public"
 407
 408     ## Finally test with the public user
 409
 410     # try and update a way without authorisation
 411     # first try to delete way without auth
 412     content current_ways(:visible_way).to_xml
 413     put :update, :id => current_ways(:visible_way).id
 414     assert_response :forbidden
 415
 416     # setup auth
 417     basic_authorization(users(:public_user).email, "test")
 418
 419     ## trying to break changesets
 420
 421     # try and update in someone else's changeset
 422     content update_changeset(current_ways(:visible_way).to_xml,
 423                              changesets(:normal_user_first_change).id)
 424     put :update, :id => current_ways(:visible_way).id
 425     assert_response :conflict, "update with other user's changeset should be rejected"
 426
 427     # try and update in a closed changeset
 428     content update_changeset(current_ways(:visible_way).to_xml,
 429                              changesets(:normal_user_closed_change).id)
 430     put :update, :id => current_ways(:visible_way).id
 431     assert_response :conflict, "update with closed changeset should be rejected"
 432
 433     # try and update in a non-existant changeset
 434     content update_changeset(current_ways(:visible_way).to_xml, 0)
 435     put :update, :id => current_ways(:visible_way).id
 436     assert_response :conflict, "update with changeset=0 should be rejected"
 437
 438     ## try and submit invalid updates
 439     content xml_replace_node(current_ways(:visible_way).to_xml, 3, 9999)
 440     put :update, :id => current_ways(:visible_way).id
 441     assert_response :precondition_failed, "way with non-existent node should be rejected"
 442
 443     content xml_replace_node(current_ways(:visible_way).to_xml, 3, current_nodes(:invisible_node).id)
 444     put :update, :id => current_ways(:visible_way).id
 445     assert_response :precondition_failed, "way with deleted node should be rejected"
 446
 447     ## next, attack the versioning
 448     current_way_version = current_ways(:visible_way).version
 449
 450     # try and submit a version behind
 451     content xml_attr_rewrite(current_ways(:visible_way).to_xml,
 452                              "version", current_way_version - 1)
 453     put :update, :id => current_ways(:visible_way).id
 454     assert_response :conflict, "should have failed on old version number"
 455
 456     # try and submit a version ahead
 457     content xml_attr_rewrite(current_ways(:visible_way).to_xml,
 458                              "version", current_way_version + 1)
 459     put :update, :id => current_ways(:visible_way).id
 460     assert_response :conflict, "should have failed on skipped version number"
 461
 462     # try and submit total crap in the version field
 463     content xml_attr_rewrite(current_ways(:visible_way).to_xml,
 464                              "version", "p1r4t3s!")
 465     put :update, :id => current_ways(:visible_way).id
 466     assert_response :conflict,
 467                     "should not be able to put 'p1r4at3s!' in the version field"
 468
 469     ## try an update with the wrong ID
 470     content current_ways(:used_way).to_xml
 471     put :update, :id => current_ways(:visible_way).id
 472     assert_response :bad_request,
 473                     "should not be able to update a way with a different ID from the XML"
 474
 475     ## try an update with a minimal valid XML doc which isn't a well-formed OSM doc.
 476     content "<update/>"
 477     put :update, :id => current_ways(:visible_way).id
 478     assert_response :bad_request,
 479                     "should not be able to update a way with non-OSM XML doc."
 480
 481     ## finally, produce a good request which should work
 482     content current_ways(:visible_way).to_xml
 483     put :update, :id => current_ways(:visible_way).id
 484     assert_response :success, "a valid update request failed"
 485   end
 486
 487   # ------------------------------------------------------------
 488   # test tags handling
 489   # ------------------------------------------------------------
 490
 491   ##
 492   # Try adding a new tag to a way
 493   def test_add_tags
 494     ## Try with the non-public user
 495     # setup auth
 496     basic_authorization(users(:normal_user).email, "test")
 497
 498     # add an identical tag to the way
 499     tag_xml = XML::Node.new("tag")
 500     tag_xml["k"] = "new"
 501     tag_xml["v"] = "yes"
 502
 503     # add the tag into the existing xml
 504     way_xml = current_ways(:visible_way).to_xml
 505     way_xml.find("//osm/way").first << tag_xml
 506
 507     # try and upload it
 508     content way_xml
 509     put :update, :id => current_ways(:visible_way).id
 510     assert_response :forbidden,
 511                     "adding a duplicate tag to a way for a non-public should fail with 'forbidden'"
 512
 513     ## Now try with the public user
 514     # setup auth
 515     basic_authorization(users(:public_user).email, "test")
 516
 517     # add an identical tag to the way
 518     tag_xml = XML::Node.new("tag")
 519     tag_xml["k"] = "new"
 520     tag_xml["v"] = "yes"
 521
 522     # add the tag into the existing xml
 523     way_xml = current_ways(:visible_way).to_xml
 524     way_xml.find("//osm/way").first << tag_xml
 525
 526     # try and upload it
 527     content way_xml
 528     put :update, :id => current_ways(:visible_way).id
 529     assert_response :success,
 530                     "adding a new tag to a way should succeed"
 531     assert_equal current_ways(:visible_way).version + 1, @response.body.to_i
 532   end
 533
 534   ##
 535   # Try adding a duplicate of an existing tag to a way
 536   def test_add_duplicate_tags
 537     ## Try with the non-public user
 538     # setup auth
 539     basic_authorization(users(:normal_user).email, "test")
 540
 541     existing = create(:way_tag, :way => current_ways(:visible_way))
 542
 543     # add an identical tag to the way
 544     tag_xml = XML::Node.new("tag")
 545     tag_xml["k"] = existing.k
 546     tag_xml["v"] = existing.v
 547
 548     # add the tag into the existing xml
 549     way_xml = current_ways(:visible_way).to_xml
 550     way_xml.find("//osm/way").first << tag_xml
 551
 552     # try and upload it
 553     content way_xml
 554     put :update, :id => current_ways(:visible_way).id
 555     assert_response :forbidden,
 556                     "adding a duplicate tag to a way for a non-public should fail with 'forbidden'"
 557
 558     ## Now try with the public user
 559     # setup auth
 560     basic_authorization(users(:public_user).email, "test")
 561
 562     # add an identical tag to the way
 563     tag_xml = XML::Node.new("tag")
 564     tag_xml["k"] = existing.k
 565     tag_xml["v"] = existing.v
 566
 567     # add the tag into the existing xml
 568     way_xml = current_ways(:visible_way).to_xml
 569     way_xml.find("//osm/way").first << tag_xml
 570
 571     # try and upload it
 572     content way_xml
 573     put :update, :id => current_ways(:visible_way).id
 574     assert_response :bad_request,
 575                     "adding a duplicate tag to a way should fail with 'bad request'"
 576     assert_equal "Element way/#{current_ways(:visible_way).id} has duplicate tags with key #{existing.k}", @response.body
 577   end
 578
 579   ##
 580   # Try adding a new duplicate tags to a way
 581   def test_new_duplicate_tags
 582     ## First test with the non-public user so should be rejected
 583     # setup auth
 584     basic_authorization(users(:normal_user).email, "test")
 585
 586     # create duplicate tag
 587     tag_xml = XML::Node.new("tag")
 588     tag_xml["k"] = "i_am_a_duplicate"
 589     tag_xml["v"] = "foobar"
 590
 591     # add the tag into the existing xml
 592     way_xml = current_ways(:visible_way).to_xml
 593
 594     # add two copies of the tag
 595     way_xml.find("//osm/way").first << tag_xml.copy(true) << tag_xml
 596
 597     # try and upload it
 598     content way_xml
 599     put :update, :id => current_ways(:visible_way).id
 600     assert_response :forbidden,
 601                     "adding new duplicate tags to a way using a non-public user should fail with 'forbidden'"
 602
 603     ## Now test with the public user
 604     # setup auth
 605     basic_authorization(users(:public_user).email, "test")
 606
 607     # create duplicate tag
 608     tag_xml = XML::Node.new("tag")
 609     tag_xml["k"] = "i_am_a_duplicate"
 610     tag_xml["v"] = "foobar"
 611
 612     # add the tag into the existing xml
 613     way_xml = current_ways(:visible_way).to_xml
 614
 615     # add two copies of the tag
 616     way_xml.find("//osm/way").first << tag_xml.copy(true) << tag_xml
 617
 618     # try and upload it
 619     content way_xml
 620     put :update, :id => current_ways(:visible_way).id
 621     assert_response :bad_request,
 622                     "adding new duplicate tags to a way should fail with 'bad request'"
 623     assert_equal "Element way/#{current_ways(:visible_way).id} has duplicate tags with key i_am_a_duplicate", @response.body
 624   end
 625
 626   ##
 627   # Try adding a new duplicate tags to a way.
 628   # But be a bit subtle - use unicode decoding ambiguities to use different
 629   # binary strings which have the same decoding.
 630   def test_invalid_duplicate_tags
 631     ## First make sure that you can't with a non-public user
 632     # setup auth
 633     basic_authorization(users(:normal_user).email, "test")
 634
 635     # add the tag into the existing xml
 636     way_str = "<osm><way changeset='1'>"
 637     way_str << "<tag k='addr:housenumber' v='1'/>"
 638     way_str << "<tag k='addr:housenumber' v='2'/>"
 639     way_str << "</way></osm>"
 640
 641     # try and upload it
 642     content way_str
 643     put :create
 644     assert_response :forbidden,
 645                     "adding new duplicate tags to a way with a non-public user should fail with 'forbidden'"
 646
 647     ## Now do it with a public user
 648     # setup auth
 649     basic_authorization(users(:public_user).email, "test")
 650
 651     # add the tag into the existing xml
 652     way_str = "<osm><way changeset='1'>"
 653     way_str << "<tag k='addr:housenumber' v='1'/>"
 654     way_str << "<tag k='addr:housenumber' v='2'/>"
 655     way_str << "</way></osm>"
 656
 657     # try and upload it
 658     content way_str
 659     put :create
 660     assert_response :bad_request,
 661                     "adding new duplicate tags to a way should fail with 'bad request'"
 662     assert_equal "Element way/ has duplicate tags with key addr:housenumber", @response.body
 663   end
 664
 665   ##
 666   # test that a call to ways_for_node returns all ways that contain the node
 667   # and none that don't.
 668   def test_ways_for_node
 669     # in current fixtures ways 1 and 3 all use node 3. ways 2 and 4
 670     # *used* to use it but doesn't.
 671     get :ways_for_node, :id => current_nodes(:used_node_1).id
 672     assert_response :success
 673     ways_xml = XML::Parser.string(@response.body).parse
 674     assert_not_nil ways_xml, "failed to parse ways_for_node response"
 675
 676     # check that the set of IDs match expectations
 677     expected_way_ids = [current_ways(:visible_way).id,
 678                         current_ways(:used_way).id]
 679     found_way_ids = ways_xml.find("//osm/way").collect { |w| w["id"].to_i }
 680     assert_equal expected_way_ids.sort, found_way_ids.sort,
 681                  "expected ways for node #{current_nodes(:used_node_1).id} did not match found"
 682
 683     # check the full ways to ensure we're not missing anything
 684     expected_way_ids.each do |id|
 685       way_xml = ways_xml.find("//osm/way[@id='#{id}']").first
 686       assert_ways_are_equal(Way.find(id),
 687                             Way.from_xml_node(way_xml))
 688     end
 689   end
 690
 691   ##
 692   # update the changeset_id of a way element
 693   def update_changeset(xml, changeset_id)
 694     xml_attr_rewrite(xml, "changeset", changeset_id)
 695   end
 696
 697   ##
 698   # update an attribute in the way element
 699   def xml_attr_rewrite(xml, name, value)
 700     xml.find("//osm/way").first[name] = value.to_s
 701     xml
 702   end
 703
 704   ##
 705   # replace a node in a way element
 706   def xml_replace_node(xml, old_node, new_node)
 707     xml.find("//osm/way/nd[@ref='#{old_node}']").first["ref"] = new_node.to_s
 708     xml
 709   end
 710 end