test: Wait for test server to close (#11117)

Author: lforst

packages/node-experimental/test/transports/http.test.ts

@@ -80,15 +80,17 @@ const defaultOptions = {
 // empty function to keep test output clean
 const consoleWarnSpy = jest.spyOn(console, 'warn').mockImplementation(() => {});
 
-describe('makeNewHttpTransport()', () => {
-  afterEach(() => {
-    jest.clearAllMocks();
-
-    if (testServer) {
-      testServer.close();
-    }
-  });
+afterEach(done => {
+  jest.clearAllMocks();
+
+  if (testServer && testServer.listening) {
+    testServer.close(done);
+  } else {
+    done();
+  }
+});
 
+describe('makeNewHttpTransport()', () => {
   describe('.send()', () => {
     it('should correctly send envelope to server', async () => {
       await setupTestServer({ statusCode: SUCCESS }, (req, body) => {

packages/node-experimental/test/transports/https.test.ts

@@ -82,15 +82,17 @@ const defaultOptions = {
   recordDroppedEvent: () => undefined, // noop
 };
 
-describe('makeNewHttpsTransport()', () => {
-  afterEach(() => {
-    jest.clearAllMocks();
-
-    if (testServer) {
-      testServer.close();
-    }
-  });
+afterEach(done => {
+  jest.clearAllMocks();
+
+  if (testServer && testServer.listening) {
+    testServer.close(done);
+  } else {
+    done();
+  }
+});
 
+describe('makeNewHttpsTransport()', () => {
   describe('.send()', () => {
     it('should correctly send envelope to server', async () => {
       await setupTestServer({ statusCode: SUCCESS }, (req, body) => {

packages/node/test/transports/http.test.ts

@@ -83,15 +83,17 @@ const defaultOptions = {
 // empty function to keep test output clean
 const consoleWarnSpy = jest.spyOn(console, 'warn').mockImplementation(() => {});
 
-describe('makeNewHttpTransport()', () => {
-  afterEach(() => {
-    jest.clearAllMocks();
-
-    if (testServer) {
-      testServer.close();
-    }
-  });
+afterEach(done => {
+  jest.clearAllMocks();
+
+  if (testServer && testServer.listening) {
+    testServer.close(done);
+  } else {
+    done();
+  }
+});
 
+describe('makeNewHttpTransport()', () => {
   describe('.send()', () => {
     it('should correctly send envelope to server', async () => {
       await setupTestServer({ statusCode: SUCCESS }, (req, body) => {

packages/node/test/transports/https.test.ts

@@ -84,15 +84,17 @@ const defaultOptions = {
   recordDroppedEvent: () => undefined, // noop
 };
 
-describe('makeNewHttpsTransport()', () => {
-  afterEach(() => {
-    jest.clearAllMocks();
-
-    if (testServer) {
-      testServer.close();
-    }
-  });
+afterEach(done => {
+  jest.clearAllMocks();
+
+  if (testServer && testServer.listening) {
+    testServer.close(done);
+  } else {
+    done();
+  }
+});
 
+describe('makeNewHttpsTransport()', () => {
   describe('.send()', () => {
     it('should correctly send envelope to server', async () => {
       await setupTestServer({ statusCode: SUCCESS }, (req, body) => {