[openstreetmap/openstreetmap-website] Add pagination to Issues page (PR #5057)

Tom Hughes notifications at github.com
Mon Aug 26 14:25:52 UTC 2024


@tomhughes commented on this pull request.



> +    end
+
+    sign_in_as(create(:administrator_user))
+
+    visit issues_path
+
+    # First Page
+    assert_no_content I18n.t("issues.index.user_not_found")
+    assert_no_content I18n.t("issues.index.issues_not_found")
+    assert_css "tr", :count => 51
+
+    # Second Page
+    click_on I18n.t("issues.page.older_issues")
+    assert_no_content I18n.t("issues.index.user_not_found")
+    assert_no_content I18n.t("issues.index.issues_not_found")
+    assert_css "tr", :count => 31, :wait => 1.5

I assume that `:wait` is a timeout? Why do we need that - what happens if we don't have it? Is there a default timeout that is too short or does it wait forever?

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

Message ID: <openstreetmap/openstreetmap-website/pull/5057/review/2260832425 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240826/da1eb5c6/attachment.htm>


More information about the rails-dev mailing list