fix: use v8 compatible test

gajus · gajus · commit 92d96f4c9bf5 · 2021-10-25T15:40:14.000-05:00
diff --git a/test/eslint-config-canonical/canonical.js b/test/eslint-config-canonical/canonical.js
@@ -1,7 +1,9 @@
+/* eslint-disable import/no-dynamic-require */
+
 const test = require('ava');
-const CLIEngine = require('eslint').CLIEngine;
+const {builtinRules} = require('eslint/use-at-your-own-risk');
 
-const configurations = [
+const configurationNames = [
   'ava',
   'eslintrc',
   'flowtype',
@@ -12,29 +14,32 @@ const configurations = [
   'react',
 ];
 
-for (const configuration of configurations) {
-  // eslint-disable-next-line import/no-dynamic-require -- Safe as for testing
-  const config = require('../../' + configuration);
+const getPluginRuleNames = (pluginName) => {
+  const dependencyName = pluginName === '@babel' ? '@babel/eslint-plugin' : 'eslint-plugin-' + pluginName;
 
-  const rulesDefinition = new CLIEngine({
-    configFile: './' + configuration + '.js',
-    useEslintrc: false,
-  })
-    .getRules();
+  return Object.keys(require(dependencyName).rules).map((ruleName) => {
+    return pluginName + '/' + ruleName;
+  });
+};
 
-  const ruleNames = Object.keys(config.rules || []);
+const main = async () => {
+  for (const configurationName of configurationNames) {
+    const configuration = require('../../' + configurationName);
 
-  for (const ruleName of ruleNames) {
-    test('plugin "' + configuration + '" has "' + ruleName + '" rule', (tst) => {
-      const pluginHasRule = rulesDefinition.has(ruleName);
+    const supportedRuleNames = [
+      ...builtinRules.keys(),
+    ];
 
-      tst.true(pluginHasRule);
-    });
-  }
+    for (const pluginName of configuration.plugins) {
+      supportedRuleNames.push(...getPluginRuleNames(pluginName));
+    }
 
-  test('plugin "' + configuration + '" does not have "does-not-exist" rule', (tst) => {
-    const pluginHasRule = rulesDefinition.has('does-not-exist');
+    for (const configurationRuleName of Object.keys(configuration.rules)) {
+      test('"' + configurationName + '" configuration has "' + configurationRuleName + '" rule', (t) => {
+        t.true(supportedRuleNames.includes(configurationRuleName));
+      });
+    }
+  }
+};
 
-    tst.true(pluginHasRule === false);
-  });
-}
+main();
