Merge branch 'no-is-array'

Author: dgraham

README.md

@@ -50,6 +50,7 @@ Add `jquery` to the plugins section of your `.eslintrc` configuration file. You
     "jquery/no-hide": 2,
     "jquery/no-html": 2,
     "jquery/no-in-array": 2,
+    "jquery/no-is-array": 2,
     "jquery/no-is": 2,
     "jquery/no-load": 2,
     "jquery/no-map": 2,

index.js

@@ -24,6 +24,7 @@ module.exports = {
     'no-hide': require('./rules/no-hide'),
     'no-html': require('./rules/no-html'),
     'no-in-array': require('./rules/no-in-array'),
+    'no-is-array': require('./rules/no-is-array'),
     'no-is': require('./rules/no-is'),
     'no-load': require('./rules/no-load'),
     'no-map': require('./rules/no-map'),
@@ -74,6 +75,7 @@ module.exports = {
         'jquery/no-hide': 2,
         'jquery/no-html': 2,
         'jquery/no-in-array': 2,
+        'jquery/no-is-array': 2,
         'jquery/no-is': 2,
         'jquery/no-load': 2,
         'jquery/no-map': 2,

rules/no-is-array.js

@@ -0,0 +1,18 @@
+'use strict'
+
+module.exports = function(context) {
+  return {
+    CallExpression: function(node) {
+      if (node.callee.type !== 'MemberExpression') return
+      if (node.callee.object.name !== '$') return
+      if (node.callee.property.name !== 'isArray') return
+
+      context.report({
+        node: node,
+        message: 'Prefer Array#isArray to $.isArray'
+      })
+    }
+  }
+}
+
+module.exports.schema = []

tests/no-is-array.js

@@ -0,0 +1,17 @@
+'use strict'
+
+const rule = require('../rules/no-is-array')
+const RuleTester = require('eslint').RuleTester
+
+const error = 'Prefer Array#isArray to $.isArray'
+
+const ruleTester = new RuleTester()
+ruleTester.run('no-in-array', rule, {
+  valid: ['isArray()', '"test".isArray()', '"test".isArray'],
+  invalid: [
+    {
+      code: '$.isArray()',
+      errors: [{message: error, type: 'CallExpression'}]
+    }
+  ]
+})