Merge pull request #511 from pmcelhaney/fix-open-handles-in-tests

make sure Jest closes all open handles

.github/workflows/coveralls.yaml

@@ -3,26 +3,24 @@ on: ["push"]
 name: Coveralls
 
 jobs:
-
   build:
     name: Coveralls
     runs-on: ubuntu-latest
     steps:
+      - uses: actions/checkout@v3
 
-    - uses: actions/checkout@v3
-
-    - name: Setup Node.js 16.x
-      uses: actions/setup-node@v3
-      with:
-        node-version: 16.x
-        cache: yarn
+      - name: Setup Node.js 16.x
+        uses: actions/setup-node@v3
+        with:
+          node-version: 16.x
+          cache: yarn
 
-    - name: install, run tests
-      run: |
-        yarn
-        yarn test --forceExit
+      - name: install, run tests
+        run: |
+          yarn
+          yarn test
 
-    - name: Coveralls
-      uses: coverallsapp/github-action@v2.2.1
-      with:
-        github-token: ${{ secrets.GITHUB_TOKEN }}
+      - name: Coveralls
+        uses: coverallsapp/github-action@v2.2.1
+        with:
+          github-token: ${{ secrets.GITHUB_TOKEN }}

.rtx.toml

@@ -0,0 +1,2 @@
+[tools]
+node = "18"

test/server/counterfact.test.js

@@ -134,5 +134,10 @@ describe("integration test", () => {
 
       await moduleLoader.stopWatching();
     });
+
+    // eslint-disable-next-line promise/avoid-new
+    await new Promise((resolve) => {
+      proxyTarget.close(resolve);
+    });
   });
 });

test/typescript-generator/specification.test.js

@@ -57,7 +57,10 @@ describe("a Specification", () => {
       hello: "world",
     });
 
-    await server.close();
+    // eslint-disable-next-line promise/avoid-new
+    await new Promise((resolve) => {
+      server.close(resolve);
+    });
   });
 
   it("returns a requirement for a URL", async () => {