Various changes (#48)

[fixes #45] [fixes #47] Added EEXIST and ENOTDIR to errors

Author: stevenvachon

.gitignore

@@ -12,3 +12,4 @@ CVS/
 .com.apple.timemachine.supported
 tests/out/
 .nyc_output
+package-lock.json

.travis.yml

@@ -2,3 +2,4 @@ language: node_js
 node_js:
 - '4'
 - '6'
+- '8'

lib/index.js

@@ -113,7 +113,13 @@ var createDirs = function(dirs, to, options, callback) {
         if (to && typeof to === 'string') {
             fs.stat(to, stack.add(function(err, s) {
                 if (s && !s.isDirectory()) {
-                    options.errors.push(new Error(to + ' exists and is not a directory, can not create'));
+                    /*istanbul ignore next*/
+                    err = new Error(to + ' exists and is not a directory, can not create');
+                    /*istanbul ignore next*/
+                    err.code = 'ENOTDIR';
+                    /*istanbul ignore next*/
+                    err.errno = 27;
+                    options.errors.push(err);
                 } else {
                     mkdirp(to, stat.mode, stack.add(function(err) {
                         /*istanbul ignore next*/
@@ -135,13 +141,19 @@ var copyFile = function(from, to, options, callback) {
     var dir = path.dirname(to);
     mkdirp(dir, function() {
         fs.stat(to, function(statError) {
+            var err;
             if(!statError && options.overwrite !== true) {
-              return callback(new Error('File '+to+' exists'));
+                /*istanbul ignore next*/
+                err = new Error('File '+to+' exists');
+                /*istanbul ignore next*/
+                err.code = 'EEXIST';
+                /*istanbul ignore next*/
+                err.errno = 47;
+                return callback(err);
             }
 
             var fromFile = fs.createReadStream(from),
                 toFile = fs.createWriteStream(to),
-                err,
                 called = false,
                 cb = function(e) {
                     if (!called) {

package.json

@@ -16,7 +16,7 @@
     "github-changes": "^1.0.4",
     "jenkins-mocha": "^4.0.0",
     "jshint": "^2.9.2",
-    "nyc": "^10.0.0",
+    "nyc": "^11.0.2",
     "yui-lint": "~0.2.0"
   },
   "contributors": [

tests/full.js

@@ -304,6 +304,8 @@ describe('cpr test suite', function() {
                 assert.ok(errs.list);
                 assert.ok(errs.list[0]);
                 assert.ok(errs.list[0].message.match(/exists and is not a directory, can not create/));
+                assert.equal(errs.list[0].code, 'ENOTDIR');
+                assert.equal(errs.list[0].errno, 27);
                 done();
             });
         });
@@ -371,6 +373,8 @@ describe('cpr test suite', function() {
                 assert.equal(undefined, status);
                 assert(err instanceof Error);
                 assert.ok(/^File .* exists$/.test(err.message));
+                assert.equal(err.code, 'EEXIST');
+                assert.equal(err.errno, 47);
                 done();
             });
         });