Merge pull request #225 from raszi/gh-211

fix #211: existing tests must clean up after themselves
raszi · Jan 27, 2020 · ec35e59 · ec35e59
2 parents 3ffd509 + 6de7196
commit ec35e59
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 8 deletions.
diff --git a/test/dir-test.js b/test/dir-test.js
@@ -63,7 +63,7 @@ describe('tmp', function () {
           try {
             assertions.assertDoesNotExist(stdout);
           } catch (err) {
-            rimraf.sync(stdout);
+            //rimraf.sync(stdout);
             return done(err);
           }
           done();
@@ -76,8 +76,8 @@ describe('tmp', function () {
           if (!stderr) return done(new Error('stderr expected'));
           try {
             assertions.assertExists(stdout);
-          } catch (err) {
             rimraf.sync(stdout);
+          } catch (err) {
             return done(err);
           }
           done();
@@ -90,8 +90,8 @@ describe('tmp', function () {
           if (stderr) return done(new Error(stderr));
           try {
             assertions.assertExists(stdout);
-          } catch (err) {
             rimraf.sync(stdout);
+          } catch (err) {
             return done(err);
           }
           done();
@@ -137,8 +137,8 @@ describe('tmp', function () {
               assertions.assertExists(path.join(stdout, 'symlinkme-target'), true);
             else
               assertions.assertExists(path.join(stdout, 'symlinkme-target'));
-          } catch (err) {
             rimraf.sync(stdout);
+          } catch (err) {
             return done(err);
           }
           done();

diff --git a/test/file-test.js b/test/file-test.js
@@ -86,8 +86,8 @@ describe('tmp', function () {
           if (!stderr) return done(new Error('stderr expected'));
           try {
             assertions.assertExists(stdout, true);
-          } catch (err) {
             rimraf.sync(stdout);
+          } catch (err) {
             return done(err);
           }
           done();
@@ -100,8 +100,8 @@ describe('tmp', function () {
           if (stderr) return done(new Error(stderr));
           try {
             assertions.assertExists(stdout, true);
-          } catch (err) {
             rimraf.sync(stdout);
+          } catch (err) {
             return done(err);
           }
           done();
@@ -139,5 +139,4 @@ describe('tmp', function () {
       });
     });
   });
-});
-
+});