Add coffeelint step in testing. Fix codestyle

cybertk · Mar 14, 2016 · 75cbd39 · 75cbd39
1 parent 029a579
commit 75cbd39
Show file tree

Hide file tree

Showing 6 changed files with 24 additions and 4 deletions.
diff --git a/Gruntfile.coffee b/Gruntfile.coffee
@@ -21,6 +21,17 @@ module.exports = (grunt) ->
         options:
           nospawn: true
 
+    coffeelint:
+      default:
+        src: [
+          'Gruntfile.coffee'
+          'src/**/*.coffee'
+          'test/**/*.coffee'
+        ]
+      options: {
+        configFile: 'coffeelint.json'
+      }
+
     coffeecov:
       compile:
         src: 'lib'
@@ -53,6 +64,7 @@ module.exports = (grunt) ->
   ]
 
   grunt.registerTask "test", [
+    "coffeelint"
     "coffeecov"
     "mochaTest"
     "uploadCoverage"

diff --git a/coffeelint.json b/coffeelint.json
@@ -0,0 +1,7 @@
+{
+  "max_line_length": {
+    "value": 120,
+    "level": "error",
+    "limitComments": true
+  }
+}
diff --git a/package.json b/package.json
@@ -33,6 +33,7 @@
     "grunt": "~0.4.5",
     "grunt-cli": "~0.1.13",
     "grunt-coffeecov": "^0.1.5",
+    "grunt-coffeelint": "^0.0.15",
     "grunt-contrib-watch": "~1.0.0",
     "grunt-mocha-test": "~0.12.7",
     "grunt-shell": "^1.1.2",

diff --git a/test/unit/abao-test.coffee b/test/unit/abao-test.coffee
@@ -21,7 +21,7 @@ Abao = proxyquire '../../', {
 }
 
 should = chai.should()
-chai.use(sinonChai);
+chai.use(sinonChai)
 
 
 describe 'Abao', ->

diff --git a/test/unit/test-runner-test.coffee b/test/unit/test-runner-test.coffee
@@ -21,7 +21,7 @@ SERVER = 'http://localhost:3000'
 
 assert = chai.assert
 should = chai.should()
-chai.use(sinonChai);
+chai.use(sinonChai)
 
 runner = null
 
@@ -170,7 +170,7 @@ describe 'Test Runner', ->
 
         runner = new TestRunner options
         sinon.stub test, 'run', (callback) ->
-           callback()
+          callback()
 
         # Mute stdout/stderr
         mute (unmute) ->

diff --git a/test/unit/test-test.coffee b/test/unit/test-test.coffee
@@ -6,7 +6,7 @@ proxyquire = require('proxyquire').noCallThru()
 
 assert = chai.assert
 should = chai.should()
-chai.use(sinonChai);
+chai.use(sinonChai)
 
 requestStub = sinon.stub()
 requestStub.restore = () ->