darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-02-03 09:51:11 -05:00
Code Issues Packages Projects Releases Wiki Activity

Updated notification retention limit validator test to account for the new 10K limit

Browse Source 
Signed-off-by: Carlo Costino <carlo.costino@gsa.gov>
This commit is contained in:
Carlo Costino
2023-11-02 17:43:30 -04:00
parent 8dac36cba4
commit 23f17b166c
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
tests/app/notifications/test_validators.py
View File

@@ -78,13 +78,13 @@ def test_check_application_over_retention_limit_fails(
service = create_service() service = create_service()
mocker.patch( mocker.patch(
"app.notifications.validators.dao_get_notification_count_for_service", "app.notifications.validators.dao_get_notification_count_for_service",
return_value="5001", return_value="10001",
) )
with pytest.raises(TotalRequestsError) as e: with pytest.raises(TotalRequestsError) as e:
check_application_over_retention_limit(key_type, service) check_application_over_retention_limit(key_type, service)
assert e.value.status_code == 429 assert e.value.status_code == 429
assert e.value.message == "Exceeded total application limits (5000) for today" assert e.value.message == "Exceeded total application limits (10000) for today"
assert e.value.fields == [] assert e.value.fields == []