darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-08-15 07:18:58 -04:00
Code Issues Packages Projects Releases Wiki Activity

reverting pr 2484 because it breaks personalization

Browse Source
This commit is contained in:
Beverly Nguyen
2025-05-13 15:35:31 -07:00
parent 83515cd321
commit 4728ee20e8
2 changed files with 1 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
tests/app/s3_client/test_s3_csv_client.py
View File

@@ -1,6 +1,6 @@
from unittest.mock import Mock
from app.s3_client.s3_csv_client import remove_blank_lines, set_metadata_on_csv_upload
from app.s3_client.s3_csv_client import set_metadata_on_csv_upload
def test_sets_metadata(client_request, mocker):
@@ -21,11 +21,3 @@ def test_sets_metadata(client_request, mocker):
MetadataDirective="REPLACE",
ServerSideEncryption="AES256",
)
def test_removes_blank_lines():
filedata = {
"data": "phone number\r\n15555555555\r\n\r\n\r\n\r\n\r\n\r\n\r\n\r\n\r\n"
}
file_data = remove_blank_lines(filedata)
assert file_data == {"data": "phone number\n15555555555"}