darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-08-14 23:07:49 -04:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #2612 from alphagov/redaction_failed_doesnt_matter_when_test_key

Browse Source 
Don't log address redaction failure when letter sent with test key
This commit is contained in:
Pea (Malgorzata Tyczynska)
2019-09-18 16:05:39 +01:00
committed by GitHub
parent f9f9092b5c 1279a46b8b
commit c1b53dce3a
2 changed files with 8 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
tests/app/celery/test_letters_pdf_tasks.py
View File

@@ -543,10 +543,13 @@ def test_process_letter_task_check_virus_scan_passed_when_redaction_fails(
assert sample_letter_notification.billable_units == 2
assert sample_letter_notification.status == notification_status
mock_copy_s3.assert_called_once_with(
bucket_name, filename,
bucket_name, 'REDACTION_FAILURE/' + filename
)
if key_type == KEY_TYPE_NORMAL:
mock_copy_s3.assert_called_once_with(
bucket_name, filename,
bucket_name, 'REDACTION_FAILURE/' + filename
)
else:
mock_copy_s3.assert_not_called()
@freeze_time('2018-01-01 18:00')