X-Git-Url: https://git.openstreetmap.org./rails.git/blobdiff_plain/427becc1db048c05576887e74b26c4118fa32d77..fac8b5d28aa81f2233c7233b6876ad31d86be28b:/test/functional/way_controller_test.rb
diff --git a/test/functional/way_controller_test.rb b/test/functional/way_controller_test.rb
index 19b7fd54c..40ac0bd71 100644
--- a/test/functional/way_controller_test.rb
+++ b/test/functional/way_controller_test.rb
@@ -33,19 +33,30 @@ class WayControllerTest < ActionController::TestCase
##
# check the "full" mode
def test_full
- get :full, :id => current_ways(:visible_way).id
- assert_response :success
- # FIXME check whether this contains the stuff we want!
- #print @response.body
- # Check the way is correctly returned
- way = current_ways(:visible_way)
- assert_select "osm way[id=#{way.id}][version=#{way.version}][visible=#{way.visible}]", 1
- assert_select "osm way nd[ref=#{way.way_nodes[0].node_id}]", 1
- # Check that the node is correctly returned
- nd = current_ways(:visible_way).nodes
- assert_equal 1, nd.count
- nda = nd[0]
- assert_select "osm node[id=#{nda.id}][version=#{nda.version}][lat=#{nda.lat}][lon=#{nda.lon}]", 1
+ Way.find(:all).each do |way|
+ get :full, :id => way.id
+
+ # full call should say "gone" for non-visible ways...
+ unless way.visible
+ assert_response :gone
+ next
+ end
+
+ # otherwise it should say success
+ assert_response :success
+
+ # Check the way is correctly returned
+ assert_select "osm way[id=#{way.id}][version=#{way.version}][visible=#{way.visible}]", 1
+
+ # check that each node in the way appears once in the output as a
+ # reference and as the node element. note the slightly dodgy assumption
+ # that nodes appear only once. this is currently the case with the
+ # fixtures, but it doesn't have to be.
+ way.nodes.each do |n|
+ assert_select "osm way nd[ref=#{n.id}]", 1
+ assert_select "osm node[id=#{n.id}][version=#{n.version}][lat=#{n.lat}][lon=#{n.lon}]", 1
+ end
+ end
end
# -------------------------------------
@@ -208,6 +219,7 @@ class WayControllerTest < ActionController::TestCase
put :update, :id => current_ways(:visible_way).id
assert_response :bad_request,
"adding a duplicate tag to a way should fail with 'bad request'"
+ assert_equal "Element way/#{current_ways(:visible_way).id} has duplicate tags with key #{current_way_tags(:t1).k}.", @response.body
end
##
@@ -232,16 +244,13 @@ class WayControllerTest < ActionController::TestCase
put :update, :id => current_ways(:visible_way).id
assert_response :bad_request,
"adding new duplicate tags to a way should fail with 'bad request'"
+ assert_equal "Element way/#{current_ways(:visible_way).id} has duplicate tags with key i_am_a_duplicate.", @response.body
end
##
# Try adding a new duplicate tags to a way.
# But be a bit subtle - use unicode decoding ambiguities to use different
# binary strings which have the same decoding.
- #
- # NOTE: I'm not sure this test is working correctly, as a lot of the tag
- # keys seem to come out as "addr��housenumber". It might be something to
- # do with Ruby's unicode handling...?
def test_invalid_duplicate_tags
# setup auth
basic_authorization(users(:normal_user).email, "test")
@@ -249,26 +258,15 @@ class WayControllerTest < ActionController::TestCase
# add the tag into the existing xml
way_str = ""
way_str << ""
+ way_str << ""
+ way_str << "";
- # all of these keys have the same unicode decoding, but are binary
- # not equal. libxml should make these identical as it decodes the
- # XML document...
- [ "addr\xc0\xbahousenumber",
- "addr\xe0\x80\xbahousenumber",
- "addr\xf0\x80\x80\xbahousenumber" ].each do |key|
- # copy the XML doc to add the tags
- way_str_copy = way_str.clone
-
- # add all new tags to the way
- way_str_copy << ""
- way_str_copy << "";
-
- # try and upload it
- content way_str_copy
- put :create
- assert_response :bad_request,
- "adding new duplicate tags to a way should fail with 'bad request'"
- end
+ # try and upload it
+ content way_str
+ put :create
+ assert_response :bad_request,
+ "adding new duplicate tags to a way should fail with 'bad request'"
+ assert_equal "Element way/ has duplicate tags with key addr:housenumber.", @response.body
end
##