100% test coverage in rule.ts

AhmedBaset · Sep 26, 2024 · e5b828e · e5b828e
1 parent 23419e6
commit e5b828e
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 7 deletions.
diff --git a/src/rules/no-phyisical-properties/rule.ts b/src/rules/no-phyisical-properties/rule.ts
@@ -107,7 +107,7 @@ function report({
   node: TSESTree.JSXAttribute;
   invalid: string;
   valid: string;
-  token: Token | null;
+  token: Token;
 }) {
   return ctx.report({
     node,
@@ -117,12 +117,10 @@ function report({
       valid,
     },
     loc: {
-      start: token?.loc?.start ?? node.loc!.start,
-      end: token?.loc?.end ?? node.loc!.end,
+      start: token?.loc?.start,
+      end: token?.loc?.end,
     },
     fix: (fixer) => {
-      if (!token) return null;
-
       return fixer.replaceText(token, token?.getRaw()?.replace(invalid, valid));
     },
   });

diff --git a/src/rules/no-phyisical-properties/test.ts b/src/rules/no-phyisical-properties/test.ts
@@ -57,7 +57,7 @@ vitest.describe(RULE_NAME, () => {
       },
       {
         name: "JSXNamespacedName",
-        code: `<svg:circle cx="50" cy="50" r="40" prop="pl-1 mr-2" />`,
+        code: `<svg:circle cx="50" cy="50" r="40" prop:class="pl-1 mr-2" />`,
       },
     ],
     invalid: [

diff --git a/src/utils/ast.ts b/src/utils/ast.ts
@@ -5,7 +5,7 @@ import {
   NO_PHYSICAL_CLASSESS,
   type Context,
   type MessageId,
-} from "../rules/no-phyisical-properties/rule";
+} from "../rules/no-phyisical-properties/rule.js";
 
 const unimplemented = new Set<string>();