[openstreetmap/openstreetmap-website] Upgrade to rails 7.1.0 (PR #4278)
Andy Allan
notifications at github.com
Wed Oct 11 18:13:05 UTC 2023
@gravitystorm commented on this pull request.
> @@ -138,7 +138,7 @@ def test_create_invalid_xml
xml = "<osm><node lat='#{lat}' lon='#{lon}' changeset='#{changeset.id}'><tag k='foo' v='#{'x' * 256}'/></node></osm>"
put node_create_path, :params => xml, :headers => auth_header
assert_response :bad_request, "node upload did not return bad_request status"
- assert_equal ["NodeTag ", " v: is too long (maximum is 255 characters) (\"#{'x' * 256}\")"], @response.body.split(/[0-9]+,foo:/)
But if the output from the API changes, will that break client apps that are parsing the previous output?
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4278#discussion_r1355533654
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/4278/review/1672173792 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20231011/b85fa5f8/attachment.htm>
More information about the rails-dev
mailing list