CCM-9247: Add error log

Author: chris-elliott-nhsd

lambdas/backend-api/src/__tests__/templates/api/process-proof.test.ts

@@ -2,6 +2,7 @@ import { mockDeep } from 'jest-mock-extended';
 import { createHandler } from '../../../templates/api/process-proof';
 import { TemplateRepository } from '../../../templates/infra';
 import { LetterFileRepository } from '@backend-api/templates/infra/letter-file-repository';
+import type { Logger } from 'nhs-notify-web-template-management-utils/logger';
 
 test.each([
   [
@@ -35,6 +36,7 @@ test.each([
     const handler = createHandler({
       templateRepository,
       letterFileRepository,
+      logger: mockDeep<Logger>(),
     });
 
     await handler({

lambdas/backend-api/src/templates/api/process-proof.ts

@@ -2,14 +2,17 @@ import { guardDutyEventValidator } from 'nhs-notify-web-template-management-util
 import type { TemplateRepository } from '../infra';
 import { LetterProofRepository } from '../infra/letter-proof-repository';
 import { LetterFileRepository } from '../infra/letter-file-repository';
+import type { Logger } from 'nhs-notify-web-template-management-utils/logger';
 
 export const createHandler =
   ({
     templateRepository,
     letterFileRepository,
+    logger,
   }: {
     templateRepository: TemplateRepository;
     letterFileRepository: LetterFileRepository;
+    logger: Logger;
   }) =>
   async (event: unknown) => {
     const {
@@ -38,6 +41,11 @@ export const createHandler =
         versionId,
         internalKey
       );
+    } else {
+      logger.error({
+        description: 'File found that did not pass virus scan',
+        objectKey,
+      });
     }
 
     // we will copy to the download bucket here as well

lambdas/backend-api/src/templates/container.ts

@@ -83,4 +83,5 @@ export const createTemplateRepositoryContainer = () => {
 export const createLetterFileRepositoryAndTemplateRepositoryContainer = () => ({
   ...createTemplateRepositoryContainer(),
   ...createLetterFileRepositoryContainer(),
+  logger,
 });