Skip to content

Commit

Permalink
small cleanup
Browse files Browse the repository at this point in the history
  • Loading branch information
mydea committed Aug 9, 2024
1 parent f456fbf commit 657939a
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
5 changes: 3 additions & 2 deletions dev-packages/node-integration-tests/utils/runner.ts
Original file line number Diff line number Diff line change
Expand Up @@ -365,15 +365,16 @@ export function createRunner(...paths: string[]) {
}
}

const serverStartup = withSentryServer ? createBasicSentryServer(newEnvelope) : Promise.resolve([]);
const serverStartup = withSentryServer
? createBasicSentryServer(newEnvelope)
: Promise.resolve([undefined, undefined]);

const dockerStartup: Promise<VoidFunction | undefined> = dockerOptions
? runDockerCompose(dockerOptions)
: Promise.resolve(undefined);

const startup = Promise.all([dockerStartup, serverStartup]);

// eslint-disable-next-line @typescript-eslint/no-floating-promises
startup
.then(([dockerChild, [mockServerPort, mockServerClose]]) => {
if (mockServerClose) {
Expand Down
2 changes: 1 addition & 1 deletion dev-packages/node-integration-tests/utils/server.ts
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ import express from 'express';
*/
export function createBasicSentryServer(
onEnvelope: (env: Envelope) => void,
): Promise<[port: number, close: (callback?: () => void) => void]> {
): Promise<[port: number, close: () => void]> {
const app = express();

app.use(express.raw({ type: () => true, inflate: true, limit: '100mb' }));
Expand Down

0 comments on commit 657939a

Please sign in to comment.