fix: fix error on windows

unknown · unknown · commit 2ad1ed2746fc · 2017-05-15T22:26:06.000+08:00
diff --git a/index.js b/index.js
@@ -4,6 +4,8 @@ const _0777 = parseInt('0777', 8);
 
 module.exports = mkdirP;
 
+const errCode = process.platform === 'win32' ? 3 : 2;
+
 function mkdirP(p, opts, made) {
   if (!opts || typeof opts !== 'object') {
     opts = {
@@ -26,7 +28,7 @@ function mkdirP(p, opts, made) {
     made = made || p;
   } catch (err0) {
     switch (err0.number) {
-      case 2:
+      case errCode:
         made = mkdirP(path.dirname(p), opts, made);
         mkdirP(p, opts, made);
         break;
diff --git a/test/chmod.test.js b/test/chmod.test.js
@@ -19,21 +19,23 @@ for (let i = 0; i < 25; i++) {
 let file = ps.join('/');
 file = path.join(__dirname, file.slice(1));
 
-describe('chmod', () => {
-  it("chmod-pre", () => {
-    const mode = _0744;
-    mkdirp(file, mode);
-    const stat = fs.stat(file);
-    assert.ok(stat && stat.isDirectory(), 'should be directory');
-    assert.equal(stat && stat.mode & _0777, mode);
-  });
+if (process.platform !== 'win32') {
+  describe('chmod', () => {
+    it("chmod-pre", () => {
+      const mode = _0744;
+      mkdirp(file, mode);
+      const stat = fs.stat(file);
+      assert.ok(stat && stat.isDirectory(), 'should be directory');
+      assert.equal(stat && stat.mode & _0777, mode);
+    });
+
+    it("chmod", () => {
+      mkdirp(file, _0755);
+      const stat = fs.stat(file);
+      assert.ok(stat && stat.isDirectory(), 'should be directory');
+    });
 
-  it("chmod", () => {
-    mkdirp(file, _0755);
-    const stat = fs.stat(file);
-    assert.ok(stat && stat.isDirectory(), 'should be directory');
   });
-
-});
+}
 
 // test.run(console.DEBUG);
diff --git a/test/clobber.test.js b/test/clobber.test.js
@@ -33,7 +33,8 @@ describe('clobber', () => {
     try {
       mkdirp(file, _0755);
     } catch (e) {
-      if (e.number === 20) {
+      const errCode = process.platform === 'win32' ? 183 : 20;
+      if (e.number === errCode) {
         return;
       }
     }
diff --git a/test/index.js b/test/index.js
@@ -9,4 +9,4 @@ filelist.map(file => {
   }
 });
 
-process.exit(-test.run());
+process.exit(-test.run(console.DEBUG));
diff --git a/test/umask.test.js b/test/umask.test.js
@@ -7,20 +7,21 @@ const _0777 = parseInt('0777', 8);
 
 test.setup();
 
-describe('umask', () => {
-  it("implicit mode from umask", () => {
-    const x = Math.floor(Math.random() * Math.pow(16, 4)).toString(16);
-    const y = Math.floor(Math.random() * Math.pow(16, 4)).toString(16);
-    const z = Math.floor(Math.random() * Math.pow(16, 4)).toString(16);
+if (process.platform !== 'win32') {
+  describe('umask', () => {
+    it("implicit mode from umask", () => {
+      const x = Math.floor(Math.random() * Math.pow(16, 4)).toString(16);
+      const y = Math.floor(Math.random() * Math.pow(16, 4)).toString(16);
+      const z = Math.floor(Math.random() * Math.pow(16, 4)).toString(16);
 
-    const file = path.join(__dirname, 'tmp/' + [x, y, z].join('/'));
+      const file = path.join(__dirname, 'tmp/' + [x, y, z].join('/'));
 
-    mkdirp(file);
-    assert.ok(fs.exists(file), 'file created');
-    const stat = fs.stat(file);
-    assert.equal(stat.mode & _0777, _0777 & (~process.umask()));
-    assert.ok(stat.isDirectory(), 'target is a directory');
+      mkdirp(file);
+      assert.ok(fs.exists(file), 'file created');
+      const stat = fs.stat(file);
+      assert.equal(stat.mode & _0777, _0777 & (~process.umask()));
+      assert.ok(stat.isDirectory(), 'target is a directory');
+    });
   });
-});
-
+}
 //test.run(console.DEBUG);

Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,8 @@ describe('clobber', () => {`
`33`	`33`	`try {`
`34`	`34`	`mkdirp(file, _0755);`
`35`	`35`	`} catch (e) {`
`36`		`- if (e.number === 20) {`
	`36`	`+ const errCode = process.platform === 'win32' ? 183 : 20;`
	`37`	`+ if (e.number === errCode) {`
`37`	`38`	`return;`
`38`	`39`	`}`
`39`	`40`	`}`
Original file line number	Diff line number	Diff line change
`@@ -9,4 +9,4 @@ filelist.map(file => {`
`9`	`9`	`}`
`10`	`10`	`});`
`11`	`11`
`12`		`-process.exit(-test.run());`
	`12`	`+process.exit(-test.run(console.DEBUG));`