Fail no-ready rule when using arrow functions

Closes #35

Author: dgraham

rules/no-ready.js

@@ -8,7 +8,8 @@ function isDirect(node) {
     node.callee.type === 'Identifier' &&
     node.callee.name === '$' &&
     node.arguments[0] &&
-    node.arguments[0].type === 'FunctionExpression'
+    (node.arguments[0].type === 'FunctionExpression' ||
+      node.arguments[0].type === 'ArrowFunctionExpression')
   )
 }
 

tests/no-ready.js

@@ -5,10 +5,11 @@ const RuleTester = require('eslint').RuleTester
 
 const error = '$.ready is not allowed'
 
-const ruleTester = new RuleTester()
+const ruleTester = new RuleTester({parserOptions: {ecmaVersion: 6}})
 ruleTester.run('no-ready', rule, {
   valid: [
     'ready(function() { })',
+    'ready(() => { })',
     'ready()',
     '[].ready()',
     'div.ready()',
@@ -22,6 +23,10 @@ ruleTester.run('no-ready', rule, {
       code: '$(function() { })',
       errors: [{message: error, type: 'CallExpression'}]
     },
+    {
+      code: '$(() => { })',
+      errors: [{message: error, type: 'CallExpression'}]
+    },
     {
       code: '$(function init() { })',
       errors: [{message: error, type: 'CallExpression'}]
@@ -30,6 +35,10 @@ ruleTester.run('no-ready', rule, {
       code: '$(document).ready(function() { })',
       errors: [{message: error, type: 'CallExpression'}]
     },
+    {
+      code: '$(document).ready(() => { })',
+      errors: [{message: error, type: 'CallExpression'}]
+    },
     {
       code: '$().ready(function() { })',
       errors: [{message: error, type: 'CallExpression'}]