Skip to content

Commit fa543fd

Browse files
committed
Removed vaccs type and supplier info from config error message
1 parent dda608a commit fa543fd

File tree

2 files changed

+1
-4
lines changed

2 files changed

+1
-4
lines changed

filenameprocessor/src/file_name_processor.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -149,8 +149,7 @@ def handle_record(record) -> dict:
149149
except Exception as error: # pylint: disable=broad-except
150150
logger.error("Error uploading to cache for file '%s': %s", file_key, error)
151151
message = "Failed to upload file content to cache"
152-
return {"statusCode": 500, "message": message, "file_key": file_key, "error": str(error),
153-
"vaccine_type": vaccine_type, "supplier": supplier}
152+
return {"statusCode": 500, "message": message, "file_key": file_key, "error": str(error)}
154153

155154
else:
156155
logger.error("Unable to process file %s due to unexpected bucket name %s", file_key, bucket_name)

filenameprocessor/tests/test_lambda_handler.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -484,8 +484,6 @@ def test_elasticcache_failure_handled(self):
484484
self.assertEqual(result["message"], "Failed to upload file content to cache")
485485
self.assertEqual(result["file_key"], "testfile.csv")
486486
self.assertIn("error", result)
487-
self.assertEqual(result["vaccine_type"], "unknown")
488-
self.assertEqual(result["supplier"], "unknown")
489487

490488
mock_logger.error.assert_called_once()
491489
logged_msg = mock_logger.error.call_args[0][0]

0 commit comments

Comments
 (0)