Add test for too-early callback when existing files are clobbered

See AvianFlu#71 and related issues.
maxg · May 21, 2015 · 3c88d8a · 3c88d8a · jprichardson · Jun 2, 2015
1 parent 7c970fb
commit 3c88d8a
Showing 1 changed file with 20 additions and 0 deletions.
diff --git a/test/ncp.js b/test/ncp.js
@@ -65,6 +65,26 @@ describe('ncp', function () {
       });
     });
 
+    describe('when using clobber=true', function () {
+      before(function () {
+        this.originalCreateReadStream = fs.createReadStream;
+      });
+
+      after(function () {
+        fs.createReadStream = this.originalCreateReadStream;
+      });
+
+      it('the copy is complete after callback', function (cb) {
+        ncp(src, out, {clobber: true}, function(err) {
+          fs.createReadStream = function() {
+            cb(new Error('createReadStream after callback'));
+          };
+          assert.ifError(err);
+          process.nextTick(cb);
+        });
+      });
+    });
+
     describe('when using clobber=false', function () {
       it('the copy is completed successfully', function (cb) {
         ncp(src, out, function() {