eslint-plugin-stratified-design 0.12.0 → 0.12.1

package/lib/rules/no-disallowed-imports.js

@@ -136,8 +136,10 @@ module.exports = {
             return [theMember, theImportSpec];
           }
 
+          const isModulePathMatched = match(fromCwd(context.cwd, modulePath))(importSpec.import.from)
+
           if (importSpec.import.member === ANY_MEMBER) {
-            return ['Any member', importSpec]
+            return isModulePathMatched ? ['Any member', importSpec] : [undefined, undefined]
           }
 
           const importedSpecifiers = node.specifiers.map((specifier) => {
@@ -152,10 +154,7 @@ module.exports = {
             importedSpecifiers.some((sp) => sp === specifier)
           );
 
-          if (
-            member &&
-            match(fromCwd(context.cwd, modulePath))(importSpec.import.from)
-          ) {
+          if (member && isModulePathMatched) {
             if (member === DEFAULT || member === NAMESPACE) {
               const theMember = node.specifiers.find(
                 ({ type }) =>

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-stratified-design",
-  "version": "0.12.0",
+  "version": "0.12.1",
   "description": "ESlint rules for stratified design",
   "keywords": [
     "eslint",

package/tests/lib/rules/no-disallowed-imports.js

@@ -131,6 +131,20 @@ ruleTester.run("no-disallowed-imports", rule, {
         },
       ],
     },
+    {
+      code: "import { foo } from 'nodeModule'",
+      filename: "./src/fileB.js",
+      options: [
+        {
+          imports: [
+            {
+              import: { member: ["*"], from: "src/fileA" },
+              disallow: ["src/**/*.js"],
+            },
+          ],
+        },
+      ],
+    },
     {
       code: "import { anyMember } from './fileA'",
       filename: "./src/fileB.js",
@@ -159,6 +173,48 @@ ruleTester.run("no-disallowed-imports", rule, {
         },
       ],
     },
+    {
+      code: "import { anyMember } from './fileA.act'",
+      filename: "./src/fileB.cal.js",
+      options: [
+        {
+          imports: [
+            {
+              import: { member: "*", from: "**/*.act" },
+              allow: ["src/**/*.cal.js"],
+            },
+          ],
+        },
+      ],
+    },
+    {
+      code: "import { anyMember } from 'fileA.cal'",
+      filename: "./src/fileB.act.js",
+      options: [
+        {
+          imports: [
+            {
+              import: { member: "*", from: "**/*.act" },
+              disallow: ["src/**/*.cal.js"],
+            },
+          ],
+        },
+      ],
+    },
+    {
+      code: "import { anyMember } from 'nodeModule'",
+      filename: "./src/fileB.cal.js",
+      options: [
+        {
+          imports: [
+            {
+              import: { member: "*", from: "**/*.act" },
+              disallow: ["src/**/*.cal.js"],
+            },
+          ],
+        },
+      ],
+    },
   ],
   invalid: [
     {
@@ -240,20 +296,20 @@ ruleTester.run("no-disallowed-imports", rule, {
       ],
     },
     {
-      code: "import { anyMember } from './fileA'",
-      filename: "./src/fileB.js",
+      code: "import { anyMember } from './fileA.act'",
+      filename: "./src/fileB.cal.js",
       options: [
         {
           imports: [
             {
-              import: { member: "*", from: "src/fileA" },
-              disallow: ["src/**/*.js"],
+              import: { member: "*", from: "**/*.act" },
+              disallow: ["**/*.cal.js"],
             },
           ],
         },
       ],
       errors: [
-        { messageId: "no-disallowed-imports", data: { member: "Any member", from: "src/fileA" } },
+        { messageId: "no-disallowed-imports", data: { member: "Any member", from: "**/*.act" } },
       ],
     },
   ],