Skip to content

Commit 3ced052

Browse files
turn it on!
1 parent 1334be2 commit 3ced052

1 file changed

Lines changed: 6 additions & 5 deletions

File tree

lambdas/supplier-allocator/src/handler/allocate-handler.ts

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ function validateType(event: unknown) {
6161
async function getSupplierFromConfig(
6262
letterEvent: PreparedEvents,
6363
deps: Deps,
64-
): Promise<SupplierDetails | undefined> {
64+
): Promise<SupplierDetails> {
6565
try {
6666
const letterVariant: LetterVariant = await getVariantDetails(
6767
letterEvent.data.letterVariantId,
@@ -129,12 +129,12 @@ async function getSupplierFromConfig(
129129
};
130130
return supplierDetails;
131131
} catch (error) {
132-
deps.logger.info({
132+
deps.logger.error({
133133
description: "Error fetching supplier from config",
134134
err: error,
135135
variantId: letterEvent.data.letterVariantId,
136136
});
137-
return undefined;
137+
throw error;
138138
}
139139
}
140140

@@ -232,11 +232,12 @@ export default function createSupplierAllocatorHandler(deps: Deps): SQSHandler {
232232

233233
validateType(letterEvent);
234234

235-
const supplierSpec = getSupplier(letterEvent as PreparedEvents, deps);
236-
const supplierDetails = await getSupplierFromConfig(
235+
getSupplier(letterEvent as PreparedEvents, deps);
236+
const supplierDetails: SupplierDetails = await getSupplierFromConfig(
237237
letterEvent as PreparedEvents,
238238
deps,
239239
);
240+
const supplierSpec = supplierDetails?.supplierSpec;
240241

241242
deps.logger.info({
242243
description: "Resolved supplier details from config",

0 commit comments

Comments
 (0)