test/functional/changeset_controller_test.rb

   1 require File.dirname(__FILE__) + '/../test_helper'
   2 require 'changeset_controller'
   3
   4 class ChangesetControllerTest < ActionController::TestCase
   5   api_fixtures
   6
   7   def basic_authorization(user, pass)
   8     @request.env["HTTP_AUTHORIZATION"] = "Basic %s" % Base64.encode64("#{user}:#{pass}")
   9   end
  10
  11   def content(c)
  12     @request.env["RAW_POST_DATA"] = c.to_s
  13   end
  14
  15   # -----------------------
  16   # Test simple changeset creation
  17   # -----------------------
  18
  19   def test_create
  20     basic_authorization "test@openstreetmap.org", "test"
  21
  22     # Create the first user's changeset
  23     content "<osm><changeset>" +
  24       "<tag k='created_by' v='osm test suite checking changesets'/>" +
  25       "</changeset></osm>"
  26     put :create
  27
  28     assert_response :success, "Creation of changeset did not return sucess status"
  29     newid = @response.body.to_i
  30
  31     # check end time, should be an hour ahead of creation time
  32     cs = Changeset.find(newid)
  33     duration = cs.closed_at - cs.created_at
  34     # the difference can either be a rational, or a floating point number
  35     # of seconds, depending on the code path taken :-(
  36     if duration.class == Rational
  37       assert_equal Rational(1,24), duration , "initial idle timeout should be an hour (#{cs.created_at} -> #{cs.closed_at})"
  38     else
  39       # must be number of seconds...
  40       assert_equal 3600.0, duration , "initial idle timeout should be an hour (#{cs.created_at} -> #{cs.closed_at})"
  41     end
  42   end
  43
  44   def test_create_invalid
  45     basic_authorization "test@openstreetmap.org", "test"
  46     content "<osm><changeset></osm>"
  47     put :create
  48     assert_response :bad_request, "creating a invalid changeset should fail"
  49   end
  50
  51   ##
  52   # check that the changeset can be read and returns the correct
  53   # document structure.
  54   def test_read
  55     changeset_id = changesets(:normal_user_first_change).id
  56     get :read, :id => changeset_id
  57     assert_response :success, "cannot get first changeset"
  58
  59     assert_select "osm[version=#{API_VERSION}][generator=\"OpenStreetMap server\"]", 1
  60     assert_select "osm>changeset[id=#{changeset_id}]", 1
  61   end
  62
  63   ##
  64   # test that the user who opened a change can close it
  65   def test_close
  66     basic_authorization "test@openstreetmap.org", "test"
  67
  68     put :close, :id => changesets(:normal_user_first_change).id
  69     assert_response :success
  70   end
  71
  72   ##
  73   # test that a different user can't close another user's changeset
  74   def test_close_invalid
  75     basic_authorization "test@example.com", "test"
  76
  77     put :close, :id => changesets(:normal_user_first_change).id
  78     assert_response :conflict
  79     assert_equal "The user doesn't own that changeset", @response.body
  80   end
  81
  82   ##
  83   # upload something simple, but valid and check that it can
  84   # be read back ok.
  85   def test_upload_simple_valid
  86     basic_authorization "test@openstreetmap.org", "test"
  87
  88     # simple diff to change a node, way and relation by removing
  89     # their tags
  90     diff = <<EOF
  91 <osmChange>
  92  <modify>
  93   <node id='1' lon='0' lat='0' changeset='1' version='1'/>
  94   <way id='1' changeset='1' version='1'>
  95    <nd ref='3'/>
  96   </way>
  97  </modify>
  98  <modify>
  99   <relation id='1' changeset='1' version='1'>
 100    <member type='way' role='some' ref='3'/>
 101    <member type='node' role='some' ref='5'/>
 102    <member type='relation' role='some' ref='3'/>
 103   </relation>
 104  </modify>
 105 </osmChange>
 106 EOF
 107
 108     # upload it
 109     content diff
 110     post :upload, :id => 1
 111     assert_response :success,
 112       "can't upload a simple valid diff to changeset: #{@response.body}"
 113
 114     # check that the changes made it into the database
 115     assert_equal 0, Node.find(1).tags.size, "node 1 should now have no tags"
 116     assert_equal 0, Way.find(1).tags.size, "way 1 should now have no tags"
 117     assert_equal 0, Relation.find(1).tags.size, "relation 1 should now have no tags"
 118   end
 119
 120   ##
 121   # upload something which creates new objects using placeholders
 122   def test_upload_create_valid
 123     basic_authorization "test@openstreetmap.org", "test"
 124
 125     # simple diff to create a node way and relation using placeholders
 126     diff = <<EOF
 127 <osmChange>
 128  <create>
 129   <node id='-1' lon='0' lat='0' changeset='1'>
 130    <tag k='foo' v='bar'/>
 131    <tag k='baz' v='bat'/>
 132   </node>
 133   <way id='-1' changeset='1'>
 134    <nd ref='3'/>
 135   </way>
 136  </create>
 137  <create>
 138   <relation id='-1' changeset='1'>
 139    <member type='way' role='some' ref='3'/>
 140    <member type='node' role='some' ref='5'/>
 141    <member type='relation' role='some' ref='3'/>
 142   </relation>
 143  </create>
 144 </osmChange>
 145 EOF
 146
 147     # upload it
 148     content diff
 149     post :upload, :id => 1
 150     assert_response :success,
 151       "can't upload a simple valid creation to changeset: #{@response.body}"
 152
 153     # check the returned payload
 154     assert_select "diffResult[version=#{API_VERSION}][generator=\"OpenStreetMap server\"]", 1
 155     assert_select "diffResult>node", 1
 156     assert_select "diffresult>way", 1
 157     assert_select "diffResult>relation", 1
 158
 159     # inspect the response to find out what the new element IDs are
 160     doc = XML::Parser.string(@response.body).parse
 161     new_node_id = doc.find("//diffResult/node").first["new_id"].to_i
 162     new_way_id = doc.find("//diffResult/way").first["new_id"].to_i
 163     new_rel_id = doc.find("//diffResult/relation").first["new_id"].to_i
 164
 165     # check the old IDs are all present and negative one
 166     assert_equal -1, doc.find("//diffResult/node").first["old_id"].to_i
 167     assert_equal -1, doc.find("//diffResult/way").first["old_id"].to_i
 168     assert_equal -1, doc.find("//diffResult/relation").first["old_id"].to_i
 169
 170     # check the versions are present and equal one
 171     assert_equal 1, doc.find("//diffResult/node").first["new_version"].to_i
 172     assert_equal 1, doc.find("//diffResult/way").first["new_version"].to_i
 173     assert_equal 1, doc.find("//diffResult/relation").first["new_version"].to_i
 174
 175     # check that the changes made it into the database
 176     assert_equal 2, Node.find(new_node_id).tags.size, "new node should have two tags"
 177     assert_equal 0, Way.find(new_way_id).tags.size, "new way should have no tags"
 178     assert_equal 0, Relation.find(new_rel_id).tags.size, "new relation should have no tags"
 179   end
 180
 181   ##
 182   # test a complex delete where we delete elements which rely on eachother
 183   # in the same transaction.
 184   def test_upload_delete
 185     basic_authorization "test@openstreetmap.org", "test"
 186
 187     diff = XML::Document.new
 188     diff.root = XML::Node.new "osmChange"
 189     delete = XML::Node.new "delete"
 190     diff.root << delete
 191     delete << current_relations(:visible_relation).to_xml_node
 192     delete << current_relations(:used_relation).to_xml_node
 193     delete << current_ways(:used_way).to_xml_node
 194     delete << current_nodes(:node_used_by_relationship).to_xml_node
 195
 196     # upload it
 197     content diff
 198     post :upload, :id => 1
 199     assert_response :success,
 200       "can't upload a deletion diff to changeset: #{@response.body}"
 201
 202     # check that everything was deleted
 203     assert_equal false, Node.find(current_nodes(:node_used_by_relationship).id).visible
 204     assert_equal false, Way.find(current_ways(:used_way).id).visible
 205     assert_equal false, Relation.find(current_relations(:visible_relation).id).visible
 206     assert_equal false, Relation.find(current_relations(:used_relation).id).visible
 207   end
 208
 209   ##
 210   # test that deleting stuff in a transaction doesn't bypass the checks
 211   # to ensure that used elements are not deleted.
 212   def test_upload_delete_invalid
 213     basic_authorization "test@openstreetmap.org", "test"
 214
 215     diff = XML::Document.new
 216     diff.root = XML::Node.new "osmChange"
 217     delete = XML::Node.new "delete"
 218     diff.root << delete
 219     delete << current_relations(:visible_relation).to_xml_node
 220     delete << current_ways(:used_way).to_xml_node
 221     delete << current_nodes(:node_used_by_relationship).to_xml_node
 222
 223     # upload it
 224     content diff
 225     post :upload, :id => 1
 226     assert_response :precondition_failed,
 227       "shouldn't be able to upload a invalid deletion diff: #{@response.body}"
 228
 229     # check that nothing was, in fact, deleted
 230     assert_equal true, Node.find(current_nodes(:node_used_by_relationship).id).visible
 231     assert_equal true, Way.find(current_ways(:used_way).id).visible
 232     assert_equal true, Relation.find(current_relations(:visible_relation).id).visible
 233   end
 234
 235   ##
 236   # upload something which creates new objects and inserts them into
 237   # existing containers using placeholders.
 238   def test_upload_complex
 239     basic_authorization "test@openstreetmap.org", "test"
 240
 241     # simple diff to create a node way and relation using placeholders
 242     diff = <<EOF
 243 <osmChange>
 244  <create>
 245   <node id='-1' lon='0' lat='0' changeset='1'>
 246    <tag k='foo' v='bar'/>
 247    <tag k='baz' v='bat'/>
 248   </node>
 249  </create>
 250  <modify>
 251   <way id='1' changeset='1' version='1'>
 252    <nd ref='-1'/>
 253    <nd ref='3'/>
 254   </way>
 255   <relation id='1' changeset='1' version='1'>
 256    <member type='way' role='some' ref='3'/>
 257    <member type='node' role='some' ref='-1'/>
 258    <member type='relation' role='some' ref='3'/>
 259   </relation>
 260  </modify>
 261 </osmChange>
 262 EOF
 263
 264     # upload it
 265     content diff
 266     post :upload, :id => 1
 267     assert_response :success,
 268       "can't upload a complex diff to changeset: #{@response.body}"
 269
 270     # check the returned payload
 271     assert_select "diffResult[version=#{API_VERSION}][generator=\"#{GENERATOR}\"]", 1
 272     assert_select "diffResult>node", 1
 273     assert_select "diffResult>way", 1
 274     assert_select "diffResult>relation", 1
 275
 276     # inspect the response to find out what the new element IDs are
 277     doc = XML::Parser.string(@response.body).parse
 278     new_node_id = doc.find("//diffResult/node").first["new_id"].to_i
 279
 280     # check that the changes made it into the database
 281     assert_equal 2, Node.find(new_node_id).tags.size, "new node should have two tags"
 282     assert_equal [new_node_id, 3], Way.find(1).nds, "way nodes should match"
 283     Relation.find(1).members.each do |type,id,role|
 284       if type == 'node'
 285         assert_equal new_node_id, id, "relation should contain new node"
 286       end
 287     end
 288   end
 289
 290   ##
 291   # create a diff which references several changesets, which should cause
 292   # a rollback and none of the diff gets committed
 293   def test_upload_invalid_changesets
 294     basic_authorization "test@openstreetmap.org", "test"
 295
 296     # simple diff to create a node way and relation using placeholders
 297     diff = <<EOF
 298 <osmChange>
 299  <modify>
 300   <node id='1' lon='0' lat='0' changeset='1' version='1'/>
 301   <way id='1' changeset='1' version='1'>
 302    <nd ref='3'/>
 303   </way>
 304  </modify>
 305  <modify>
 306   <relation id='1' changeset='1' version='1'>
 307    <member type='way' role='some' ref='3'/>
 308    <member type='node' role='some' ref='5'/>
 309    <member type='relation' role='some' ref='3'/>
 310   </relation>
 311  </modify>
 312  <create>
 313   <node id='-1' changeset='4'>
 314    <tag k='foo' v='bar'/>
 315    <tag k='baz' v='bat'/>
 316   </node>
 317  </create>
 318 </osmChange>
 319 EOF
 320     # cache the objects before uploading them
 321     node = current_nodes(:visible_node)
 322     way = current_ways(:visible_way)
 323     rel = current_relations(:visible_relation)
 324
 325     # upload it
 326     content diff
 327     post :upload, :id => 1
 328     assert_response :conflict,
 329       "uploading a diff with multiple changsets should have failed"
 330
 331     # check that objects are unmodified
 332     assert_nodes_are_equal(node, Node.find(1))
 333     assert_ways_are_equal(way, Way.find(1))
 334   end
 335
 336   ##
 337   # upload multiple versions of the same element in the same diff.
 338   def test_upload_multiple_valid
 339     basic_authorization "test@openstreetmap.org", "test"
 340
 341     # change the location of a node multiple times, each time referencing
 342     # the last version. doesn't this depend on version numbers being
 343     # sequential?
 344     diff = <<EOF
 345 <osmChange>
 346  <modify>
 347   <node id='1' lon='0' lat='0' changeset='1' version='1'/>
 348   <node id='1' lon='1' lat='0' changeset='1' version='2'/>
 349   <node id='1' lon='1' lat='1' changeset='1' version='3'/>
 350   <node id='1' lon='1' lat='2' changeset='1' version='4'/>
 351   <node id='1' lon='2' lat='2' changeset='1' version='5'/>
 352   <node id='1' lon='3' lat='2' changeset='1' version='6'/>
 353   <node id='1' lon='3' lat='3' changeset='1' version='7'/>
 354   <node id='1' lon='9' lat='9' changeset='1' version='8'/>
 355  </modify>
 356 </osmChange>
 357 EOF
 358
 359     # upload it
 360     content diff
 361     post :upload, :id => 1
 362     assert_response :success,
 363       "can't upload multiple versions of an element in a diff: #{@response.body}"
 364   end
 365
 366   ##
 367   # upload multiple versions of the same element in the same diff, but
 368   # keep the version numbers the same.
 369   def test_upload_multiple_duplicate
 370     basic_authorization "test@openstreetmap.org", "test"
 371
 372     diff = <<EOF
 373 <osmChange>
 374  <modify>
 375   <node id='1' lon='0' lat='0' changeset='1' version='1'/>
 376   <node id='1' lon='1' lat='1' changeset='1' version='1'/>
 377  </modify>
 378 </osmChange>
 379 EOF
 380
 381     # upload it
 382     content diff
 383     post :upload, :id => 1
 384     assert_response :conflict,
 385       "shouldn't be able to upload the same element twice in a diff: #{@response.body}"
 386   end
 387
 388   ##
 389   # try to upload some elements without specifying the version
 390   def test_upload_missing_version
 391     basic_authorization "test@openstreetmap.org", "test"
 392
 393     diff = <<EOF
 394 <osmChange>
 395  <modify>
 396   <node id='1' lon='1' lat='1' changeset='1'/>
 397  </modify>
 398 </osmChange>
 399 EOF
 400
 401     # upload it
 402     content diff
 403     post :upload, :id => 1
 404     assert_response :bad_request,
 405       "shouldn't be able to upload an element without version: #{@response.body}"
 406   end
 407
 408   ##
 409   # try to upload with commands other than create, modify, or delete
 410   def test_action_upload_invalid
 411     basic_authorization "test@openstreetmap.org", "test"
 412
 413     diff = <<EOF
 414 <osmChange>
 415   <ping>
 416     <node id='1' lon='1' lat='1' changeset='1' />
 417   </ping>
 418 </osmChange>
 419 EOF
 420   content diff
 421   post :upload, :id => 1
 422   assert_response :bad_request, "Shouldn't be able to upload a diff with the action ping"
 423   assert_equal @response.body, "Unknown action ping, choices are create, modify, delete."
 424   end
 425
 426   ##
 427   # when we make some simple changes we get the same changes back from the
 428   # diff download.
 429   def test_diff_download_simple
 430     basic_authorization(users(:normal_user).email, "test")
 431
 432     # create a temporary changeset
 433     content "<osm><changeset>" +
 434       "<tag k='created_by' v='osm test suite checking changesets'/>" +
 435       "</changeset></osm>"
 436     put :create
 437     assert_response :success
 438     changeset_id = @response.body.to_i
 439
 440     # add a diff to it
 441     diff = <<EOF
 442 <osmChange>
 443  <modify>
 444   <node id='1' lon='0' lat='0' changeset='#{changeset_id}' version='1'/>
 445   <node id='1' lon='1' lat='0' changeset='#{changeset_id}' version='2'/>
 446   <node id='1' lon='1' lat='1' changeset='#{changeset_id}' version='3'/>
 447   <node id='1' lon='1' lat='2' changeset='#{changeset_id}' version='4'/>
 448   <node id='1' lon='2' lat='2' changeset='#{changeset_id}' version='5'/>
 449   <node id='1' lon='3' lat='2' changeset='#{changeset_id}' version='6'/>
 450   <node id='1' lon='3' lat='3' changeset='#{changeset_id}' version='7'/>
 451   <node id='1' lon='9' lat='9' changeset='#{changeset_id}' version='8'/>
 452  </modify>
 453 </osmChange>
 454 EOF
 455
 456     # upload it
 457     content diff
 458     post :upload, :id => changeset_id
 459     assert_response :success,
 460       "can't upload multiple versions of an element in a diff: #{@response.body}"
 461
 462     get :download, :id => changeset_id
 463     assert_response :success
 464
 465     assert_select "osmChange", 1
 466     assert_select "osmChange>modify", 8
 467     assert_select "osmChange>modify>node", 8
 468   end
 469
 470   ##
 471   # culled this from josm to ensure that nothing in the way that josm
 472   # is formatting the request is causing it to fail.
 473   #
 474   # NOTE: the error turned out to be something else completely!
 475   def test_josm_upload
 476     basic_authorization(users(:normal_user).email, "test")
 477
 478     # create a temporary changeset
 479     content "<osm><changeset>" +
 480       "<tag k='created_by' v='osm test suite checking changesets'/>" +
 481       "</changeset></osm>"
 482     put :create
 483     assert_response :success
 484     changeset_id = @response.body.to_i
 485
 486     diff = <<OSM
 487 <osmChange version="0.6" generator="JOSM">
 488 <create version="0.6" generator="JOSM">
 489   <node id='-1' visible='true' changeset='#{changeset_id}' lat='51.49619982187321' lon='-0.18722061869438314' />
 490   <node id='-2' visible='true' changeset='#{changeset_id}' lat='51.496359883909605' lon='-0.18653093576241928' />
 491   <node id='-3' visible='true' changeset='#{changeset_id}' lat='51.49598132358285' lon='-0.18719613290981638' />
 492   <node id='-4' visible='true' changeset='#{changeset_id}' lat='51.4961591711078' lon='-0.18629015888084607' />
 493   <node id='-5' visible='true' changeset='#{changeset_id}' lat='51.49582126021711' lon='-0.18708186591517145' />
 494   <node id='-6' visible='true' changeset='#{changeset_id}' lat='51.49591018437858' lon='-0.1861432441734455' />
 495   <node id='-7' visible='true' changeset='#{changeset_id}' lat='51.49560784152179' lon='-0.18694719410005425' />
 496   <node id='-8' visible='true' changeset='#{changeset_id}' lat='51.49567389979617' lon='-0.1860289771788006' />
 497   <node id='-9' visible='true' changeset='#{changeset_id}' lat='51.49543761398892' lon='-0.186820684213126' />
 498   <way id='-10' action='modiy' visible='true' changeset='#{changeset_id}'>
 499     <nd ref='-1' />
 500     <nd ref='-2' />
 501     <nd ref='-3' />
 502     <nd ref='-4' />
 503     <nd ref='-5' />
 504     <nd ref='-6' />
 505     <nd ref='-7' />
 506     <nd ref='-8' />
 507     <nd ref='-9' />
 508     <tag k='highway' v='residential' />
 509     <tag k='name' v='Foobar Street' />
 510   </way>
 511 </create>
 512 </osmChange>
 513 OSM
 514
 515     # upload it
 516     content diff
 517     post :upload, :id => changeset_id
 518     assert_response :success,
 519       "can't upload a diff from JOSM: #{@response.body}"
 520
 521     get :download, :id => changeset_id
 522     assert_response :success
 523
 524     assert_select "osmChange", 1
 525     assert_select "osmChange>create>node", 9
 526     assert_select "osmChange>create>way", 1
 527     assert_select "osmChange>create>way>nd", 9
 528     assert_select "osmChange>create>way>tag", 2
 529   end
 530
 531   ##
 532   # when we make some complex changes we get the same changes back from the
 533   # diff download.
 534   def test_diff_download_complex
 535     basic_authorization(users(:normal_user).email, "test")
 536
 537     # create a temporary changeset
 538     content "<osm><changeset>" +
 539       "<tag k='created_by' v='osm test suite checking changesets'/>" +
 540       "</changeset></osm>"
 541     put :create
 542     assert_response :success
 543     changeset_id = @response.body.to_i
 544
 545     # add a diff to it
 546     diff = <<EOF
 547 <osmChange>
 548  <delete>
 549   <node id='1' lon='0' lat='0' changeset='#{changeset_id}' version='1'/>
 550  </delete>
 551  <create>
 552   <node id='-1' lon='9' lat='9' changeset='#{changeset_id}' version='0'/>
 553   <node id='-2' lon='8' lat='9' changeset='#{changeset_id}' version='0'/>
 554   <node id='-3' lon='7' lat='9' changeset='#{changeset_id}' version='0'/>
 555  </create>
 556  <modify>
 557   <node id='3' lon='20' lat='15' changeset='#{changeset_id}' version='1'/>
 558   <way id='1' changeset='#{changeset_id}' version='1'>
 559    <nd ref='3'/>
 560    <nd ref='-1'/>
 561    <nd ref='-2'/>
 562    <nd ref='-3'/>
 563   </way>
 564  </modify>
 565 </osmChange>
 566 EOF
 567
 568     # upload it
 569     content diff
 570     post :upload, :id => changeset_id
 571     assert_response :success,
 572       "can't upload multiple versions of an element in a diff: #{@response.body}"
 573
 574     get :download, :id => changeset_id
 575     assert_response :success
 576
 577     assert_select "osmChange", 1
 578     assert_select "osmChange>create", 3
 579     assert_select "osmChange>delete", 1
 580     assert_select "osmChange>modify", 2
 581     assert_select "osmChange>create>node", 3
 582     assert_select "osmChange>delete>node", 1
 583     assert_select "osmChange>modify>node", 1
 584     assert_select "osmChange>modify>way", 1
 585   end
 586
 587   ##
 588   # check that the bounding box of a changeset gets updated correctly
 589   def test_changeset_bbox
 590     basic_authorization "test@openstreetmap.org", "test"
 591
 592     # create a new changeset
 593     content "<osm><changeset/></osm>"
 594     put :create
 595     assert_response :success, "Creating of changeset failed."
 596     changeset_id = @response.body.to_i
 597
 598     # add a single node to it
 599     with_controller(NodeController.new) do
 600       content "<osm><node lon='1' lat='2' changeset='#{changeset_id}'/></osm>"
 601       put :create
 602       assert_response :success, "Couldn't create node."
 603     end
 604
 605     # get the bounding box back from the changeset
 606     get :read, :id => changeset_id
 607     assert_response :success, "Couldn't read back changeset."
 608     assert_select "osm>changeset[min_lon=1.0]", 1
 609     assert_select "osm>changeset[max_lon=1.0]", 1
 610     assert_select "osm>changeset[min_lat=2.0]", 1
 611     assert_select "osm>changeset[max_lat=2.0]", 1
 612
 613     # add another node to it
 614     with_controller(NodeController.new) do
 615       content "<osm><node lon='2' lat='1' changeset='#{changeset_id}'/></osm>"
 616       put :create
 617       assert_response :success, "Couldn't create second node."
 618     end
 619
 620     # get the bounding box back from the changeset
 621     get :read, :id => changeset_id
 622     assert_response :success, "Couldn't read back changeset for the second time."
 623     assert_select "osm>changeset[min_lon=1.0]", 1
 624     assert_select "osm>changeset[max_lon=2.0]", 1
 625     assert_select "osm>changeset[min_lat=1.0]", 1
 626     assert_select "osm>changeset[max_lat=2.0]", 1
 627
 628     # add (delete) a way to it
 629     with_controller(WayController.new) do
 630       content update_changeset(current_ways(:visible_way).to_xml,
 631                                changeset_id)
 632       put :delete, :id => current_ways(:visible_way).id
 633       assert_response :success, "Couldn't delete a way."
 634     end
 635
 636     # get the bounding box back from the changeset
 637     get :read, :id => changeset_id
 638     assert_response :success, "Couldn't read back changeset for the third time."
 639     assert_select "osm>changeset[min_lon=1.0]", 1
 640     assert_select "osm>changeset[max_lon=3.1]", 1
 641     assert_select "osm>changeset[min_lat=1.0]", 1
 642     assert_select "osm>changeset[max_lat=3.1]", 1
 643   end
 644
 645   ##
 646   # test that the changeset :include method works as it should
 647   def test_changeset_include
 648     basic_authorization "test@openstreetmap.org", "test"
 649
 650     # create a new changeset
 651     content "<osm><changeset/></osm>"
 652     put :create
 653     assert_response :success, "Creating of changeset failed."
 654     changeset_id = @response.body.to_i
 655
 656     # NOTE: the include method doesn't over-expand, like inserting
 657     # a real method does. this is because we expect the client to
 658     # know what it is doing!
 659     check_after_include(changeset_id,  1,  1, [ 1,  1,  1,  1])
 660     check_after_include(changeset_id,  3,  3, [ 1,  1,  3,  3])
 661     check_after_include(changeset_id,  4,  2, [ 1,  1,  4,  3])
 662     check_after_include(changeset_id,  2,  2, [ 1,  1,  4,  3])
 663     check_after_include(changeset_id, -1, -1, [-1, -1,  4,  3])
 664     check_after_include(changeset_id, -2,  5, [-2, -1,  4,  5])
 665   end
 666
 667   ##
 668   # test the query functionality of changesets
 669   def test_query
 670     get :query, :bbox => "-10,-10, 10, 10"
 671     assert_response :success, "can't get changesets in bbox"
 672     assert_changesets [1,4]
 673
 674     get :query, :bbox => "4.5,4.5,4.6,4.6"
 675     assert_response :success, "can't get changesets in bbox"
 676     assert_changesets [1]
 677
 678     # can't get changesets of user 1 without authenticating
 679     get :query, :user => users(:normal_user).id
 680     assert_response :not_found, "shouldn't be able to get changesets by non-public user"
 681
 682     # but this should work
 683     basic_authorization "test@openstreetmap.org", "test"
 684     get :query, :user => users(:normal_user).id
 685     assert_response :success, "can't get changesets by user"
 686     assert_changesets [1,3,4]
 687
 688     get :query, :user => users(:normal_user).id, :open => true
 689     assert_response :success, "can't get changesets by user and open"
 690     assert_changesets [1,4]
 691
 692     get :query, :time => '2007-12-31'
 693     assert_response :success, "can't get changesets by time-since"
 694     assert_changesets [1,2,4,5]
 695
 696     get :query, :time => '2008-01-01T12:34Z'
 697     assert_response :success, "can't get changesets by time-since with hour"
 698     assert_changesets [1,2,4,5]
 699
 700     get :query, :time => '2007-12-31T23:59Z,2008-01-01T00:01Z'
 701     assert_response :success, "can't get changesets by time-range"
 702     assert_changesets [1,4,5]
 703
 704     get :query, :open => 'true'
 705     assert_response :success, "can't get changesets by open-ness"
 706     assert_changesets [1,2,4]
 707   end
 708
 709   ##
 710   # check that errors are returned if garbage is inserted
 711   # into query strings
 712   def test_query_invalid
 713     [ "abracadabra!",
 714       "1,2,3,F",
 715       ";drop table users;"
 716       ].each do |bbox|
 717       get :query, :bbox => bbox
 718       assert_response :bad_request, "'#{bbox}' isn't a bbox"
 719     end
 720
 721     [ "now()",
 722       "00-00-00",
 723       ";drop table users;",
 724       ",",
 725       "-,-"
 726       ].each do |time|
 727       get :query, :time => time
 728       assert_response :bad_request, "'#{time}' isn't a valid time range"
 729     end
 730
 731     [ "me",
 732       "foobar",
 733       "-1",
 734       "0"
 735       ].each do |uid|
 736       get :query, :user => uid
 737       assert_response :bad_request, "'#{uid}' isn't a valid user ID"
 738     end
 739   end
 740
 741   ##
 742   # check updating tags on a changeset
 743   def test_changeset_update
 744     changeset = changesets(:normal_user_first_change)
 745     new_changeset = changeset.to_xml
 746     new_tag = XML::Node.new "tag"
 747     new_tag['k'] = "tagtesting"
 748     new_tag['v'] = "valuetesting"
 749     new_changeset.find("//osm/changeset").first << new_tag
 750     content new_changeset
 751
 752     # try without any authorization
 753     put :update, :id => changeset.id
 754     assert_response :unauthorized
 755
 756     # try with the wrong authorization
 757     basic_authorization "test@example.com", "test"
 758     put :update, :id => changeset.id
 759     assert_response :conflict
 760
 761     # now this should work...
 762     basic_authorization "test@openstreetmap.org", "test"
 763     put :update, :id => changeset.id
 764     assert_response :success
 765
 766     assert_select "osm>changeset[id=#{changeset.id}]", 1
 767     assert_select "osm>changeset>tag", 2
 768     assert_select "osm>changeset>tag[k=tagtesting][v=valuetesting]", 1
 769   end
 770
 771   ##
 772   # check that a user different from the one who opened the changeset
 773   # can't modify it.
 774   def test_changeset_update_invalid
 775     basic_authorization "test@example.com", "test"
 776
 777     changeset = changesets(:normal_user_first_change)
 778     new_changeset = changeset.to_xml
 779     new_tag = XML::Node.new "tag"
 780     new_tag['k'] = "testing"
 781     new_tag['v'] = "testing"
 782     new_changeset.find("//osm/changeset").first << new_tag
 783
 784     content new_changeset
 785     put :update, :id => changeset.id
 786     assert_response :conflict
 787   end
 788
 789   ##
 790   # check that a changeset can contain a certain max number of changes.
 791   def test_changeset_limits
 792     basic_authorization "test@openstreetmap.org", "test"
 793
 794     # open a new changeset
 795     content "<osm><changeset/></osm>"
 796     put :create
 797     assert_response :success, "can't create a new changeset"
 798     cs_id = @response.body.to_i
 799
 800     # start the counter just short of where the changeset should finish.
 801     offset = 10
 802     # alter the database to set the counter on the changeset directly,
 803     # otherwise it takes about 6 minutes to fill all of them.
 804     changeset = Changeset.find(cs_id)
 805     changeset.num_changes = Changeset::MAX_ELEMENTS - offset
 806     changeset.save!
 807
 808     with_controller(NodeController.new) do
 809       # create a new node
 810       content "<osm><node changeset='#{cs_id}' lat='0.0' lon='0.0'/></osm>"
 811       put :create
 812       assert_response :success, "can't create a new node"
 813       node_id = @response.body.to_i
 814
 815       get :read, :id => node_id
 816       assert_response :success, "can't read back new node"
 817       node_doc = XML::Parser.string(@response.body).parse
 818       node_xml = node_doc.find("//osm/node").first
 819
 820       # loop until we fill the changeset with nodes
 821       offset.times do |i|
 822         node_xml['lat'] = rand.to_s
 823         node_xml['lon'] = rand.to_s
 824         node_xml['version'] = (i+1).to_s
 825
 826         content node_doc
 827         put :update, :id => node_id
 828         assert_response :success, "attempt #{i} should have succeeded"
 829       end
 830
 831       # trying again should fail
 832       node_xml['lat'] = rand.to_s
 833       node_xml['lon'] = rand.to_s
 834       node_xml['version'] = offset.to_s
 835
 836       content node_doc
 837       put :update, :id => node_id
 838       assert_response :conflict, "final attempt should have failed"
 839     end
 840
 841     changeset = Changeset.find(cs_id)
 842     assert_equal Changeset::MAX_ELEMENTS + 1, changeset.num_changes
 843   end
 844
 845   #------------------------------------------------------------
 846   # utility functions
 847   #------------------------------------------------------------
 848
 849   ##
 850   # boilerplate for checking that certain changesets exist in the
 851   # output.
 852   def assert_changesets(ids)
 853     assert_select "osm>changeset", ids.size
 854     ids.each do |id|
 855       assert_select "osm>changeset[id=#{id}]", 1
 856     end
 857   end
 858
 859   ##
 860   # call the include method and assert properties of the bbox
 861   def check_after_include(changeset_id, lon, lat, bbox)
 862     content "<osm><node lon='#{lon}' lat='#{lat}'/></osm>"
 863     post :expand_bbox, :id => changeset_id
 864     assert_response :success, "Setting include of changeset failed: #{@response.body}"
 865
 866     # check exactly one changeset
 867     assert_select "osm>changeset", 1
 868     assert_select "osm>changeset[id=#{changeset_id}]", 1
 869
 870     # check the bbox
 871     doc = XML::Parser.string(@response.body).parse
 872     changeset = doc.find("//osm/changeset").first
 873     assert_equal bbox[0], changeset['min_lon'].to_f, "min lon"
 874     assert_equal bbox[1], changeset['min_lat'].to_f, "min lat"
 875     assert_equal bbox[2], changeset['max_lon'].to_f, "max lon"
 876     assert_equal bbox[3], changeset['max_lat'].to_f, "max lat"
 877   end
 878
 879   ##
 880   # update the changeset_id of a way element
 881   def update_changeset(xml, changeset_id)
 882     xml_attr_rewrite(xml, 'changeset', changeset_id)
 883   end
 884
 885   ##
 886   # update an attribute in a way element
 887   def xml_attr_rewrite(xml, name, value)
 888     xml.find("//osm/way").first[name] = value.to_s
 889     return xml
 890   end
 891
 892 end