darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2025-12-17 10:42:25 -05:00
Code Issues Packages Projects Releases Wiki Activity

Fixed tests with appropriate parameters

Browse Source
This commit is contained in:
Andrew Shumway
2023-10-27 11:17:22 -06:00
committed by Carlo Costino
parent 948bb17a6c
commit 67d38ef32d
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/notifications/validators.py
View File

@@ -69,7 +69,7 @@ def check_service_over_total_message_limit(key_type, service):
def check_application_over_retention_limit(key_type, service): def check_application_over_retention_limit(key_type, service):
if key_type == KEY_TYPE_TEST or not current_app.config["REDIS_ENABLED"]: if key_type == KEY_TYPE_TEST or not current_app.config["REDIS_ENABLED"]:
return 0 return 0
total_stats = dao_get_notification_count_for_service(service=service) total_stats = dao_get_notification_count_for_service(service_id=service.id)
daily_message_limit = current_app.config["DAILY_MESSAGE_LIMIT"] daily_message_limit = current_app.config["DAILY_MESSAGE_LIMIT"]

2
tests/app/dao/notification_dao/test_notification_dao.py
View File

@@ -602,7 +602,7 @@ def test_dao_get_notification_count_for_service(notify_db_session):
create_notification(template) create_notification(template)
assert dao_get_notification_count_for_service(service=service) == 1 assert dao_get_notification_count_for_service(service_id=service.id) == 1
def test_dao_get_notification_count_for_job_id_returns_zero_for_no_notifications_for_job( def test_dao_get_notification_count_for_job_id_returns_zero_for_no_notifications_for_job(