Fixed problem where tag lengths were generating a 422 error with no message. They...

[rails.git] / test / functional / way_controller_test.rb

diff --git a/test/functional/way_controller_test.rb b/test/functional/way_controller_test.rb
index 9c32ae90400508d7d3654fec1021833fd4ae6627..862e700a68c85fa84eba541fc4b07aca1c076fac 100644 (file)
--- a/test/functional/way_controller_test.rb
+++ b/test/functional/way_controller_test.rb
@@ -164,6 +164,7 @@ class WayControllerTest < ActionController::TestCase
     # expect failure
     assert_response :precondition_failed, 
         "way upload with invalid node did not return 'precondition failed'"
     # expect failure
     assert_response :precondition_failed, 
         "way upload with invalid node did not return 'precondition failed'"

+    assert_equal "Precondition failed: Way  requires the node with id 0, which either does not exist, or is not visible.", @response.body

 
     # create a way with no nodes
     content "<osm><way changeset='#{open_changeset_id}'>" +
 
     # create a way with no nodes
     content "<osm><way changeset='#{open_changeset_id}'>" +


@@ -172,6 +173,7 @@ class WayControllerTest < ActionController::TestCase
     # expect failure
     assert_response :precondition_failed, 
         "way upload with no node did not return 'precondition failed'"
     # expect failure
     assert_response :precondition_failed, 
         "way upload with no node did not return 'precondition failed'"

+    assert_equal "Precondition failed: Cannot create way: data is invalid.", @response.body

 
     # create a way inside a closed changeset
     content "<osm><way changeset='#{closed_changeset_id}'>" +
 
     # create a way inside a closed changeset
     content "<osm><way changeset='#{closed_changeset_id}'>" +


@@ -180,6 +182,16 @@ class WayControllerTest < ActionController::TestCase
     # expect failure
     assert_response :conflict, 
         "way upload to closed changeset did not return 'conflict'"    
     # expect failure
     assert_response :conflict, 
         "way upload to closed changeset did not return 'conflict'"    

+
+    # create a way with a tag which is too long
+    content "<osm><way changeset='#{open_changeset_id}'>" +
+      "<nd ref='#{nid1}'/>" +
+      "<tag k='foo' v='#{'x'*256}'/>" +
+      "</way></osm>"
+    put :create
+    # expect failure
+    assert_response :bad_request, 
+        "way upload to with too long tag did not return 'bad_request'"

   end
 
   # -------------------------------------
   end
 
   # -------------------------------------


@@ -286,6 +298,7 @@ class WayControllerTest < ActionController::TestCase
     delete :delete, :id => current_ways(:used_way).id
     assert_response :precondition_failed, 
        "shouldn't be able to delete a way used in a relation (#{@response.body})"
     delete :delete, :id => current_ways(:used_way).id
     assert_response :precondition_failed, 
        "shouldn't be able to delete a way used in a relation (#{@response.body})"

+    assert_equal "Precondition failed: Way 3 still used by relation 1.", @response.body

 
     # this won't work since the way never existed
     delete :delete, :id => 0
 
     # this won't work since the way never existed
     delete :delete, :id => 0