Disallow removeData usage

Author: dgraham

rules/no-data.js

@@ -6,13 +6,16 @@ module.exports = function(context) {
   return {
     CallExpression: function(node) {
       if (node.callee.type !== 'MemberExpression') return
-      if (node.callee.property.name !== 'data') return
+      if (!utils.isjQuery(node)) return
 
-      if (utils.isjQuery(node)) {
-        context.report({
-          node: node,
-          message: 'Prefer WeakMap to $.data'
-        })
+      const name = node.callee.property.name
+      switch (name) {
+        case 'data':
+        case 'removeData':
+          context.report({
+            node: node,
+            message: 'Prefer WeakMap to $.' + name
+          })
       }
     }
   }

tests/no-data.js

@@ -4,14 +4,20 @@ const rule = require('../rules/no-data')
 const RuleTester = require('eslint').RuleTester
 
 const error = 'Prefer WeakMap to $.data'
+const removeError = 'Prefer WeakMap to $.removeData'
 
 const ruleTester = new RuleTester()
 ruleTester.run('no-data', rule, {
   valid: [
     'data()',
     '[].data()',
     'div.data()',
-    'div.data'
+    'div.data',
+
+    'removeData()',
+    '[].removeData()',
+    'div.removeData()',
+    'div.removeData'
   ],
   invalid: [
     {
@@ -29,6 +35,14 @@ ruleTester.run('no-data', rule, {
     {
       code: '$("div").append($("input").data())',
       errors: [{message: error, type: 'CallExpression'}]
+    },
+    {
+      code: '$("div").removeData()',
+      errors: [{message: removeError, type: 'CallExpression'}]
+    },
+    {
+      code: '$div.removeData()',
+      errors: [{message: removeError, type: 'CallExpression'}]
     }
   ]
 })