CCM-9692: fix test helper simulatePassedValidation

Author: alexnuttall

tests/test-team/helpers/use-cases/simulate-passed-virus-scan.ts

@@ -6,6 +6,7 @@ import { Template } from '../types';
 type Config = {
   templateId: string;
   templateOwner: string;
+  hasTestData: boolean;
 };
 
 export class SimulatePassedValidation implements IUseCase<Template> {
@@ -28,8 +29,10 @@ export class SimulatePassedValidation implements IUseCase<Template> {
         },
         UpdateExpression: [
           'SET files.pdfTemplate.virusScanStatus = :virusScanStatus',
-          'files.testDataCsv.virusScanStatus = :virusScanStatus',
           'templateStatus = :readyForSubmissionStatus',
+          ...(this.#config.hasTestData
+            ? ['files.testDataCsv.virusScanStatus = :virusScanStatus']
+            : []),
         ].join(', '),
         ExpressionAttributeValues: {
           ':virusScanStatus': 'PASSED',

tests/test-team/template-mgmt-api-tests/delete-template.api.spec.ts

@@ -211,6 +211,7 @@ test.describe('DELETE /v1/template/:templateId', () => {
         new SimulatePassedValidation({
           templateId: createResult.template.id,
           templateOwner: user1.userId,
+          hasTestData: true,
         })
       );
 

tests/test-team/template-mgmt-api-tests/submit-template.api.spec.ts

@@ -149,6 +149,7 @@ test.describe('POST /v1/template/:templateId/submit', () => {
         new SimulatePassedValidation({
           templateId: createResult.template.id,
           templateOwner: user1.userId,
+          hasTestData: true,
         })
       );
 
@@ -244,6 +245,7 @@ test.describe('POST /v1/template/:templateId/submit', () => {
         new SimulatePassedValidation({
           templateId: createResult.template.id,
           templateOwner: user1.userId,
+          hasTestData: true,
         })
       );