re-transformed rules to new format

import-js · Sep 24, 2016 · 1f7a9ca · 1f7a9ca
1 parent a560d31
commit 1f7a9ca
Show file tree

Hide file tree

Showing 2 changed files with 100 additions and 88 deletions.
diff --git a/src/rules/extensions.js b/src/rules/extensions.js
@@ -6,93 +6,99 @@ import assign from 'object-assign'
 import resolve from 'eslint-module-utils/resolve'
 import { isBuiltIn } from '../core/importType'
 
-module.exports = function (context) {
-  const configuration = context.options[0] || 'never'
-  const defaultConfig = typeof configuration === 'string' ? configuration : null
-  const modifiers = assign(
-    {},
-    typeof configuration === 'object' ? configuration : context.options[1]
-  )
+const enumValues = { enum: [ 'always', 'never' ] }
+const patternProperties = {
+  type: 'object',
+  patternProperties: { '.*': enumValues },
+}
 
-  function isUseOfExtensionRequired(extension) {
-    if (!has(modifiers, extension)) { modifiers[extension] = defaultConfig }
-    return modifiers[extension] === 'always'
-  }
+module.exports = {
+  meta: {
+    docs: {},
 
-  function isUseOfExtensionForbidden(extension) {
-    if (!has(modifiers, extension)) { modifiers[extension] = defaultConfig }
-    return modifiers[extension] === 'never'
-  }
+    schema: {
+      anyOf: [
+        {
+          type: 'array',
+          items: [enumValues],
+          additionalItems: false,
+        },
+        {
+          type: 'array',
+          items: [patternProperties],
+          additionalItems: false,
+        },
+        {
+          type: 'array',
+          items: [
+            enumValues,
+            patternProperties,
+          ],
+          additionalItems: false,
+        },
+      ],
+    },
+  },
 
-  function isResolvableWithoutExtension(file) {
-    const extension = path.extname(file)
-    const fileWithoutExtension = file.slice(0, -extension.length)
-    const resolvedFileWithoutExtension = resolve(fileWithoutExtension, context)
+  create: function (context) {
+    const configuration = context.options[0] || 'never'
+    const defaultConfig = typeof configuration === 'string' ? configuration : null
+    const modifiers = assign(
+      {},
+      typeof configuration === 'object' ? configuration : context.options[1]
+    )
 
-    return resolvedFileWithoutExtension === resolve(file, context)
-  }
+    function isUseOfExtensionRequired(extension) {
+      if (!has(modifiers, extension)) { modifiers[extension] = defaultConfig }
+      return modifiers[extension] === 'always'
+    }
 
-  function checkFileExtension(node) {
-    const { source } = node
-    const importPath = source.value
+    function isUseOfExtensionForbidden(extension) {
+      if (!has(modifiers, extension)) { modifiers[extension] = defaultConfig }
+      return modifiers[extension] === 'never'
+    }
 
-    // don't enforce anything on builtins
-    if (isBuiltIn(importPath, context.settings)) return
+    function isResolvableWithoutExtension(file) {
+      const extension = path.extname(file)
+      const fileWithoutExtension = file.slice(0, -extension.length)
+      const resolvedFileWithoutExtension = resolve(fileWithoutExtension, context)
 
-    const resolvedPath = resolve(importPath, context)
+      return resolvedFileWithoutExtension === resolve(file, context)
+    }
 
-    // get extension from resolved path, if possible.
-    // for unresolved, use source value.
-    const extension = path.extname(resolvedPath || importPath).substring(1)
+    function checkFileExtension(node) {
+      const { source } = node
+      const importPath = source.value
 
-    if (!extension || !endsWith(importPath, extension)) {
-      if (isUseOfExtensionRequired(extension) && !isUseOfExtensionForbidden(extension)) {
-        context.report({
-          node: source,
-          message:
-            `Missing file extension ${extension ? `"${extension}" ` : ''}for "${importPath}"`,
-        })
-      }
-    } else if (extension) {
-      if (isUseOfExtensionForbidden(extension) && isResolvableWithoutExtension(importPath)) {
-        context.report({
-          node: source,
-          message: `Unexpected use of file extension "${extension}" for "${importPath}"`,
-        })
-      }
-    }
-  }
+      // don't enforce anything on builtins
+      if (isBuiltIn(importPath, context.settings)) return
 
-  return {
-    ImportDeclaration: checkFileExtension,
-  }
-}
+      const resolvedPath = resolve(importPath, context)
 
-const enumValues = { enum: [ 'always', 'never' ] }
-const patternProperties = {
-  type: 'object',
-  patternProperties: { '.*': enumValues },
-}
+      // get extension from resolved path, if possible.
+      // for unresolved, use source value.
+      const extension = path.extname(resolvedPath || importPath).substring(1)
 
-module.exports.schema = {
-  anyOf: [
-    {
-      type: 'array',
-      items: [enumValues],
-      additionalItems: false,
-    },
-    {
-      type: 'array',
-      items: [patternProperties],
-      additionalItems: false,
-    },
-    {
-      type: 'array',
-      items: [
-        enumValues,
-        patternProperties,
-      ],
-      additionalItems: false,
-    },
-  ],
+      if (!extension || !endsWith(importPath, extension)) {
+        if (isUseOfExtensionRequired(extension) && !isUseOfExtensionForbidden(extension)) {
+          context.report({
+            node: source,
+            message:
+              `Missing file extension ${extension ? `"${extension}" ` : ''}for "${importPath}"`,
+          })
+        }
+      } else if (extension) {
+        if (isUseOfExtensionForbidden(extension) && isResolvableWithoutExtension(importPath)) {
+          context.report({
+            node: source,
+            message: `Unexpected use of file extension "${extension}" for "${importPath}"`,
+          })
+        }
+      }
+    }
+
+    return {
+      ImportDeclaration: checkFileExtension,
+    }
+  },
 }
diff --git a/src/rules/no-dynamic-require.js b/src/rules/no-dynamic-require.js
@@ -11,15 +11,21 @@ function isStaticValue(arg) {
     (arg.type === 'TemplateLiteral' && arg.expressions.length === 0)
 }
 
-module.exports = function (context) {
-  return {
-    CallExpression(node) {
-      if (isRequire(node) && !isStaticValue(node.arguments[0])) {
-        context.report({
-          node,
-          message: 'Calls to require() should use string literals',
-        })
-      }
-    },
-  }
+module.exports = {
+  meta: {
+    docs: {},
+  },
+
+  create: function (context) {
+    return {
+      CallExpression(node) {
+        if (isRequire(node) && !isStaticValue(node.arguments[0])) {
+          context.report({
+            node,
+            message: 'Calls to require() should use string literals',
+          })
+        }
+      },
+    }
+  },
 }