[Tests] jsx-no-useless-fragment, jsx-fragments, function-component-definition: use output: null to assert no autofix

mdjermanovic · ljharb · commit fbb1e0f082cc · 2024-05-20T07:40:22.000+02:00
diff --git a/tests/lib/rules/function-component-definition.js b/tests/lib/rules/function-component-definition.js
@@ -807,11 +807,7 @@ ruleTester.run('function-component-definition', rule, {
           return <div/>;
         }
       `,
-      output: `
-        var Hello: React.FC<Test> = function(props) {
-          return <div/>;
-        }
-      `,
+      output: null,
       options: [{ namedComponents: 'function-declaration' }],
       errors: [{ messageId: 'function-declaration' }],
       features: ['types'],
@@ -822,11 +818,7 @@ ruleTester.run('function-component-definition', rule, {
           return <div/>;
         };
       `,
-      output: `
-        var Hello: React.FC<Test> = (props) => {
-          return <div/>;
-        };
-      `,
+      output: null,
       options: [{ namedComponents: 'function-declaration' }],
       errors: [{ messageId: 'function-declaration' }],
       features: ['types'],
@@ -852,11 +844,7 @@ ruleTester.run('function-component-definition', rule, {
           return <div/>;
         }
       `,
-      output: `
-        function Hello<Test>(props: Test) {
-          return <div/>;
-        }
-      `,
+      output: null,
       options: [{ namedComponents: 'arrow-function' }],
       errors: [{ messageId: 'arrow-function' }],
       features: ['types'],
@@ -963,13 +951,7 @@ ruleTester.run('function-component-definition', rule, {
           }
         }
       `,
-      output: `
-        function wrap(Component) {
-          return function<Test>(props) {
-            return <div><Component {...props}/></div>
-          }
-        }
-      `,
+      output: null,
       errors: [{ messageId: 'arrow-function' }],
       options: [{ unnamedComponents: 'arrow-function' }],
       features: ['types'],
diff --git a/tests/lib/rules/jsx-fragments.js b/tests/lib/rules/jsx-fragments.js
@@ -142,7 +142,7 @@ ruleTester.run('jsx-fragments', rule, {
     },
     {
       code: '<><Foo /></>',
-      output: '<><Foo /></>', // should get '<Act.Frag><Foo /></Act.Frag>', but the old TS parser lacks opening/closing Fragment info
+      output: null, // should get '<Act.Frag><Foo /></Act.Frag>', but the old TS parser lacks opening/closing Fragment info
       features: ['fragment', 'no-babel', 'ts', 'no-ts-new'],
       options: ['element'],
       settings,
diff --git a/tests/lib/rules/jsx-no-useless-fragment.js b/tests/lib/rules/jsx-no-useless-fragment.js
@@ -194,7 +194,7 @@ ruleTester.run('jsx-no-useless-fragment', rule, {
     },
     {
       code: '<div><>{"a"}{"b"}</></div>',
-      output: '<div><>{"a"}{"b"}</></div>',
+      output: null,
       errors: [{ messageId: 'ChildOfHtmlElement', type: 'JSXFragment' }],
       features: ['fragment', 'ts-old', 'no-ts-new', 'no-babel', 'no-default'],
     },

Original file line number	Diff line number	Diff line change
`@@ -142,7 +142,7 @@ ruleTester.run('jsx-fragments', rule, {`
`142`	`142`	`},`
`143`	`143`	`{`
`144`	`144`	`code: '<><Foo /></>',`
`145`		`- output: '<><Foo /></>', // should get '<Act.Frag><Foo /></Act.Frag>', but the old TS parser lacks opening/closing Fragment info`
	`145`	`+ output: null, // should get '<Act.Frag><Foo /></Act.Frag>', but the old TS parser lacks opening/closing Fragment info`
`146`	`146`	`features: ['fragment', 'no-babel', 'ts', 'no-ts-new'],`
`147`	`147`	`options: ['element'],`
`148`	`148`	`settings,`
Original file line number	Diff line number	Diff line change
`@@ -194,7 +194,7 @@ ruleTester.run('jsx-no-useless-fragment', rule, {`
`194`	`194`	`},`
`195`	`195`	`{`
`196`	`196`	`code: '<div><>{"a"}{"b"}</></div>',`
`197`		`- output: '<div><>{"a"}{"b"}</></div>',`
	`197`	`+ output: null,`
`198`	`198`	`errors: [{ messageId: 'ChildOfHtmlElement', type: 'JSXFragment' }],`
`199`	`199`	`features: ['fragment', 'ts-old', 'no-ts-new', 'no-babel', 'no-default'],`
`200`	`200`	`},`