Gate useEvent handling in ExhaustiveDeps as experimental

To keep the test simple, just imperatively only run the test if the EXPERIMENTAL flag is set.
facebook · Sep 23, 2022 · 9550edb · 9550edb
1 parent 16e669b
commit 9550edb
Showing 1 changed file with 15 additions and 12 deletions.
diff --git a/packages/eslint-plugin-react-hooks/__tests__/ESLintRuleExhaustiveDeps-test.js b/packages/eslint-plugin-react-hooks/__tests__/ESLintRuleExhaustiveDeps-test.js
@@ -1452,18 +1452,6 @@ const tests = {
         }
       `,
     },
-    {
-      code: normalizeIndent`
-        function MyComponent({ theme }) {
-          const onStuff = experimental_useEvent(() => {
-            showNotification(theme);
-          });
-          useEffect(() => {
-            onStuff();
-          }, []);
-        }
-      `,
-    },
   ],
   invalid: [
     {
@@ -7642,6 +7630,21 @@ const tests = {
   ],
 };
 
+if (__EXPERIMENTAL__) {
+  tests.valid.push({
+    code: normalizeIndent`
+      function MyComponent({ theme }) {
+        const onStuff = experimental_useEvent(() => {
+          showNotification(theme);
+        });
+        useEffect(() => {
+          onStuff();
+        }, []);
+      }
+    `,
+  });
+}
+
 // Tests that are only valid/invalid across parsers supporting Flow
 const testsFlow = {
   valid: [