chore: reorder rule test cases

Author: Rel1cx

plugins/eslint-plugin-react-debug/src/rules/function-component/function-component.spec.ts

@@ -27,17 +27,19 @@ ruleTester.run(RULE_NAME, rule, {
     },
     {
       code: tsx`
-        // 🔴 Avoid: A Hook that doesn't use Hooks
-        function useSorted(items) {
-          return items.slice().sort();
+        function useMultipleHooks() {
+          const [state] = useState(0);
+          useEffect(() => {});
+          useCallback(() => {}, []);
+          return state;
         }
       `,
       errors: [
         {
           data: {
             json: stringify({
-              name: "useSorted",
-              hookCalls: 0,
+              name: "useMultipleHooks",
+              hookCalls: 3,
             }),
           },
           messageId: "default",
@@ -46,26 +48,12 @@ ruleTester.run(RULE_NAME, rule, {
     },
     {
       code: tsx`
-        function useToggle() {
-            const [value, setValue] = useState(false);
-            return [value, () => setValue(x => !x)];
-        }
-
         // 🔴 Avoid: A Hook that doesn't use Hooks
         function useSorted(items) {
           return items.slice().sort();
         }
       `,
       errors: [
-        {
-          data: {
-            json: stringify({
-              name: "useToggle",
-              hookCalls: 1,
-            }),
-          },
-          messageId: "default",
-        },
         {
           data: {
             json: stringify({
@@ -79,7 +67,7 @@ ruleTester.run(RULE_NAME, rule, {
     },
     {
       code: tsx`
-        const useClassnames = (obj) => {
+        function useClassnames(obj) {
             // Invalid, because useClassnames doesn't use any other React Hooks.
             var k, cls='';
             for (k in obj) {
@@ -105,7 +93,7 @@ ruleTester.run(RULE_NAME, rule, {
     },
     {
       code: tsx`
-        function useClassnames(obj) {
+        const useClassnames = (obj) => {
             // Invalid, because useClassnames doesn't use any other React Hooks.
             var k, cls='';
             for (k in obj) {
@@ -131,20 +119,55 @@ ruleTester.run(RULE_NAME, rule, {
     },
     {
       code: tsx`
-        export function useNestedHook() {
-            const [state, setState] = useState("state");
-            const useInnerHook = () => {
-                return "inner hook";
-            };
+        const useExpressionBody = () => useState(0);
+      `,
+      errors: [
+        {
+          data: {
+            json: stringify({
+              name: "useExpressionBody",
+              hookCalls: 1,
+            }),
+          },
+          messageId: "default",
+        },
+      ],
+    },
+    {
+      code: tsx`
+        const useExpressionHook = function() {
+          return useState(0);
+        };
+      `,
+      errors: [
+        {
+          data: {
+            json: stringify({
+              name: "useExpressionHook",
+              hookCalls: 1,
+            }),
+          },
+          messageId: "default",
+        },
+      ],
+    },
+    {
+      code: tsx`
+        function useToggle() {
+            const [value, setValue] = useState(false);
+            return [value, () => setValue(x => !x)];
+        }
 
-            return [state, setState, useInnerHook] as const;
+        // 🔴 Avoid: A Hook that doesn't use Hooks
+        function useSorted(items) {
+          return items.slice().sort();
         }
       `,
       errors: [
         {
           data: {
             json: stringify({
-              name: "useNestedHook",
+              name: "useToggle",
               hookCalls: 1,
             }),
           },
@@ -153,7 +176,7 @@ ruleTester.run(RULE_NAME, rule, {
         {
           data: {
             json: stringify({
-              name: "useInnerHook",
+              name: "useSorted",
               hookCalls: 0,
             }),
           },
@@ -164,9 +187,9 @@ ruleTester.run(RULE_NAME, rule, {
     {
       code: tsx`
         export function useNestedHook() {
+            const [state, setState] = useState("state");
             const useInnerHook = () => {
-                const [state, setState] = useState("state");
-                return state;
+                return "inner hook";
             };
 
             return [state, setState, useInnerHook] as const;
@@ -177,7 +200,7 @@ ruleTester.run(RULE_NAME, rule, {
           data: {
             json: stringify({
               name: "useNestedHook",
-              hookCalls: 0,
+              hookCalls: 1,
             }),
           },
           messageId: "default",
@@ -186,7 +209,7 @@ ruleTester.run(RULE_NAME, rule, {
           data: {
             json: stringify({
               name: "useInnerHook",
-              hookCalls: 1,
+              hookCalls: 0,
             }),
           },
           messageId: "default",
@@ -196,7 +219,7 @@ ruleTester.run(RULE_NAME, rule, {
     {
       code: tsx`
         export function useNestedHook() {
-            const fn = () => {
+            const useInnerHook = () => {
                 const [state, setState] = useState("state");
                 return state;
             };
@@ -214,17 +237,10 @@ ruleTester.run(RULE_NAME, rule, {
           },
           messageId: "default",
         },
-      ],
-    },
-    {
-      code: tsx`
-        const useExpressionBody = () => useState(0);
-      `,
-      errors: [
         {
           data: {
             json: stringify({
-              name: "useExpressionBody",
+              name: "useInnerHook",
               hookCalls: 1,
             }),
           },
@@ -234,19 +250,21 @@ ruleTester.run(RULE_NAME, rule, {
     },
     {
       code: tsx`
-        function useMultipleHooks() {
-          const [state] = useState(0);
-          useEffect(() => {});
-          useCallback(() => {}, []);
-          return state;
+        export function useNestedHook() {
+            const fn = () => {
+                const [state, setState] = useState("state");
+                return state;
+            };
+
+            return [state, setState, useInnerHook] as const;
         }
       `,
       errors: [
         {
           data: {
             json: stringify({
-              name: "useMultipleHooks",
-              hookCalls: 3,
+              name: "useNestedHook",
+              hookCalls: 0,
             }),
           },
           messageId: "default",
@@ -271,24 +289,6 @@ ruleTester.run(RULE_NAME, rule, {
         },
       ],
     },
-    {
-      code: tsx`
-        const useExpressionHook = function() {
-          return useState(0);
-        };
-      `,
-      errors: [
-        {
-          data: {
-            json: stringify({
-              name: "useExpressionHook",
-              hookCalls: 1,
-            }),
-          },
-          messageId: "default",
-        },
-      ],
-    },
     {
       code: tsx`
         function useMotionStyle() {