[eslint] Correct comments and add another test (facebook#17220)

dprgarner · Nov 9, 2019 · 54a180a · 54a180a
1 parent 37e3554
commit 54a180a
Showing 1 changed file with 14 additions and 4 deletions.
diff --git a/packages/eslint-plugin-react-hooks/__tests__/ESLintRulesOfHooks-test.js b/packages/eslint-plugin-react-hooks/__tests__/ESLintRulesOfHooks-test.js
@@ -303,17 +303,17 @@ const tests = {
       }
     `,
     `
-      // Similarly, this is valid because "use"-prefixed functions called in
-      // the callbacks of unrecognised functions are not assumed to be hooks.
+      // This is valid because "use"-prefixed functions called in
+      // unnamed function arguments are not assumed to be hooks.
       React.unknownFunction((foo, bar) => {
         if (foo) {
           useNotAHook(bar)
         }
       });
     `,
     `
-      // Similarly, this is valid because "use"-prefixed functions called in
-      // the callbacks of unrecognised functions are not assumed to be hooks.
+      // This is valid because "use"-prefixed functions called in
+      // unnamed function arguments are not assumed to be hooks.
       unknownFunction(function(foo, bar) {
         if (foo) {
           useNotAHook(bar)
@@ -818,6 +818,16 @@ const tests = {
       `,
       errors: [conditionalError('useCustomHook')],
     },
+    {
+      code: `
+        // This is invalid because "use"-prefixed functions used in named
+        // functions are assumed to be hooks.
+        React.unknownFunction(function notAComponent(foo, bar) {
+          useProbablyAHook(bar)
+        });
+      `,
+      errors: [functionError('useProbablyAHook', 'notAComponent')],
+    },
     {
       code: `
         // Invalid because it's dangerous.