Use new "Checker#execute" method

Which is introduced in 2.4.0

Fixes #96
Fixes #95
Closes gh-97

Author: markelog

.jscsrc

@@ -1,3 +1,3 @@
 {
-	"preset": "jquery"
+    "preset": "jquery"
 }

README.md

@@ -47,17 +47,12 @@ jscs: {
         config: ".jscsrc",
         esnext: true, // If you use ES6 http://jscs.info/overview.html#esnext
         verbose: true, // If you need output with rule names http://jscs.info/overview.html#verbose
+        fix: true, // Autofix code style violations when possible.
         requireCurlyBraces: [ "if" ]
     }
 }
 ```
 
-#### fix
-Type: `Boolean`
-Default value: `false`
-
-Auto-fix code style violations when possible.
-
 #### force
 Type: `Boolean`
 Default value: `false`

package.json

@@ -14,7 +14,7 @@
   },
   "dependencies": {
     "hooker": "~0.2.3",
-    "jscs": "~2.1.0",
+    "jscs": "~2.4.0",
     "lodash": "~3.10.0",
     "vow": "~0.4.1"
   },

tasks/jscs.js

@@ -13,15 +13,13 @@ module.exports = function( grunt ) {
 
                 // `null` is a default value, but its equivalent to `true`,
                 // With this way it's easy to distinguish specified value
-                config: null,
-
-                fix: false
+                config: null
             } ),
 
             jscs = new JSCS( options ),
-            checks = this.filesSrc.map( options.fix ? jscs.fix : jscs.check, jscs );
+            runs = this.filesSrc.map( jscs.checker.execute, jscs.checker );
 
-        Vow.allResolved( checks ).spread( function() {
+        Vow.allResolved( runs ).spread( function() {
             var results = [];
 
             // Filter unsuccessful promises

tasks/lib/jscs.js

@@ -9,7 +9,7 @@ var Checker = require( "jscs" ),
 exports.init = function( grunt ) {
 
     // Task specific options
-    var taskOptions = [ "config", "force", "reporter", "reporterOutput", "fix" ];
+    var taskOptions = [ "config", "force", "reporter", "reporterOutput" ];
 
     /**
      * @see jQuery.isEmptyObject
@@ -58,29 +58,16 @@ exports.init = function( grunt ) {
     }
 
     /**
-     * @see Checker#checkPath
+     * @see Checker#execute
      */
-    JSCS.prototype.check = function( path ) {
-        var checkPath = this.checker.checkPath( path );
+    JSCS.prototype.execute = function( path ) {
+        var run = this.checker.execute( path );
 
-        checkPath.fail( function( error ) {
+        run.fail( function( error ) {
             grunt.warn( error );
         } );
 
-        return checkPath;
-    };
-
-    /**
-     * @see Checker#fixPath
-     */
-    JSCS.prototype.fix = function( path ) {
-        var fixPath = this.checker.fixPath( path );
-
-        fixPath.fail( function( error ) {
-            grunt.warn( error );
-        } );
-
-        return fixPath;
+        return run;
     };
 
     /**

test/methods.js

@@ -26,7 +26,7 @@ module.exports = {
             config: "test/configs/fail.json"
         } );
 
-        fixture.check( "test/fixtures/fixture.js" ).then( function( collection ) {
+        fixture.execute( "test/fixtures/fixture.js" ).then( function( collection ) {
             fixture.setErrors( errors = collection );
             done();
         } );
@@ -230,7 +230,7 @@ module.exports = {
             "requireCurlyBraces": [ "while" ]
         } );
 
-        jscs.check( "test/fixtures/fixture.js" ).then( function( errorsCollection ) {
+        jscs.execute( "test/fixtures/fixture.js" ).then( function( errorsCollection ) {
 
             // "grunt-contrib-nodeunit" package through which these tests are run,
             // Mutes grunt log actions so it wouldn't interfeare with tests output,
@@ -269,7 +269,7 @@ module.exports = {
             "excludeFiles": [ "test/fixtures/exclude.js" ]
         } );
 
-        jscs.check( "test/fixtures/exclude.js" ).then( function( errors ) {
+        jscs.execute( "test/fixtures/exclude.js" ).then( function( errors ) {
             test.equal( jscs.setErrors( errors ).count(), 0,
                 "should not find any errors in excluded file" );
             test.done();
@@ -283,7 +283,7 @@ module.exports = {
             config: "empty"
         } );
 
-        jscs.check( "test/fixtures/fixture.js" ).then( function( errorsCollection ) {
+        jscs.execute( "test/fixtures/fixture.js" ).then( function( errorsCollection ) {
             errorsCollection.forEach( function( errors ) {
                 errors.getErrorList().forEach( function( error ) {
                     test.equal( error.message, "test", "should add additional rule" );
@@ -300,7 +300,7 @@ module.exports = {
             reporterOutput: "test.xml"
         } );
 
-        jscs.check( "test/fixtures/fixture.js" ).then( function( errorsCollection ) {
+        jscs.execute( "test/fixtures/fixture.js" ).then( function( errorsCollection ) {
             jscs.setErrors( errorsCollection ).report();
 
             test.ok( grunt.file.exists( "test.xml" ), "test.xml should exist" );
@@ -317,7 +317,7 @@ module.exports = {
             "requireCurlyBraces": [ "while" ]
         } );
 
-        jscs.check( "test/fixtures/broken.js" ).then( function( errorsCollection ) {
+        jscs.execute( "test/fixtures/broken.js" ).then( function( errorsCollection ) {
             errorsCollection.forEach( function( errors ) {
                 test.equal(
                     errors.getErrorList()[ 0 ].message,
@@ -328,20 +328,5 @@ module.exports = {
                 test.done();
             } );
         } ).fail( test.done );
-    },
-
-    "#fix send it's path to the jscs checker": function( test ) {
-        var jscs = new JSCS( {} );
-
-        hooker.hook( jscs.checker, "fixPath", function( path ) {
-            test.equal( path, "path" );
-            test.done();
-
-            return hooker.override( {
-                fail: function() {}
-            } );
-        } );
-
-        jscs.fix( "path" );
     }
 };