Make test passes

Author: igorkamyshev

rules/enforce-store-naming-convention/enforce-store-naming-convention.js

@@ -14,6 +14,11 @@ const {
 const { createLinkToRule } = require("../../utils/create-link-to-rule");
 const { nodeTypeIs } = require("../../utils/node-type-is");
 const { traverseParentByType } = require("../../utils/traverse-parent-by-type");
+const {
+  traverseRealDeclaration,
+} = require("../../utils/traverse-real-declaration");
+const { nodeName } = require("../../utils/node-name");
+const { nodeRange } = require("../../utils/node-range");
 
 module.exports = {
   meta: {
@@ -47,23 +52,27 @@ module.exports = {
     if (parserServices?.program) {
       return {
         VariableDeclarator(node) {
-          const isEffectorStore = nodeTypeIs.store({
-            node,
-            context,
-          });
-
-          if (!isEffectorStore) {
-            return;
-          }
-
-          const storeName = node.id.name;
+          const realNodes = traverseRealDeclaration(node);
 
-          if (namingOf.store.isInvalid({ name: storeName, context })) {
-            reportStoreNameConventionViolation({
+          for (const realNode of realNodes) {
+            const isEffectorStore = nodeTypeIs.store({
+              node: realNode,
               context,
-              node,
-              storeName,
             });
+
+            if (!isEffectorStore) {
+              continue;
+            }
+
+            const storeName = nodeName(realNode);
+
+            if (namingOf.store.isInvalid({ name: storeName, context })) {
+              reportStoreNameConventionViolation({
+                context,
+                node: realNode,
+                storeName,
+              });
+            }
           }
         },
       };
@@ -197,7 +206,7 @@ function reportStoreNameConventionViolation({ context, node, storeName }) {
         messageId: "renameStore",
         data: { storeName, correctedStoreName },
         fix(fixer) {
-          return fixer.replaceTextRange(node.id.range, correctedStoreName);
+          return fixer.replaceTextRange(nodeRange(node), correctedStoreName);
         },
       },
     ],

rules/enforce-store-naming-convention/prefix/enforce-store-naming-convention-prefix.ts.test.js

@@ -25,7 +25,7 @@ ruleTester.run("enforce-store-naming-convention-prefix.ts.test", rule, {
 
   invalid: [
     // Errors
-    ...["incorrect-store-naming.ts", "incorrect-store-after-destruction.ts"]
+    ...["incorrect-store-naming.ts"]
       .map(readExampleForTheRule)
       .map((result) => ({
         ...result,
@@ -42,5 +42,22 @@ ruleTester.run("enforce-store-naming-convention-prefix.ts.test", rule, {
           },
         ],
       })),
+    ...["incorrect-store-after-destruction.ts"]
+      .map(readExampleForTheRule)
+      .map((result) => ({
+        ...result,
+        errors: [
+          {
+            messageId: "invalidName",
+            type: "Identifier",
+            suggestions: [
+              {
+                messageId: "renameStore",
+                output: result.code.replace("justStore", "$justStore"),
+              },
+            ],
+          },
+        ],
+      })),
   ],
 });

utils/node-name.js

@@ -0,0 +1,5 @@
+function nodeName(node) {
+  return node.id?.name ?? node.name;
+}
+
+module.exports = { nodeName };

utils/node-range.js

@@ -0,0 +1,5 @@
+function nodeRange(node) {
+  return node.id?.range ?? node.range;
+}
+
+module.exports = { nodeRange };

utils/traverse-real-declaration.js

@@ -0,0 +1,9 @@
+function traverseRealDeclaration(node) {
+  if (node?.id?.type === "ArrayPattern") {
+    return node.id.elements;
+  }
+
+  return [node];
+}
+
+module.exports = { traverseRealDeclaration };