@AntonKhorev commented on this pull request.


> @@ -421,7 +421,7 @@ def test_create
     post user_blocks_path
     assert_response :not_found
     assert_template "users/no_such_user"
-    assert_select "h1", "The user  does not exist"
+    assert_select "h1", "The user does not exist"

This breaks because the message itself doesn't make sense, if we take i18n into 
account. There's an empty username with spaces around it. Temporarily I'd do 
`:html` fix like in 
[changesets_helper_test](https://github.com/openstreetmap/openstreetmap-website/pull/6318/files#r2273910522)
 because it allows commits to be in a different order.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6318#pullrequestreview-3116675579
You are receiving this because you are subscribed to this thread.

Message ID: 
<openstreetmap/openstreetmap-website/pull/6318/review/3116675...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to