Skip to content

Commit e688206

Browse files
committed
CCM-11352: update file checking logic
1 parent 203d386 commit e688206

File tree

2 files changed

+19
-13
lines changed

2 files changed

+19
-13
lines changed

tests/test-team/helpers/events/event-cache-helper.ts

Lines changed: 19 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,12 @@ import {
99
$TemplateDeletedEventV1,
1010
$TemplateDraftedEventV1,
1111
} from '@nhsdigital/nhs-notify-event-schemas-template-management';
12-
import { differenceInSeconds, addHours } from 'date-fns';
12+
import {
13+
differenceInSeconds,
14+
addHours,
15+
startOfHour,
16+
endOfHour,
17+
} from 'date-fns';
1318
import { S3Helper } from '../s3-helper';
1419

1520
const $NHSNotifyTemplateEvent = z.discriminatedUnion('type', [
@@ -95,16 +100,16 @@ export class EventCacheHelper {
95100
JSON.parse(line)
96101
);
97102

98-
if (!success) {
99-
throw new Error(
100-
`Unrecognized event schema detected in S3 file: ${fileName}`,
101-
{
102-
cause: { error },
103-
}
104-
);
103+
if (success) {
104+
return data;
105105
}
106106

107-
return data;
107+
throw new Error(
108+
`Unrecognized event schema detected in S3 file: ${fileName}`,
109+
{
110+
cause: { error },
111+
}
112+
);
108113
});
109114

110115
events.push(...parsedEvents);
@@ -123,11 +128,13 @@ export class EventCacheHelper {
123128
private buildFilePaths(start: Date, toleranceInSeconds = 30): string[] {
124129
const paths = [this.buildPathPrefix(start)];
125130

126-
const end = addHours(start, 1);
131+
const end = addHours(startOfHour(start), 1);
127132

128-
const difference = differenceInSeconds(end, start);
133+
const difference = differenceInSeconds(endOfHour(start), start, {
134+
roundingMethod: 'ceil',
135+
});
129136

130-
if (difference >= toleranceInSeconds) {
137+
if (toleranceInSeconds >= difference) {
131138
paths.push(this.buildPathPrefix(end));
132139
}
133140

tests/test-team/template-mgmt-event-tests/digital-templates.event.spec.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,6 @@ test.describe('Event publishing - Digital', () => {
7272

7373
expect(updateResponse.status()).toBe(200);
7474

75-
// submit template - should be a completed event SUBMITTED
7675
const submitResponse = await request.patch(
7776
`${process.env.API_BASE_URL}/v1/template/${templateId}/submit`,
7877
{

0 commit comments

Comments
 (0)