test: add more test cases

Author: Rel1cx

packages/plugins/eslint-plugin-react-dom/src/rules/no-render.spec.ts

@@ -41,6 +41,44 @@ ruleTester.run(RULE_NAME, rule, {
         createRoot(document.getElementById("app")).render(<Component />);
       `,
     },
+    {
+      code: tsx`
+        import React from "react";
+        import ReactDom from "react-dom";
+        import Component from "Component";
+
+        ReactDom.render(<Component />, document.getElementById("app")!);
+      `,
+      errors: [{ messageId: "noRender" }],
+      output: tsx`
+        import { createRoot } from "react-dom/client";
+        import React from "react";
+        import ReactDom from "react-dom";
+        import Component from "Component";
+
+        createRoot(document.getElementById("app")!).render(<Component />);
+      `,
+    },
+    {
+      code: tsx`
+        import React from "react";
+        import ReactDom from "react-dom";
+        import Component from "Component";
+
+        const rootEl = document.getElementById("app")!;
+        ReactDom.render(<Component />, rootEl);
+      `,
+      errors: [{ messageId: "noRender" }],
+      output: tsx`
+        import { createRoot } from "react-dom/client";
+        import React from "react";
+        import ReactDom from "react-dom";
+        import Component from "Component";
+
+        const rootEl = document.getElementById("app")!;
+        createRoot(rootEl).render(<Component />);
+      `,
+    },
   ],
   valid: [
     tsx`

packages/plugins/eslint-plugin-react-dom/src/rules/no-render.ts

@@ -93,15 +93,9 @@ function getFix(context: RuleContext, node: TSESTree.CallExpression) {
   return (fixer: RuleFixer) => {
     const [arg0, arg1] = node.arguments;
     if (arg0 == null || arg1 == null) return null;
-    const fixedCallExpressionText = [
-      "createRoot",
-      "(" + getText(arg1) + ")" + ".",
-      "render",
-      "(" + getText(arg0) + ")",
-    ].join("");
     return [
       fixer.insertTextBefore(context.sourceCode.ast, 'import { createRoot } from "react-dom/client";\n'),
-      fixer.replaceText(node, fixedCallExpressionText),
+      fixer.replaceText(node, `createRoot(${getText(arg1)}).render(${getText(arg0)})`),
     ];
   };
 }