Merge pull request #92 from segayuu/Refacter-test

Refacter test

Author: segayuu

test/index.js

@@ -4,7 +4,7 @@ const chai = require('chai');
 
 chai.use(require('chai-as-promised'));
 
-describe('hexo-cli', function() {
+describe('hexo-cli', () => {
   require('./scripts/find_pkg');
   require('./scripts/context');
   require('./scripts/hexo');

test/scripts/context.js

@@ -1,42 +1,40 @@
 'use strict';
 
-const should = require('chai').should(); // eslint-disable-line
+require('chai').should();
 const sinon = require('sinon');
 
-describe('context', function() {
+describe('context', () => {
   const Context = require('../../lib/context');
 
-  describe('call', function() {
+  describe('call', () => {
     const hexo = new Context();
-    const spy = sinon.spy(function(args) {
-      return args;
-    });
+    const spy = sinon.spy(args => args);
 
     hexo.extend.console.register('test', spy);
 
-    it('success', function() {
+    it('success', () => {
       const args = {foo: 'bar'};
 
-      return hexo.call('test', args).then(function(result) {
+      return hexo.call('test', args).then(result => {
         result.should.eql(args);
         spy.calledOnce.should.be.true;
         spy.lastCall.args[0].should.eql(args);
         spy.resetHistory();
       });
     });
 
-    it('console not registered', function() {
+    it('console not registered', () => {
       const hexo = new Context();
 
-      return hexo.call('wtf').catch(function(err) {
+      return hexo.call('wtf').catch(err => {
         err.should.have.property('message', 'Console `wtf` has not been registered yet!');
       });
     });
 
-    it('with callback', function(done) {
+    it('with callback', done => {
       const args = {foo: 'bar'};
 
-      hexo.call('test', args, function(err, result) {
+      hexo.call('test', args, (err, result) => {
         if (err) return done(err);
 
         result.should.eql(args);
@@ -47,8 +45,8 @@ describe('context', function() {
       });
     });
 
-    it('with callback but no args', function(done) {
-      hexo.call('test', function(err) {
+    it('with callback but no args', done => {
+      hexo.call('test', err => {
         if (err) return done(err);
 
         spy.calledOnce.should.be.true;
@@ -58,20 +56,20 @@ describe('context', function() {
     });
   });
 
-  describe('exit', function() {
+  describe('exit', () => {
     let hexo, fatal;
 
-    beforeEach(function() {
+    beforeEach(() => {
       hexo = new Context();
       fatal = hexo.log.fatal = sinon.spy();
     });
 
-    it('no error', function() {
+    it('no error', () => {
       hexo.exit();
       fatal.called.should.be.false;
     });
 
-    it('with error', function() {
+    it('with error', () => {
       hexo.exit(new Error('error test'));
       fatal.calledOnce.should.be.true;
     });

test/scripts/find_pkg.js

@@ -1,80 +1,64 @@
 'use strict';
 
-const should = require('chai').should(); // eslint-disable-line
+const should = require('chai').should();
 const fs = require('hexo-fs');
 const pathFn = require('path');
 
-describe('Find package', function() {
+describe('Find package', () => {
   const findPkg = require('../../lib/find_pkg');
   const baseDir = pathFn.join(__dirname, 'find_pkg_test');
 
-  after(function() {
-    return fs.rmdir(baseDir);
-  });
+  after(() => fs.rmdir(baseDir));
 
-  it('not found', function() {
-    return findPkg(baseDir, {}).then(function(path) {
-      should.not.exist(path);
-    });
-  });
+  it('not found', () => findPkg(baseDir, {}).then(path => {
+    should.not.exist(path);
+  }));
 
-  it('found', function() {
+  it('found', () => {
     const pkgPath = pathFn.join(baseDir, 'package.json');
 
-    return fs.writeFile(pkgPath, '{"hexo": {}}').then(function() {
-      return findPkg(baseDir, {});
-    }).then(function(path) {
+    return fs.writeFile(pkgPath, '{"hexo": {}}').then(() => findPkg(baseDir, {})).then(path => {
       path.should.eql(baseDir);
       return fs.unlink(pkgPath);
     });
   });
 
-  it('found in parent directory', function() {
+  it('found in parent directory', () => {
     const pkgPath = pathFn.join(baseDir, '../package.json');
 
-    return fs.writeFile(pkgPath, '{"hexo": {}}').then(function() {
-      return findPkg(baseDir, {});
-    }).then(function(path) {
+    return fs.writeFile(pkgPath, '{"hexo": {}}').then(() => findPkg(baseDir, {})).then(path => {
       path.should.eql(pathFn.dirname(pkgPath));
       return fs.unlink(pkgPath);
     });
   });
 
-  it('found but don\'t have hexo data', function() {
+  it('found but don\'t have hexo data', () => {
     const pkgPath = pathFn.join(baseDir, 'package.json');
 
-    return fs.writeFile(pkgPath, '{"name": "hexo"}').then(function() {
-      return findPkg(baseDir, {});
-    }).then(function(path) {
+    return fs.writeFile(pkgPath, '{"name": "hexo"}').then(() => findPkg(baseDir, {})).then(path => {
       should.not.exist(path);
       return fs.unlink(pkgPath);
     });
   });
 
-  it('relative cwd', function() {
+  it('relative cwd', () => {
     const pkgPath = pathFn.join(baseDir, 'test', 'package.json');
 
-    return fs.writeFile(pkgPath, '{"hexo": {}}').then(function() {
-      return findPkg(baseDir, {cwd: 'test'});
-    }).then(function(path) {
+    return fs.writeFile(pkgPath, '{"hexo": {}}').then(() => findPkg(baseDir, {cwd: 'test'})).then(path => {
       path.should.eql(pathFn.dirname(pkgPath));
       return fs.unlink(pkgPath);
     });
   });
 
-  it('specify cwd but don\'t have hexo data', function() {
-    return findPkg(baseDir, {cwd: 'test'}).then(function(path) {
-      should.not.exist(path);
-    });
-  });
+  it('specify cwd but don\'t have hexo data', () => findPkg(baseDir, {cwd: 'test'}).then(path => {
+    should.not.exist(path);
+  }));
 
-  it('absolute cwd', function() {
+  it('absolute cwd', () => {
     const pkgPath = pathFn.join(baseDir, 'test', 'package.json');
     const cwd = pathFn.dirname(pkgPath);
 
-    return fs.writeFile(pkgPath, '{"hexo": {}}').then(function() {
-      return findPkg(baseDir, {cwd: cwd});
-    }).then(function(path) {
+    return fs.writeFile(pkgPath, '{"hexo": {}}').then(() => findPkg(baseDir, {cwd})).then(path => {
       path.should.eql(cwd);
       return fs.unlink(pkgPath);
     });