darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-09-05 14:28:25 -04:00
Code Issues Packages Projects Releases Wiki Activity

code review feedback and merge from main

Browse Source
This commit is contained in:
Kenneth Kehl
2024-11-21 09:59:34 -08:00
parent 1a1de39949 7f9f205fdc
commit d2836131a1
3 changed files with 1 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/config.py
View File

@@ -166,7 +166,7 @@ class Config(object):
current_minute = (datetime.now().minute + 1) % 60 current_minute = (datetime.now().minute + 1) % 60
CELERY = { CELERY = {
"worker_max_tasks_per_child": 20, "worker_max_tasks_per_child": 200,
"broker_url": REDIS_URL, "broker_url": REDIS_URL,
"broker_transport_options": { "broker_transport_options": {
"visibility_timeout": 310, "visibility_timeout": 310,

1
app/performance_dashboard/rest.py
View File

@@ -100,7 +100,6 @@ def transform_into_notification_by_type_json(total_notifications):
def transform_processing_time_results_to_json(processing_time_results): def transform_processing_time_results_to_json(processing_time_results):
j = [] j = []
for x in processing_time_results: for x in processing_time_results:
print(f"HERE IS A PROCESSING TIME RESULT {x}")
j.append({"date": x.date, "percentage_under_10_seconds": x.percentage}) j.append({"date": x.date, "percentage_under_10_seconds": x.percentage})
return j return j

1
tests/app/dao/test_fact_processing_time_dao.py
View File

@@ -79,7 +79,6 @@ def test_get_processing_time_percentage_for_date_range_handles_zero_cases(
) )
results = get_processing_time_percentage_for_date_range("2021-02-21", "2021-02-22") results = get_processing_time_percentage_for_date_range("2021-02-21", "2021-02-22")
print(f"HERE ARE THE RESULTS {results}")
assert len(results) == 2 assert len(results) == 2
assert results[0].date == "2021-02-21" assert results[0].date == "2021-02-21"
assert results[0].messages_total == 0 assert results[0].messages_total == 0