Merge eab54ca into 52b4f48

lib/__tests__/index.js

@@ -4,6 +4,7 @@ const rule = require("../").rules;
 const RuleTester = require("eslint").RuleTester;
 
 RuleTester.setDefaultConfig({
+  parser: "babel-eslint",
   parserOptions: {
     ecmaVersion: 6,
     sourceType: "module"
@@ -13,6 +14,30 @@ RuleTester.setDefaultConfig({
 var ruleTester = new RuleTester();
 
 describe("ESLint plugin", () => {
+  ruleTester.run("no-dynamic-import-moment", rule["no-dynamic-import-moment"], {
+    valid: ["import('date-fns').then()"],
+    invalid: [
+      {
+        code: "import('moment').then(()=>{})",
+        errors: [
+          {
+            message: "Use date-fns or Native Date methods instead of moment.js",
+            type: "CallExpression"
+          }
+        ]
+      },
+      {
+        code: "(async () => { const m = await import('moment'); })()",
+        errors: [
+          {
+            message: "Use date-fns or Native Date methods instead of moment.js",
+            type: "CallExpression"
+          }
+        ]
+      }
+    ]
+  });
+
   ruleTester.run("no-import-moment", rule["no-import-moment"], {
     valid: [
       "import { format, formatDistance, formatRelative, subDays } from 'date-fns'"

lib/index.js

@@ -1,5 +1,6 @@
 "use strict";
 const rules = {
+  "no-dynamic-import-moment": require("./rules/no-dynamic-import-moment"),
   "no-import-moment": require("./rules/no-import-moment"),
   "no-moment-constructor": require("./rules/no-moment-constructor"),
   ...require("./rules/no-moment-methods")

lib/rules/no-dynamic-import-moment.js

@@ -0,0 +1,17 @@
+var message = "Use date-fns or Native Date methods instead of moment.js";
+
+module.exports = function(context) {
+  return {
+    CallExpression: function(node) {
+      if (node.callee.type !== "Import") {
+        return;
+      }
+
+      var arg = node.arguments[0];
+
+      if (arg.type === "Literal" && arg.value === "moment") {
+        context.report(node, message);
+      }
+    }
+  };
+};