Fix RulesOfHooks test case indentation

poteto · poteto · commit 49ae0fad8459 · 2022-09-30T15:16:22.000-04:00
Just a small formatting fix. ghstack-source-id: c71ff02 Pull Request resolved: #25368
diff --git a/packages/eslint-plugin-react-hooks/__tests__/ESLintRulesOfHooks-test.js b/packages/eslint-plugin-react-hooks/__tests__/ESLintRulesOfHooks-test.js
@@ -978,142 +978,142 @@ if (__EXPERIMENTAL__) {
   tests.valid = [
     ...tests.valid,
     `
-    // Valid because functions created with useEvent can be called in a useEffect.
-    function MyComponent({ theme }) {
-      const onClick = useEvent(() => {
-        showNotification(theme);
-      });
-      useEffect(() => {
-        onClick();
-      });
-    }
-  `,
-    `
-    // Valid because functions created with useEvent can be called in closures.
-    function MyComponent({ theme }) {
-      const onClick = useEvent(() => {
-        showNotification(theme);
-      });
-      return <Child onClick={() => onClick()}></Child>;
-    }
-  `,
-    `
-    // Valid because functions created with useEvent can be called in closures.
-    function MyComponent({ theme }) {
-      const onClick = useEvent(() => {
-        showNotification(theme);
-      });
-      const onClick2 = () => { onClick() };
-      const onClick3 = useCallback(() => onClick(), []);
-      return <>
-        <Child onClick={onClick2}></Child>
-        <Child onClick={onClick3}></Child>
-      </>;
-    }
-  `,
-    `
-    // Valid because functions created with useEvent can be passed by reference in useEffect
-    // and useEvent.
-    function MyComponent({ theme }) {
-      const onClick = useEvent(() => {
-        showNotification(theme);
-      });
-      const onClick2 = useEvent(() => {
-        debounce(onClick);
-      });
-      useEffect(() => {
-        let id = setInterval(onClick, 100);
-        return () => clearInterval(onClick);
-      }, []);
-      return <Child onClick={() => onClick2()} />
-    }
-  `,
-    `
-    const MyComponent = ({theme}) => {
-      const onClick = useEvent(() => {
-        showNotification(theme);
-      });
-      return <Child onClick={() => onClick()}></Child>;
-    };
-  `,
-    `
-    function MyComponent({ theme }) {
-      const notificationService = useNotifications();
-      const showNotification = useEvent((text) => {
-        notificationService.notify(theme, text);
-      });
-      const onClick = useEvent((text) => {
-        showNotification(text);
-      });
-      return <Child onClick={(text) => onClick(text)} />
-    }
-  `,
+      // Valid because functions created with useEvent can be called in a useEffect.
+      function MyComponent({ theme }) {
+        const onClick = useEvent(() => {
+          showNotification(theme);
+        });
+        useEffect(() => {
+          onClick();
+        });
+      }
+    `,
     `
-    function MyComponent({ theme }) {
-      useEffect(() => {
-        onClick();
-      });
-      const onClick = useEvent(() => {
-        showNotification(theme);
-      });
-    }
-  `,
-  ];
-  tests.invalid = [
-    ...tests.invalid,
-    {
-      code: `
+      // Valid because functions created with useEvent can be called in closures.
       function MyComponent({ theme }) {
         const onClick = useEvent(() => {
           showNotification(theme);
         });
-        return <Child onClick={onClick}></Child>;
+        return <Child onClick={() => onClick()}></Child>;
       }
     `,
-      errors: [useEventError('onClick')],
-    },
-    {
-      code: `
-      // This should error even though it shares an identifier name with the below
-      function MyComponent({theme}) {
+    `
+      // Valid because functions created with useEvent can be called in closures.
+      function MyComponent({ theme }) {
         const onClick = useEvent(() => {
-          showNotification(theme)
+          showNotification(theme);
         });
-        return <Child onClick={onClick} />
+        const onClick2 = () => { onClick() };
+        const onClick3 = useCallback(() => onClick(), []);
+        return <>
+          <Child onClick={onClick2}></Child>
+          <Child onClick={onClick3}></Child>
+        </>;
       }
-
-      // The useEvent function shares an identifier name with the above
-      function MyOtherComponent({theme}) {
+    `,
+    `
+      // Valid because functions created with useEvent can be passed by reference in useEffect
+      // and useEvent.
+      function MyComponent({ theme }) {
         const onClick = useEvent(() => {
-          showNotification(theme)
+          showNotification(theme);
         });
-        return <Child onClick={() => onClick()} />
+        const onClick2 = useEvent(() => {
+          debounce(onClick);
+        });
+        useEffect(() => {
+          let id = setInterval(onClick, 100);
+          return () => clearInterval(onClick);
+        }, []);
+        return <Child onClick={() => onClick2()} />
       }
     `,
-      errors: [{...useEventError('onClick'), line: 7}],
-    },
-    {
-      code: `
-      const MyComponent = ({ theme }) => {
+    `
+      const MyComponent = ({theme}) => {
         const onClick = useEvent(() => {
           showNotification(theme);
         });
-        return <Child onClick={onClick}></Child>;
+        return <Child onClick={() => onClick()}></Child>;
+      };
+    `,
+    `
+      function MyComponent({ theme }) {
+        const notificationService = useNotifications();
+        const showNotification = useEvent((text) => {
+          notificationService.notify(theme, text);
+        });
+        const onClick = useEvent((text) => {
+          showNotification(text);
+        });
+        return <Child onClick={(text) => onClick(text)} />
       }
     `,
-      errors: [useEventError('onClick')],
-    },
-    {
-      code: `
-      // Invalid because onClick is being aliased to foo but not invoked
+    `
       function MyComponent({ theme }) {
+        useEffect(() => {
+          onClick();
+        });
         const onClick = useEvent(() => {
           showNotification(theme);
         });
-        let foo = onClick;
-        return <Bar onClick={foo} />
       }
     `,
+  ];
+  tests.invalid = [
+    ...tests.invalid,
+    {
+      code: `
+        function MyComponent({ theme }) {
+          const onClick = useEvent(() => {
+            showNotification(theme);
+          });
+          return <Child onClick={onClick}></Child>;
+        }
+      `,
+      errors: [useEventError('onClick')],
+    },
+    {
+      code: `
+        // This should error even though it shares an identifier name with the below
+        function MyComponent({theme}) {
+          const onClick = useEvent(() => {
+            showNotification(theme)
+          });
+          return <Child onClick={onClick} />
+        }
+
+        // The useEvent function shares an identifier name with the above
+        function MyOtherComponent({theme}) {
+          const onClick = useEvent(() => {
+            showNotification(theme)
+          });
+          return <Child onClick={() => onClick()} />
+        }
+      `,
+      errors: [{...useEventError('onClick'), line: 7}],
+    },
+    {
+      code: `
+        const MyComponent = ({ theme }) => {
+          const onClick = useEvent(() => {
+            showNotification(theme);
+          });
+          return <Child onClick={onClick}></Child>;
+        }
+      `,
+      errors: [useEventError('onClick')],
+    },
+    {
+      code: `
+        // Invalid because onClick is being aliased to foo but not invoked
+        function MyComponent({ theme }) {
+          const onClick = useEvent(() => {
+            showNotification(theme);
+          });
+          let foo = onClick;
+          return <Bar onClick={foo} />
+        }
+      `,
       errors: [{...useEventError('onClick'), line: 7}],
     },
     {
