Chore: refactor utils.getRuleInfo

Author: not-an-aardvark

lib/utils.js

@@ -20,7 +20,7 @@ module.exports = {
   from the file, the return value will be `null`.
   */
   getRuleInfo (ast) {
-    const INTERESTING_KEYS = ['create', 'meta'];
+    const INTERESTING_KEYS = new Set(['create', 'meta']);
     let exportsVarOverridden = false;
     let exportsIsFunction = false;
 
@@ -46,13 +46,8 @@ module.exports = {
 
             exportsIsFunction = false;
             return node.right.properties.reduce((parsedProps, prop) => {
-              const keyValue = prop.key.type === 'Literal'
-                ? prop.key.value
-                : prop.key.type === 'Identifier'
-                  ? prop.key.name
-                  : null;
-
-              if (INTERESTING_KEYS.indexOf(keyValue) !== -1) {
+              const keyValue = module.exports.getKeyName(prop);
+              if (INTERESTING_KEYS.has(keyValue)) {
                 parsedProps[keyValue] = prop.value;
               }
               return parsedProps;
@@ -64,15 +59,15 @@ module.exports = {
           node.left.object.type === 'MemberExpression' &&
           node.left.object.object.type === 'Identifier' && node.left.object.object.name === 'module' &&
           node.left.object.property.type === 'Identifier' && node.left.object.property.name === 'exports' &&
-          node.left.property.type === 'Identifier' && INTERESTING_KEYS.indexOf(node.left.property.name) !== -1
+          node.left.property.type === 'Identifier' && INTERESTING_KEYS.has(node.left.property.name)
         ) {
           // Check `module.exports.create = () => {}`
 
           currentExports[node.left.property.name] = node.right;
         } else if (
           !exportsVarOverridden &&
           node.left.object.type === 'Identifier' && node.left.object.name === 'exports' &&
-          node.left.property.type === 'Identifier' && INTERESTING_KEYS.indexOf(node.left.property.name) !== -1
+          node.left.property.type === 'Identifier' && INTERESTING_KEYS.has(node.left.property.name)
         ) {
           // Check `exports.create = () => {}`