Skip to content

Commit 8c02be7

Browse files
committed
fix: respond to review comments
1 parent a9bfc15 commit 8c02be7

File tree

2 files changed

+6
-8
lines changed

2 files changed

+6
-8
lines changed

src/scripts/cronjobs/updateBreachesInRemoteSettings/updateBreachesInRemoteSettings.ts

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -59,15 +59,13 @@ export async function main(parentLogger: Logger) {
5959

6060
// Add handlers for ensuring all logs are sent before shutdown
6161
async function shutdown() {
62-
return new Promise<void>(async (resolve) => {
63-
logger.on("finish", () => {
64-
resolve();
65-
});
62+
await new Promise<void>((resolve) => {
63+
logger.on("finish", () => resolve());
6664
logger.end();
67-
if (Sentry.isInitialized()) {
68-
await Sentry.flush();
69-
}
7065
});
66+
if (Sentry.isInitialized()) {
67+
await Sentry.flush();
68+
}
7169
}
7270

7371
process.on("SIGTERM", async () => {

src/test/helpers/mockLogger.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ export function mockLogger() {
1010
return {
1111
info: jest.fn(),
1212
error: jest.fn(),
13-
wran: jest.fn(),
13+
warn: jest.fn(),
1414
debug: jest.fn(),
1515
} as unknown as Logger;
1616
}

0 commit comments

Comments
 (0)