darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-02-01 15:46:07 -05:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #1281 from GSA/1280-change-page_size-to-20-rather-than-set-at-50

Browse Source 
update page_size to 20 from 50
This commit is contained in:
Carlo Costino
2024-08-20 15:54:40 -04:00
committed by GitHub
parent d144cae799 78f76dfc25
commit e1ba1ac847
3 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
tests/app/inbound_sms/test_rest.py
View File

@@ -212,7 +212,7 @@ def test_get_inbound_sms_by_id_with_invalid_service_id_returns_404(
@pytest.mark.parametrize(
"page_given, expected_rows, has_next_link",
[(True, 10, False), (False, 50, True)],
[(True, 20, True), (False, 20, True)],
)
def test_get_most_recent_inbound_sms_for_service(
admin_request,

4
tests/app/service/test_rest.py
View File

@@ -2036,10 +2036,10 @@ def test_get_notifications_for_service_pagination_links(
resp = admin_request.get(
"service.get_all_notifications_for_service",
service_id=sample_template.service_id,
page=3,
page=6,
)
assert "?page=2" in resp["links"]["prev"]
assert "?page=5" in resp["links"]["prev"]
assert "next" not in resp["links"]