cst1/openstreetmap-website
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Test and fix for #1567, for teh betterer osmChange compliance.

Browse source
This commit is contained in:
Matt Amos 2009-02-08 16:54:28 +00:00
parent 12b779f0b4
commit 5143fbcd5b
2 changed files with 37 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

23
test/functional/changeset_controller_test.rb
View file

@ -217,6 +217,29 @@ EOF
assert_equal false, Relation.find(current_relations(:used_relation).id).visible
end
##
# test uploading a delete with no lat/lon, as they are optional in
# the osmChange spec.
def test_upload_nolatlon_delete
basic_authorization "test@openstreetmap.org", "test"
node = current_nodes(:visible_node)
cs = changesets(:normal_user_first_change)
diff = "<osmChange><delete><node id='#{node.id}' version='#{node.version}' changeset='#{cs.id}'/></delete></osmChange>"
# upload it
content diff
post :upload, :id => cs.id
assert_response :success,
"can't upload a deletion diff to changeset: #{@response.body}"
# check the response is well-formed
assert_select "diffResult>node", 1
# check that everything was deleted
assert_equal false, Node.find(node.id).visible
end
##
# test that deleting stuff in a transaction doesn't bypass the checks
# to ensure that used elements are not deleted.