Skip to content

Commit 0c8fb79

Browse files
committed
CCM-14480: Increase all test timeout
1 parent 6338cb3 commit 0c8fb79

11 files changed

Lines changed: 53 additions & 48 deletions

tests/playwright/digital-letters-component-tests/file-scanner.component.spec.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@ test('should extract PDF from DocumentReference and store in unscanned bucket wi
9999
});
100100

101101
test('should handle validation errors by sending messages to DLQ', async () => {
102-
test.setTimeout(120_000);
102+
test.setTimeout(160_000);
103103
const messageReference = uuidv4();
104104
const senderId = 'TEST_SENDER_002';
105105
const documentReferenceKey = `document-reference/${messageReference}`;
@@ -150,7 +150,7 @@ test('should handle validation errors by sending messages to DLQ', async () => {
150150
const expectedKey = `${ENV}/${messageReference}.pdf`;
151151
const expectedUri = `s3://${UNSCANNED_FILES_BUCKET}/${expectedKey}`;
152152
await expect(getS3ObjectBufferFromUri(expectedUri)).rejects.toThrow();
153-
}, 100);
153+
}, 150);
154154
// Verify there is a message in the DLQ
155-
await expectMessageContainingString(FILE_SCANNER_DLQ_NAME, eventId, 100);
155+
await expectMessageContainingString(FILE_SCANNER_DLQ_NAME, eventId, 150);
156156
});

tests/playwright/digital-letters-component-tests/mesh-acknowledge.component.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -137,7 +137,7 @@ test.describe('Digital Letters - Mesh Acknowledger', () => {
137137
});
138138

139139
test('should send invalid event to dlq', async () => {
140-
test.setTimeout(120_000);
140+
test.setTimeout(160_000);
141141

142142
const letterId = uuidv4();
143143

@@ -161,7 +161,7 @@ test.describe('Digital Letters - Mesh Acknowledger', () => {
161161
await expectMessageContainingString(
162162
MESH_ACKNOWLEDGE_DLQ_NAME,
163163
letterId,
164-
100,
164+
150,
165165
);
166166
});
167167
});

tests/playwright/digital-letters-component-tests/mesh-poll-download.component.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,7 @@ test.describe('Digital Letters - MESH Poll and Download', () => {
112112
});
113113

114114
test('should send message to mesh-download DLQ when download fails', async () => {
115-
test.setTimeout(120_000);
115+
test.setTimeout(160_000);
116116

117117
const invalidMeshMessageId = `${Date.now()}_DLQ_${uuidv4().slice(0, 8)}`;
118118
const messageReference = uuidv4();
@@ -148,7 +148,7 @@ test.describe('Digital Letters - MESH Poll and Download', () => {
148148
await expectMessageContainingString(
149149
MESH_DOWNLOAD_DLQ_NAME,
150150
invalidMeshMessageId,
151-
100,
151+
150,
152152
);
153153
});
154154

tests/playwright/digital-letters-component-tests/move-scanned-files.component.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -179,7 +179,7 @@ test.describe('Digital Letters - Move Scanned Files', () => {
179179
});
180180

181181
test('given file without REQUIRED metadata, when uploaded into S3 bucket and guardduty scan passes then it goes to DLQ', async () => {
182-
test.setTimeout(120_000);
182+
test.setTimeout(160_000);
183183
const messageReference = uuidv4();
184184
const objectKey = `${PREFIX_DL_FILES}${messageReference}.txt`;
185185

@@ -202,7 +202,7 @@ test.describe('Digital Letters - Move Scanned Files', () => {
202202
await expectMessageContainingString(
203203
MOVE_SCANNED_FILES_DLQ_NAME,
204204
objectKey,
205-
100,
205+
150,
206206
);
207207
});
208208
});

tests/playwright/digital-letters-component-tests/pdm-poll.component.spec.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -236,7 +236,7 @@ test.describe('PDM Poll', () => {
236236
});
237237

238238
test('should send invalid event to poll dlq', async () => {
239-
test.setTimeout(120_000);
239+
test.setTimeout(160_000);
240240

241241
const eventId = uuidv4();
242242
const resourceId = 'b8f2b194-31e1-3719-aaf9-a9195e35e692';
@@ -272,9 +272,9 @@ test.describe('PDM Poll', () => {
272272
);
273273

274274
expect(eventLogEntry.length).toEqual(1);
275-
}, 100),
275+
}, 150),
276276

277-
expectMessageContainingString(PDM_POLL_DLQ_NAME, eventId, 100),
277+
expectMessageContainingString(PDM_POLL_DLQ_NAME, eventId, 150),
278278
]);
279279
});
280280
});

tests/playwright/digital-letters-component-tests/print-analyser.component.spec.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@ test.describe('Print analyser', () => {
8686
});
8787

8888
test('should send invalid event to print analyser dlq', async () => {
89-
test.setTimeout(120_000);
89+
test.setTimeout(160_000);
9090

9191
// Send file.safe event with missing data properties
9292
const messageReference = uuidv4();
@@ -111,12 +111,12 @@ test.describe('Print analyser', () => {
111111
);
112112

113113
expect(eventLogEntry.length).toEqual(1);
114-
}, 100),
114+
}, 150),
115115

116116
expectMessageContainingString(
117117
PRINT_ANALYSER_DLQ_NAME,
118118
messageReference,
119-
100,
119+
150,
120120
),
121121
]);
122122
});

tests/playwright/digital-letters-component-tests/print-sender.component.spec.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ test.describe('Digital Letters - Print Sender', () => {
7171
});
7272

