fix eslint error, add test case

Author: 刘祺

.eslintrc

@@ -25,10 +25,7 @@
         "SwitchCase": 1
       }
     ],
-    "comma-style": [
-      2,
-      "first"
-    ],
+    "comma-style": 2,
     "max-len": [
       2,
       {

index.js

@@ -25,10 +25,10 @@ module.exports = withConfigLoader(function (loadConfig) {
       : {}
 
     var options = {
-      from: file.path
-    , to: file.path
+      from: file.path,
+      to: file.path,
       // Generate a separate source map for gulp-sourcemaps
-    , map: file.sourceMap ? { annotation: false } : false
+      map: file.sourceMap ? { annotation: false } : false
     }
 
     loadConfig(file)
@@ -104,8 +104,8 @@ function withConfigLoader(cb) {
     if (Array.isArray(plugins)) {
       return cb(function () {
         return Promise.resolve({
-          plugins: plugins
-        , options: options
+          plugins: plugins,
+          options: options
         })
       })
     } else if (typeof plugins === 'function') {
@@ -127,8 +127,9 @@ function withConfigLoader(cb) {
           configPath = file.dirname
         }
         return postcssLoadConfig(
-          { file: file
-          , options: contextOptions
+          {
+            file: file,
+            options: contextOptions
           },
           configPath
         )

package.json

@@ -4,7 +4,7 @@
     "lines": 100,
     "statements": 100,
     "functions": 100,
-    "branches": 97,
+    "branches": 100,
     "reporter": [
       "lcov",
       "text"
@@ -18,7 +18,7 @@
   "main": "index.js",
   "scripts": {
     "coveralls": "coveralls < coverage/lcov.info",
-    "pretest": "eslint index.js",
+    "pretest": "eslint *.js",
     "test": "nyc mocha test.js"
   },
   "repository": {

test.js

@@ -1,4 +1,5 @@
-/* global it, afterEach, beforeEach, describe, Promise */
+/* eslint-env node, mocha */
+/* eslint max-len: ["off"] */
 
 var assert = require('assert')
 var gutil = require('gulp-util')
@@ -45,6 +46,28 @@ it('should transform css with multiple processors', function (cb) {
 
 })
 
+it('should not transform css with out any processor', function (cb) {
+
+  var css = 'a { color: black }'
+
+  var stream = postcss(function(){
+    return {}
+  })
+
+  stream.on('data', function (file) {
+    var result = file.contents.toString('utf8')
+    var target = css
+    assert.equal( result, target )
+    cb()
+  })
+
+  stream.write(new gutil.File({
+    contents: new Buffer(css)
+  }))
+
+  stream.end()
+
+})
 
 it('should correctly wrap postcss errors', function (cb) {
 
@@ -89,7 +112,7 @@ it('should respond with error on stream files', function (cb) {
     isStream: function () { return true },
     isNull: function() { return false },
     path: path.resolve('testpath')
-  };
+  }
 
   stream.write(streamFile)
 
@@ -163,27 +186,27 @@ describe('PostCSS Guidelines', function () {
       return this.source
     }
     this.toString = function(){
-      var code = this.showSourceCode();
+      var code = this.showSourceCode()
       if ( code ) {
-          code = '\n\n' + code + '\n';
+        code = '\n\n' + code + '\n'
       }
-      return this.name + ': ' + this.message + code;
+      return this.name + ': ' + this.message + code
     }
   }
   var postcssStub = {
-    use: function () {}
-  , process: function () {}
+    use: function () {},
+    process: function () {}
   }
   var postcssLoadConfigStub
   var postcss = proxyquire('./index', {
     postcss: function (plugins) {
       postcssStub.use(plugins)
       return postcssStub
-    }
-  , 'postcss-load-config': function (ctx, configPath) {
+    },
+    'postcss-load-config': function (ctx, configPath) {
       return postcssLoadConfigStub(ctx, configPath)
-    }
-  , 'vinyl-sourcemaps-apply': function () {
+    },
+    'vinyl-sourcemaps-apply': function () {
       return {}
     }
   })
@@ -203,8 +226,8 @@ describe('PostCSS Guidelines', function () {
     var stream = postcss([ doubler ])
     var cssPath = __dirname + '/src/fixture.css'
     postcssStub.process.returns(Promise.resolve({
-      css: ''
-    , warnings: function () {
+      css: '',
+      warnings: function () {
         return []
       }
     }))
@@ -216,8 +239,8 @@ describe('PostCSS Guidelines', function () {
     })
 
     stream.write(new gutil.File({
-      contents: new Buffer('a {}')
-    , path: cssPath
+      contents: new Buffer('a {}'),
+      path: cssPath
     }))
 
     stream.end()
@@ -228,8 +251,8 @@ describe('PostCSS Guidelines', function () {
 
     var stream = postcss([ doubler ], {to: 'overriden'})
     postcssStub.process.returns(Promise.resolve({
-      css: ''
-    , warnings: function () {
+      css: '',
+      warnings: function () {
         return []
       }
     }))
@@ -251,19 +274,19 @@ describe('PostCSS Guidelines', function () {
 
     var cssPath = __dirname + '/fixture.css'
     var file = new gutil.File({
-      contents: new Buffer('a {}')
-    , path: cssPath
+      contents: new Buffer('a {}'),
+      path: cssPath
     })
     var plugins = [ doubler ]
     var callback = sandbox.stub().returns({
-      plugins: plugins
-    , options: { to: 'overriden' }
+      plugins: plugins,
+      options: { to: 'overriden' }
     })
     var stream = postcss(callback)
 
     postcssStub.process.returns(Promise.resolve({
-      css: ''
-    , warnings: function () {
+      css: '',
+      warnings: function () {
         return []
       }
     }))
@@ -283,28 +306,28 @@ describe('PostCSS Guidelines', function () {
 
     var cssPath = __dirname + '/fixture.css'
     var file = new gutil.File({
-      contents: new Buffer('a {}')
-    , path: cssPath
+      contents: new Buffer('a {}'),
+      path: cssPath
     })
     var stream = postcss({ to: 'initial' })
     var plugins = [ doubler ]
 
     postcssLoadConfigStub.returns(Promise.resolve({
-      plugins: plugins
-    , options: { to: 'overriden' }
+      plugins: plugins,
+      options: { to: 'overriden' }
     }))
 
     postcssStub.process.returns(Promise.resolve({
-      css: ''
-    , warnings: function () {
+      css: '',
+      warnings: function () {
         return []
       }
     }))
 
     stream.on('data', function () {
       assert.deepEqual(postcssLoadConfigStub.getCall(0).args[0], {
-        file: file
-      , options: { to: 'initial' }
+        file: file,
+        options: { to: 'initial' }
       })
       assert.equal(postcssStub.use.getCall(0).args[0], plugins)
       assert.equal(postcssStub.process.getCall(0).args[1].to, 'overriden')
@@ -320,8 +343,8 @@ describe('PostCSS Guidelines', function () {
     var stream = postcss()
     postcssLoadConfigStub.returns(Promise.resolve({ plugins: [] }))
     postcssStub.process.returns(Promise.resolve({
-      css: ''
-    , warnings: function () {
+      css: '',
+      warnings: function () {
         return []
       }
     }))
@@ -330,8 +353,8 @@ describe('PostCSS Guidelines', function () {
       cb()
     })
     stream.end(new gutil.File({
-      contents: new Buffer('a {}')
-    , path: cssPath
+      contents: new Buffer('a {}'),
+      path: cssPath
     }))
   })
 
@@ -340,8 +363,8 @@ describe('PostCSS Guidelines', function () {
     var stream = postcss({ config: '/absolute/path' })
     postcssLoadConfigStub.returns(Promise.resolve({ plugins: [] }))
     postcssStub.process.returns(Promise.resolve({
-      css: ''
-    , warnings: function () {
+      css: '',
+      warnings: function () {
         return []
       }
     }))
@@ -350,8 +373,8 @@ describe('PostCSS Guidelines', function () {
       cb()
     })
     stream.end(new gutil.File({
-      contents: new Buffer('a {}')
-    , path: cssPath
+      contents: new Buffer('a {}'),
+      path: cssPath
     }))
   })
 
@@ -360,8 +383,8 @@ describe('PostCSS Guidelines', function () {
     var stream = postcss({ config: './relative/path' })
     postcssLoadConfigStub.returns(Promise.resolve({ plugins: [] }))
     postcssStub.process.returns(Promise.resolve({
-      css: ''
-    , warnings: function () {
+      css: '',
+      warnings: function () {
         return []
       }
     }))
@@ -370,21 +393,21 @@ describe('PostCSS Guidelines', function () {
       cb()
     })
     stream.end(new gutil.File({
-      contents: new Buffer('a {}')
-    , path: cssPath
-    , base: __dirname
+      contents: new Buffer('a {}'),
+      path: cssPath,
+      base: __dirname
     }))
   })
 
   it('should not override `from` and `map` if using gulp-sourcemaps', function (cb) {
     var stream = postcss([ doubler ], { from: 'overriden', map: 'overriden' })
     var cssPath = __dirname + '/fixture.css'
     postcssStub.process.returns(Promise.resolve({
-      css: ''
-    , warnings: function () {
+      css: '',
+      warnings: function () {
         return []
-      }
-    , map: {
+      },
+      map: {
         toJSON: function () {
           return {
             sources: [],
@@ -407,8 +430,8 @@ describe('PostCSS Guidelines', function () {
     })
 
     var file = new gutil.File({
-      contents: new Buffer('a {}')
-    , path: cssPath
+      contents: new Buffer('a {}'),
+      path: cssPath
     })
     file.sourceMap = {}
     stream.end(file)
@@ -448,8 +471,8 @@ describe('PostCSS Guidelines', function () {
 
     sandbox.stub(gutil, 'log')
     postcssStub.process.returns(Promise.resolve({
-      css: ''
-    , warnings: function () {
+      css: '',
+      warnings: function () {
         return [new Warning('msg1'), new Warning('msg2')]
       }
     }))
@@ -460,8 +483,8 @@ describe('PostCSS Guidelines', function () {
     })
 
     stream.write(new gutil.File({
-      contents: new Buffer('a {}')
-    , path: cssPath
+      contents: new Buffer('a {}'),
+      path: cssPath
     }))
 
     stream.end()
@@ -478,8 +501,8 @@ describe('PostCSS Guidelines', function () {
     var stream = postcss([ doubler ], options)
     var cssPath = __dirname + '/src/fixture.css'
     postcssStub.process.returns(Promise.resolve({
-      css: ''
-    , warnings: function () {
+      css: '',
+      warnings: function () {
         return []
       }
     }))
@@ -495,8 +518,8 @@ describe('PostCSS Guidelines', function () {
     })
 
     stream.write(new gutil.File({
-      contents: new Buffer('a {}')
-    , path: cssPath
+      contents: new Buffer('a {}'),
+      path: cssPath
     }))
 
     stream.end()