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