7373
test('should send invalid event to print sender dlq', async () => {
74-
test.setTimeout(120_000);
74+
test.setTimeout(160_000);
7575

7676
const messageReference = uuidv4();
7777
const event = {
@@ -110,12 +110,12 @@ test.describe('Digital Letters - Print Sender', () => {
110110
);
111111

112112
expect(eventLogEntry.length).toBeGreaterThanOrEqual(1);
113-
}, 100),
113+
}, 150),
114114

115115
expectMessageContainingString(
116116
PRINT_SENDER_DLQ_NAME,
117117
messageReference,
118-
100,
118+
150,
119119
),
120120
]);
121121
});

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

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ test.describe('Print status handler', () => {
9595
}
9696

9797
test('should send invalid event to print status handler dlq', async () => {
98-
test.setTimeout(120_000);
98+
test.setTimeout(160_000);
9999

100100
const messageReference = uuidv4();
101101

@@ -129,12 +129,12 @@ test.describe('Print status handler', () => {
129129
);
130130

131131
expect(eventLogEntry.length).toEqual(1);
132-
}, 100),
132+
}, 150),
133133

134134
expectMessageContainingString(
135135
PRINT_STATUS_HANDLER_DLQ_NAME,
136136
messageReference,
137-
100,
137+
150,
138138
),
139139
]);
140140
});

tests/playwright/digital-letters-component-tests/send-reports-trust.component.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@ test.describe('Digital Letters - Send reports to Trust', () => {
9999
});
100100

101101
test('should send message to report-sender DLQ when file does not exists', async () => {
102-
test.setTimeout(120_000);
102+
test.setTimeout(160_000);
103103

104104
const missingReportFileName = 'report-does-not-exist.csv';
105105

@@ -108,7 +108,7 @@ test.describe('Digital Letters - Send reports to Trust', () => {
108108
await expectMessageContainingString(
109109
REPORT_SENDER_DLQ_NAME,
110110
missingReportFileName,
111-
100,
111+
150,
112112
);
113113
});
114114
});

tests/playwright/digital-letters-component-tests/ttl-create.component.spec.ts

Lines changed: 28 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ import { getTtl } from 'helpers/dynamodb-helpers';
1515
import eventPublisher from 'helpers/event-bus-helpers';
1616
import expectToPassEventually from 'helpers/expectations';
1717
import { expectMessageContainingString, purgeQueue } from 'helpers/sqs-helpers';
18+
import { promiseHooks } from 'node:v8';
1819
import { v4 as uuidv4 } from 'uuid';
1920

2021
test.describe('Digital Letters - Create TTL', () => {
@@ -128,7 +129,7 @@ test.describe('Digital Letters - Create TTL', () => {
128129
});
129130

130131
test('should send invalid event to dlq', async () => {
131-
test.setTimeout(250_000);
132+
test.setTimeout(160_000);
132133

133134
const letterId = uuidv4();
134135
const messageUri = `https://example.com/ttl/resource/${letterId}`;
@@ -149,23 +150,25 @@ test.describe('Digital Letters - Create TTL', () => {
149150
() => true,
150151
);
151152

152-
await expectToPassEventually(async () => {
153-
const eventLogEntry = await getLogsFromCloudwatch(
154-
CREATE_TTL_LAMBDA_LOG_GROUP_NAME,
155-
[
156-
'$.message.description = "Error parsing ttl queue entry"',
157-
`$.message.err[0].params.additionalProperty = "${unexpectedField}"`,
158-
],
159-
);
160-
161-
expect(eventLogEntry.length).toEqual(1);
162-
}, 120);
163-
164-
await expectMessageContainingString(CREATE_TTL_DLQ_NAME, letterId, 120);
153+
await Promise.all([
154+
expectToPassEventually(async () => {
155+
const eventLogEntry = await getLogsFromCloudwatch(
156+
CREATE_TTL_LAMBDA_LOG_GROUP_NAME,
157+
[
158+
'$.message.description = "Error parsing ttl queue entry"',
159+
`$.message.err[0].params.additionalProperty = "${unexpectedField}"`,
160+
],
161+
);
162+
163+
expect(eventLogEntry.length).toEqual(1);
164+
}, 150),
165+
166+
expectMessageContainingString(CREATE_TTL_DLQ_NAME, letterId, 150),
167+
]);
165168
});
166169

167170
test('should send events from unknown sender to dlq', async () => {
168-
test.setTimeout(250_000);
171+
test.setTimeout(160_000);
169172

170173
const letterId = uuidv4();
171174
const messageUri = `https://example.com/ttl/resource/${letterId}`;
@@ -186,15 +189,17 @@ test.describe('Digital Letters - Create TTL', () => {
186189
messageDownloadedValidator,
187190
);
188191

189-
await expectToPassEventually(async () => {
190-
const eventLogEntry = await getLogsFromCloudwatch(
191-
CREATE_TTL_LAMBDA_LOG_GROUP_NAME,
192-
[`$.message.description = "Sender ${senderId} could not be retrieved"`],
193-
);
192+
await Promise.all([
193+
expectToPassEventually(async () => {
194+
const eventLogEntry = await getLogsFromCloudwatch(
195+
CREATE_TTL_LAMBDA_LOG_GROUP_NAME,
196+
[`$.message.description = "Sender ${senderId} could not be retrieved"`],
197+
);
194198

195-
expect(eventLogEntry.length).toEqual(1);
196-
}, 120);
199+
expect(eventLogEntry.length).toEqual(1);
200+
}, 150),
197201

198-
await expectMessageContainingString(CREATE_TTL_DLQ_NAME, letterId, 120);
202+
expectMessageContainingString(CREATE_TTL_DLQ_NAME, letterId, 150),
203+
]);
199204
});
200205
});

0 commit comments

Comments
 (0)