Wait for test hooks to resolve

OperationSpark · Feb 6, 2024 · 37b4459 · 37b4459
1 parent 8b7c355
commit 37b4459
Showing 1 changed file with 23 additions and 21 deletions.
diff --git a/test/test-projects.js b/test/test-projects.js
@@ -236,19 +236,25 @@ describe('projects', function () {
   });
 
   describe('#shelveProject()', function () {
-    function removeTestProjectFiles() {
-      [
-        `${projectsDirectory}/underpants`,
-        `${projectsDirectory}/scratch-pad`,
-        `${projectsDirectory}/_underpants`,
-        `${projectsDirectory}/__underpants`,
-        `${projectsDirectory}/_scratch-pad`,
-        `${projectsDirectory}/__scratch-pad`
-      ].forEach(path => {
-        if (fs.existsSync(path)) {
-          rimraf(path, () => {});
-        }
-      });
+    async function removeTestProjectFiles() {
+      return Promise.all(
+        [
+          `${projectsDirectory}/underpants`,
+          `${projectsDirectory}/scratch-pad`,
+          `${projectsDirectory}/_underpants`,
+          `${projectsDirectory}/__underpants`,
+          `${projectsDirectory}/_scratch-pad`,
+          `${projectsDirectory}/__scratch-pad`
+        ].map(
+          path =>
+            new Promise((resolve, reject) => {
+              if (fs.existsSync(path)) {
+                rimraf(path, resolve);
+              }
+              resolve(true);
+            })
+        )
+      );
     }
 
     beforeEach(removeTestProjectFiles);
@@ -269,14 +275,10 @@ describe('projects', function () {
       ).to.be.false;
       bddStdin('y\n');
       projects.shelveProject(dummySession.PROJECT[0]).then(function (resPath) {
-        expect(
-          fs.existsSync(path),
-          `path: "${path}" should not exist`
-        ).to.be.false;
-        expect(
-          fs.existsSync(newPath),
-          `path: "${newPath}" should exist`
-        ).to.be.true;
+        expect(fs.existsSync(path), `path: "${path}" should not exist`).to.be
+          .false;
+        expect(fs.existsSync(newPath), `path: "${newPath}" should exist`).to.be
+          .true;
         expect(resPath).to.equal(newPath);
         done();
       });