Skip to content

Commit 516ac9a

Browse files
committed
resolve lint2
1 parent bac82fe commit 516ac9a

File tree

2 files changed

+1
-4
lines changed

2 files changed

+1
-4
lines changed

lambdas/id_sync/src/id_sync.py

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@
1717
@logging_decorator(prefix="id_sync", stream_name=STREAM_NAME)
1818
def handler(event_data: Dict[str, Any], _context) -> Dict[str, Any]:
1919
try:
20-
logger.info("id_sync handler invoked")
2120
event = AwsLambdaEvent(event_data)
2221
records = event.records
2322

@@ -44,11 +43,9 @@ def handler(event_data: Dict[str, Any], _context) -> Dict[str, Any]:
4443
return response
4544

4645
except IdSyncException as e:
47-
# Preserve domain exceptions but ensure they're logged
4846
logger.exception(f"id_sync error: {e.message}")
4947
raise
5048
except Exception:
5149
msg = "Error processing id_sync event"
5250
logger.exception(msg)
53-
# Raise a domain exception with a predictable message for callers/tests
5451
raise IdSyncException(message=msg)

lambdas/id_sync/tests/test_record_processor.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ def test_process_record_pds_returns_none_id(self):
8989
# Act & Assert
9090
result = process_record(test_record)
9191
self.assertEqual(result["status"], "success")
92-
self.assertEqual(result["message"], f"No patient ID found for NHS number")
92+
self.assertEqual(result["message"], "No patient ID found for NHS number")
9393
self.mock_ieds_check_exist.assert_not_called()
9494
self.mock_ieds_update_patient_id.assert_not_called()
9595

0 commit comments

Comments
 (0)