Fix copy(Sync)Fallback tests for real

lib/copy-sync/__tests__/copyfilesync-fallback.test.js

@@ -1,25 +1,29 @@
-const fs = require('../../')
+const fs = require('graceful-fs')
 const path = require('path')
 
-/* global describe */
+/* global describe, beforeEach, afterEach */
 
 if (typeof fs.copyFileSync === 'function') {
-  // Also load copy-sync tests without fs.copyFileSync
-  describe('> when using copyFileSyncFallback', () => {
-    const originalCopyFile = fs.copyFileSync
+  // Also load copy tests without fs.copyFileSync
+  describe('> when using copyFileFallback', () => {
+    const originalCopyFileSync = fs.copyFileSync
+
+    beforeEach(() => {
+      // reset stubs
+      delete fs.copyFileSync
+    })
+
+    afterEach(() => {
+      fs.copyFileSync = originalCopyFileSync
+    })
 
-    delete fs.copyFileSync
     const fallbackTest = path.basename(__filename)
-    try {
-      fs.readdirSync(__dirname)
-        .filter(filename => filename.endsWith('.test.js') && filename !== fallbackTest)
-        .map(filename => {
-          const testModule = `./${filename}`
-          delete require.cache[require.resolve(testModule)]
-          require(testModule)
-        })
-    } finally {
-      fs.copyFileSync = originalCopyFile
-    }
+    fs.readdirSync(__dirname)
+      .filter(filename => filename.endsWith('.test.js') && filename !== fallbackTest)
+      .map(filename => {
+        const testModule = `./${filename}`
+        delete require.cache[require.resolve(testModule)]
+        require(testModule)
+      })
   })
 }

lib/copy/__tests__/copyfile-fallback.test.js

@@ -1,25 +1,29 @@
-const fs = require('../../')
+const fs = require('graceful-fs')
 const path = require('path')
 
-/* global describe */
+/* global describe, beforeEach, afterEach */
 
 if (typeof fs.copyFile === 'function') {
   // Also load copy tests without fs.copyFile
   describe('> when using copyFileFallback', () => {
     const originalCopyFile = fs.copyFile
 
-    delete fs.copyFile
-    const fallbackTest = path.basename(__filename)
-    try {
-      fs.readdirSync(__dirname)
-        .filter(filename => filename.endsWith('.test.js') && filename !== fallbackTest)
-        .map(filename => {
-          const testModule = `./${filename}`
-          delete require.cache[require.resolve(testModule)]
-          require(testModule)
-        })
-    } finally {
+    beforeEach(() => {
+      // reset stubs
+      delete fs.copyFile
+    })
+
+    afterEach(() => {
       fs.copyFile = originalCopyFile
-    }
+    })
+
+    const fallbackTest = path.basename(__filename)
+    fs.readdirSync(__dirname)
+      .filter(filename => filename.endsWith('.test.js') && filename !== fallbackTest)
+      .map(filename => {
+        const testModule = `./${filename}`
+        delete require.cache[require.resolve(testModule)]
+        require(testModule)
+      })
   })
 }