Merge pull request #11 from htmlacademy/feature/check-unused-params

Unused function args validation

Author: zeckson

es5.js

@@ -84,7 +84,7 @@ module.exports = {
     // Variables
     // http://eslint.org/docs/rules/#variables
     // ---------------------------------------
-    'no-unused-vars': ['error', {args: 'none'}], // eslint:recommended
+    'no-unused-vars': ['error', {args: 'all', argsIgnorePattern: '^_'}], // check that all args are used¬
     'no-delete-var': 'error', // eslint:recommended
     'no-label-var': 'error',
     'no-shadow': 'error',

test/es5/space-in-parens.js

@@ -1,10 +1,11 @@
 'use strict';
 
 var fun = function (variable) {
-
+  return variable;
 };
 
-fun(function () {});
+fun(function () {
+});
 
 fun(4, (4 + 5), 7);
 

test/es5/unused-params.js

@@ -0,0 +1,25 @@
+'use strict';
+
+var bar = function (param2) {
+  return param2;
+};
+
+var foo = function () {
+  bar();
+};
+
+var myObject = {
+  method1: function () {
+    return '12';
+  },
+  methodWithIgnoredParam: function (_ignore) {
+    return 42;
+  }
+};
+
+bar(function () {
+  foo(function () {
+    myObject.method1();
+    myObject.methodWithIgnoredParam(122);
+  });
+});