Skip to content

Commit 88d42d3

Browse files
committed
chore: msg code as structured data not text
1 parent 799ac74 commit 88d42d3

File tree

3 files changed

+6
-2
lines changed

3 files changed

+6
-2
lines changed

packages/common/utilities/src/index.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
export {testPrescriptionsConfig, TestPrescriptionsConfig, getTestPrescriptions} from "./testConfig.js"
22
export const LOG_MESSAGES = {
33
PSU0001: "Transitioning item status.",
4-
PSU0002: "PSU0002 Notify request",
4+
PSU0002: "Notify request",
55
PSU0003: "Updated notification state"
66
} as const
77

packages/nhsNotifyLambda/src/utils/notify.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -124,9 +124,12 @@ function logNotificationRequest(logger: Logger,
124124
messageStatus: messageStatus
125125
})
126126
// Log each message individually for less memory intensive reporting
127+
const code = Object.keys(LOG_MESSAGES)
128+
.find(key => LOG_MESSAGES[key as keyof typeof LOG_MESSAGES] === LOG_MESSAGES.PSU0002)
127129
messages.forEach((message, index) => {
128130
const correspondingData = data.find(item => item.messageReference === message.messageReference)
129131
logger.info(LOG_MESSAGES.PSU0002, {
132+
reportCode: code,
130133
messageBatchReference,
131134
messageIndex: index,
132135
nhsNumber: message.recipient.nhsNumber,

packages/nhsNotifyLambda/tests/testUtils.test.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -845,8 +845,9 @@ describe("NHS notify lambda helper functions", () => {
845845
})
846846
)
847847
expect(infoSpy).toHaveBeenCalledWith(
848-
"PSU0002 Notify request",
848+
LOG_MESSAGES.PSU0002,
849849
expect.objectContaining({
850+
reportCode: "PSU0002",
850851
messageBatchReference: expect.any(String),
851852
messageIndex: expect.any(Number)
852853
})

0 commit comments

Comments
 (0)