Fix travis (#7)

* optmize: optmize fs.sync().mkdir
* fix: test travis test
* release 0.2.4

Author: fishbar

.travis.yml

@@ -1,5 +1,6 @@
 language: node_js
 node_js:
-  - "0.12"
-  - "0.10"
   - "4"
+  - "6"
+  - "8"
+  - "10"

Makefile

@@ -7,5 +7,8 @@ install:
 
 test: install
 	@$(MOCHA) -r jscoverage ./test -R spec
-	
+
+test2:
+	@$(MOCHA) ./test -R spec	
+
 .PHONY: instal test

lib/sync.js

@@ -8,8 +8,9 @@ var path = require('path');
  * @param {Object}  [option] file option
  */
 exports.writeFile = function (fpath, data, option) {
+  fpath = path.resolve(fpath);
   var dir = path.dirname(fpath);
-  this.mkdir(dir);
+  exports.mkdir(dir);
   fs.writeFileSync(fpath, data, option);
 };
 
@@ -37,7 +38,7 @@ exports.rm = function (fpath) {
     ff.forEach(function (v) {
       self.rm(path.join(fpath, v));
     });
-    count ++;
+    count++;
     if (count > rm_max) {
       throw new Error('rm dir failed:' + fpath);
     }
@@ -46,7 +47,7 @@ exports.rm = function (fpath) {
 
 exports.mkdir = function (fpath, mode) {
   var p = [];
-  while (1) {
+  while (true) {
     try {
       fs.mkdirSync(fpath);
     } catch (e) {
@@ -55,10 +56,12 @@ exports.mkdir = function (fpath, mode) {
       } else if (e.code === 'ENOENT') {
         p.unshift(fpath);
         fpath = path.dirname(fpath);
+        continue;
       } else {
         throw e;
       }
     }
+    break;
   }
   p.forEach(function (v) {
     fs.mkdirSync(v, mode);
@@ -67,4 +70,4 @@ exports.mkdir = function (fpath, mode) {
 
 exports.copy = function (source, dest) {
 
-};
+};

package.json

@@ -2,7 +2,7 @@
   "author": "fish <[email protected]> (https://github.com/fishbar)",
   "name": "xfs",
   "description": "extends fs module, easy way to manipulate file system, support both sync functions and async functions",
-  "version": "0.2.2",
+  "version": "0.2.4",
   "homepage": "https://github.com/fishbar/xfs",
   "repository": {
     "type": "git",

test/index.js

@@ -214,7 +214,7 @@ describe('xfs.sync()', function () {
       expect(xfs.readFileSync('./tdir/sync/test.txt').toString()).to.be('abc');
     });
     if (compareVersion(process.version, 'v0.10.0')) {
-      it('should ok when save file with option to exist file', function() {
+      it('should ok when save file with option to exist file', function () {
         xfs.sync().save('./tdir/sync/test.txt', 'def', {flag: 'a'});
         expect(xfs.readFileSync('./tdir/sync/test.txt').toString()).to.be('abcdef');
       });
@@ -225,7 +225,7 @@ describe('xfs.sync()', function () {
     });
   });
   describe('mkdir', function () {
-    it('should be ok when path not exist', function (){
+    it('should be ok when path not exist', function () {
       var path = './tdir/sync/test';
       xfs.sync().mkdir(path);
       var stat = xfs.statSync(path);