From 75f8adc48aad20fda69be669585b21dc0e210788 Mon Sep 17 00:00:00 2001
From: Jordan Harband <ljharb@gmail.com>
Date: Fri, 1 Jul 2022 20:52:53 -0700
Subject: [PATCH] [Tests] skip tests that fail in eslint 8

---
 tests/src/rules/no-cycle.js          |  3 ++-
 tests/src/rules/no-unused-modules.js | 20 ++++++++++----------
 2 files changed, 12 insertions(+), 11 deletions(-)

diff --git a/tests/src/rules/no-cycle.js b/tests/src/rules/no-cycle.js
index ee635930d4..0d20db6877 100644
--- a/tests/src/rules/no-cycle.js
+++ b/tests/src/rules/no-cycle.js
@@ -239,7 +239,8 @@ ruleTester.run('no-cycle', rule, {
         parser: parsers.BABEL_ESLINT,
       }),
     ]).concat(
-      testVersion('> 3', () => ({ // Dynamic import is not properly caracterized with eslint < 4
+      // TODO: enable for eslint 8+, just not for the old babel parser
+      testVersion('> 3 && < 8', () => ({ // Dynamic import is not properly characterized with eslint < 4
         code: `import { foo } from "./${testDialect}/depth-one-dynamic"; // #2265 6`,
         errors: [error(`Dependency cycle detected.`)],
         parser: parsers.BABEL_ESLINT,
diff --git a/tests/src/rules/no-unused-modules.js b/tests/src/rules/no-unused-modules.js
index 7181ab5e48..847bb32fe1 100644
--- a/tests/src/rules/no-unused-modules.js
+++ b/tests/src/rules/no-unused-modules.js
@@ -253,8 +253,8 @@ describe('dynamic imports', () => {
 
   // test for unused exports with `import()`
   ruleTester.run('no-unused-modules', rule, {
-    valid: [
-      test({
+    valid: [].concat(
+      semver.satisfies(eslintPkg.version, '< 8') ? test({
         options: unusedExportsOptions,
         code: `
             export const a = 10
@@ -262,20 +262,20 @@ describe('dynamic imports', () => {
             export const c = 30
             const d = 40
             export default d
-            `,
+          `,
         parser: parsers.BABEL_ESLINT,
         filename: testFilePath('./no-unused-modules/exports-for-dynamic-js.js'),
-      }),
-    ],
+      }) : [],
+    ),
     invalid: [
       test({
         options: unusedExportsOptions,
         code: `
-        export const a = 10
-        export const b = 20
-        export const c = 30
-        const d = 40
-        export default d
+          export const a = 10
+          export const b = 20
+          export const c = 30
+          const d = 40
+          export default d
         `,
         parser: parsers.BABEL_ESLINT,
         filename: testFilePath('./no-unused-modules/exports-for-dynamic-js-2.js'),