ci: Properly handle open connections after the tests finish (#2185)

Author: dplewis

integration/test/helper.js

@@ -93,16 +93,6 @@ const defaultConfiguration = {
 };
 
 const openConnections = {};
-const destroyAliveConnections = function () {
-  for (const socketId in openConnections) {
-    try {
-      openConnections[socketId].destroy();
-      delete openConnections[socketId];
-    } catch (e) {
-      /* */
-    }
-  }
-};
 let parseServer;
 let server;
 
@@ -175,17 +165,18 @@ beforeAll(async () => {
 
 afterEach(async () => {
   await Parse.User.logOut();
-  // Connection close events are not immediate on node 10+... wait a bit
-  await sleep(0);
-  if (Object.keys(openConnections).length > 0) {
-    console.warn('There were open connections to the server left after the test finished');
-  }
   Parse.Storage._clear();
   await TestUtils.destroyAllDataPermanently(true);
-  destroyAliveConnections();
   if (didChangeConfiguration) {
     await reconfigureServer();
   }
 });
 
+afterAll(() => {
+  // Jasmine process counts as one open connection
+  if (Object.keys(openConnections).length > 1) {
+    console.warn('There were open connections to the server left after the test finished');
+  }
+});
+
 module.exports = { twitterAuthData };