Skip to content

Commit e3fae42

Browse files
CCM-14616: Address review comments
1 parent d0a09d9 commit e3fae42

3 files changed

Lines changed: 11 additions & 9 deletions

File tree

lambdas/print-status-handler/src/__tests__/test-data.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,8 @@ export const acceptedLetterEvent = {
99
'letter-origin/digital-letters/letter/f47ac10b-58cc-4372-a567-0e02b2c3d479_2503cbd5-6722-4e90-9fbd-5f1e96d65c22',
1010
type: 'uk.nhs.notify.supplier-api.letter.ACCEPTED.v1',
1111
dataschema:
12-
'https://notify.nhs.uk/cloudevents/schemas/supplier-api/letter.ACCEPTED.1.0.16.schema.json',
13-
dataschemaversion: '1.0.16',
12+
'https://notify.nhs.uk/cloudevents/schemas/supplier-api/letter.ACCEPTED.1.0.17.schema.json',
13+
dataschemaversion: '1.0.17',
1414
time: '2023-06-20T12:00:00Z',
1515
recordedtime: '2023-06-20T12:00:00.250Z',
1616
severitynumber: 2,
@@ -44,8 +44,8 @@ export const failedLetterEvent = {
4444
'letter-origin/digital-letters/letter/f47ac10b-58cc-4372-a567-0e02b2c3d480_2503cbd5-6722-4e90-9fbd-5f1e96d65c22',
4545
type: 'uk.nhs.notify.supplier-api.letter.FAILED.v1',
4646
dataschema:
47-
'https://notify.nhs.uk/cloudevents/schemas/supplier-api/letter.FAILED.1.0.16.schema.json',
48-
dataschemaversion: '1.0.16',
47+
'https://notify.nhs.uk/cloudevents/schemas/supplier-api/letter.FAILED.1.0.17.schema.json',
48+
dataschemaversion: '1.0.17',
4949
time: '2023-06-20T13:00:00Z',
5050
recordedtime: '2023-06-20T13:00:00.250Z',
5151
severitynumber: 3,

lambdas/print-status-handler/src/apis/sqs-handler.ts

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -68,9 +68,11 @@ function validateRecord(
6868
return null;
6969
}
7070

71-
logger.info(
72-
`Successfully validated SQS record with messageId: ${messageId}, subject: ${item.data.origin.subject}`,
73-
);
71+
logger.info({
72+
description: 'Successfully validated SQS record',
73+
messageId,
74+
subject: item.data.origin.subject,
75+
});
7476

7577
return { messageId, event: item };
7678
} catch (error) {

tests/playwright/digital-letters-component-tests/print-status-handler.component.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ const baseLetterEvent = {
1515
id: '550e8400-e29b-41d4-a716-446655440001',
1616
specversion: '1.0',
1717
source: '/data-plane/supplier-api/prod/update-status',
18-
dataschemaversion: '1.0.16',
18+
dataschemaversion: '1.0.17',
1919
time: '2023-06-20T12:00:00Z',
2020
recordedtime: '2023-06-20T12:00:00.250Z',
2121
severitynumber: 2,
@@ -65,7 +65,7 @@ test.describe('Print status handler', () => {
6565
...baseLetterEvent,
6666
subject: `letter-origin/letter-rendering/letter/00f3b388-bbe9-41c9-9e76-052d37ee8988_${messageReference}`,
6767
type: `uk.nhs.notify.supplier-api.letter.${status}.v1`,
68-
dataschema: `https://notify.nhs.uk/cloudevents/schemas/supplier-api/letter.${status}.1.0.16.schema.json`,
68+
dataschema: `https://notify.nhs.uk/cloudevents/schemas/supplier-api/letter.${status}.1.0.17.schema.json`,
6969
data: {
7070
...baseLetterEvent.data,
7171
status,

0 commit comments

Comments
 (0)