refactor(plugins/x): revert 'no-useless-fragment' and update its test cases

Author: Rel1cx

packages/plugins/eslint-plugin-react-x/src/rules/no-useless-fragment.spec.ts

@@ -81,7 +81,7 @@ ruleTester.run(RULE_NAME, rule, {
       errors: [{ type: AST_NODE_TYPES.JSXElement, messageId: "noUselessFragmentInBuiltIn" }],
     },
     {
-      // whitepace tricky case
+      // whitespace tricky case
       code: /* tsx */ `
         <section>
           git<>
@@ -161,6 +161,16 @@ ruleTester.run(RULE_NAME, rule, {
     "<Fooo content={<>eeee ee eeeeeee eeeeeeee</>} />",
     "<>{foos.map(foo => foo)}</>",
     "<>{moo}</>",
+    /* tsx */ `
+      function Foo() {
+      	return <>&nbsp;</>;
+      }
+    `,
+    /* tsx */ `
+      function Foo() {
+      	return <>a&nbsp;b</>;
+      }
+    `,
     /* tsx */ `
       <>
         {moo}
@@ -184,16 +194,6 @@ ruleTester.run(RULE_NAME, rule, {
         {moo}
       </>
     `,
-    /* tsx */ `
-      function Foo() {
-      	return <Bar><>&nbsp;</></Bar>;
-      }
-    `,
-    /* tsx */ `
-      function Foo() {
-      	return <Bar><>a&nbsp;b</></Bar>;
-      }
-    `,
     /* tsx */ `<>{cloneElement(children, { ref: childrenRef })}</>`,
     {
       code: /* tsx */ `

packages/plugins/eslint-plugin-react-x/src/rules/no-useless-fragment.ts

@@ -24,7 +24,6 @@ function check(
   if (JSX.isKeyedElement(node, initialScope)) return;
   if (JSX.isBuiltInElement(node.parent)) context.report({ messageId: "noUselessFragmentInBuiltIn", node });
   if (node.children.length === 0) return context.report({ messageId: "noUselessFragment", node });
-  if (node.children.some((child) => JSX.isLiteral(child) && child.value !== child.raw)) return;
   const isChildren = AST.isOneOf([AST_NODE_TYPES.JSXElement, AST_NODE_TYPES.JSXFragment])(node.parent);
   const [firstChildren] = node.children;
   // <Foo content={<>ee eeee eeee ...</>} />