Skip to content

Commit 5fef8a7

Browse files
Fix rename again
1 parent 5712e5e commit 5fef8a7

3 files changed

Lines changed: 10 additions & 6 deletions

File tree

lambdas/letter-updates-transformer/src/letter-updates-transformer.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ import {
99
PublishBatchCommand,
1010
PublishBatchRequestEntry,
1111
} from "@aws-sdk/client-sns";
12-
import { LetterEvent } from "@nhsdigital/nhs-notify-event-schemas-supplier-api/src";
12+
import { LetterStatusChangeEvent } from "@nhsdigital/nhs-notify-event-schemas-supplier-api/src";
1313
import { MetricEntry, buildEMFObject } from "@internal/helpers";
1414
import { Letter, LetterSchema } from "@internal/datastore";
1515
import { mapLetterToCloudEvent } from "@internal/event-builders/src";
@@ -33,7 +33,7 @@ export default function createHandler(deps: Deps): Handler<KinesisStreamEvent> {
3333
extractPayload(record, deps),
3434
);
3535

36-
const cloudEvents: LetterEvent[] = ddbRecords
36+
const cloudEvents: LetterStatusChangeEvent[] = ddbRecords
3737
.filter((record) => filterRecord(record, deps))
3838
.map((element) => extractNewLetter(element))
3939
.map((element) => mapLetterToCloudEvent(element, deps.env.EVENT_SOURCE));
@@ -59,7 +59,7 @@ export default function createHandler(deps: Deps): Handler<KinesisStreamEvent> {
5959
};
6060
}
6161

62-
function populateEventTypeMap(cloudEvents: LetterEvent[]) {
62+
function populateEventTypeMap(cloudEvents: LetterStatusChangeEvent[]) {
6363
const evtMap = new Map<string, number>();
6464
for (const event of cloudEvents) {
6565
evtMap.set(event.type, (evtMap.get(event.type) || 0) + 1);
@@ -142,14 +142,14 @@ function extractNewLetter(record: DynamoDBRecord): Letter {
142142
return LetterSchema.parse(unmarshall(newImage as any));
143143
}
144144

145-
function* generateBatches(events: LetterEvent[]) {
145+
function* generateBatches(events: LetterStatusChangeEvent[]) {
146146
for (let i = 0; i < events.length; i += BATCH_SIZE) {
147147
yield events.slice(i, i + BATCH_SIZE);
148148
}
149149
}
150150

151151
function buildMessage(
152-
event: LetterEvent,
152+
event: LetterStatusChangeEvent,
153153
index: number,
154154
): PublishBatchRequestEntry {
155155
return {

lambdas/upsert-letter/src/handler/upsert-handler.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,9 @@ function mapToInsertLetter(
9393
};
9494
}
9595

96-
function mapToUpdateLetter(upsertRequest: LetterStatusChangeEvent): UpdateLetter {
96+
function mapToUpdateLetter(
97+
upsertRequest: LetterStatusChangeEvent,
98+
): UpdateLetter {
9799
return {
98100
id: upsertRequest.data.domainId,
99101
eventId: upsertRequest.id,

tests/component-tests/integration-tests/urgent-letter-priority.spec.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -90,6 +90,8 @@ test.describe("Urgent Letter Priority Tests", () => {
9090

9191
function getVariantsWithUrgency(urgency: number) {
9292
const variants = Object.keys(variantUrgencyMap).filter(
93+
// eslint-disable-next-line security/detect-object-injection
94+
// safe has comes from map's keys which are controlled by us
9395
(variant) => variantUrgencyMap[variant] === urgency,
9496
);
9597
if (variants.length === 0) {

0 commit comments

Comments
 (0)