[openstreetmap/openstreetmap-website] Use `assert_content` instead of `assert page.has_content?` (PR #3411)

[Tom Hughes]

@tomhughes commented on this pull request.



> @@ -113,7 +113,7 @@ def test_issue_index_with_multiple_roles
 
     visit issues_path
 
-    assert page.has_link?(I18n.t("issues.index.reports_count", :count => issue1.reports_count), :href => issue_path(issue1))
-    assert page.has_link?(I18n.t("issues.index.reports_count", :count => issue2.reports_count), :href => issue_path(issue2))
+    assert_link I18n.t("issues.index.reports_count", :count => issue1.reports_count), :href => issue_path(issue2)

Why does this change from `issue1` to `issue2`?

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

Message ID: <openstreetmap/openstreetmap-website/pull/3411/review/845703610 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20220106/40f15995/attachment.htm>