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

Add created_by_name to the Notification.serialize_for_csv.

Browse Source 
The concern about performnace degrading has been thought through. We do not believe there will be an adverse effect since the high volume users do not send off messages.
This commit is contained in:
Rebecca Law
2018-09-07 10:22:45 +01:00
parent db64423ca8
commit 8aacfb289d
2 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
tests/app/job/test_rest.py
View File

@@ -815,4 +815,5 @@ def test_get_all_notifications_for_job_returns_csv_format(
assert len(resp['notifications']) == 1
notification = resp['notifications'][0]
assert set(notification.keys()) == \
set(['created_at', 'template_type', 'template_name', 'job_name', 'status', 'row_number', 'recipient'])
set(['created_at', 'created_by_name', 'template_type',
'template_name', 'job_name', 'status', 'row_number', 'recipient'])