fix: don't include deprecated rules in all config

Resolves #663
jest-community · Sep 8, 2020 · 87e29fc · 87e29fc
1 parent d16b84c
commit 87e29fc
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 5 deletions.
diff --git a/src/__tests__/rules.test.ts b/src/__tests__/rules.test.ts
@@ -2,8 +2,11 @@ import { existsSync } from 'fs';
 import { resolve } from 'path';
 import plugin from '../';
 
-const ruleNames = Object.keys(plugin.rules);
 const numberOfRules = 44;
+const ruleNames = Object.keys(plugin.rules);
+const deprecatedRules = Object.entries(plugin.rules)
+  .filter(([, rule]) => rule.meta.deprecated)
+  .map(([name]) => name);
 
 describe('rules', () => {
   it('should have a corresponding doc for each rule', () => {
@@ -54,7 +57,7 @@ describe('rules', () => {
       'style',
     ]);
     expect(Object.keys(recommendedConfigs.all.rules)).toHaveLength(
-      ruleNames.length,
+      ruleNames.length - deprecatedRules.length,
     );
     const allConfigRules = Object.values(recommendedConfigs)
       .map(config => Object.keys(config.rules))

diff --git a/src/index.ts b/src/index.ts
@@ -48,9 +48,15 @@ const recommendedRules = Object.entries(rules)
     {},
   );
 
-const allRules = Object.keys(rules).reduce<
-  Record<string, TSESLint.Linter.RuleLevel>
->((rules, key) => ({ ...rules, [`jest/${key}`]: 'error' }), {});
+const allRules = Object.entries(rules)
+  // .filter(([, rule]) => !rule.meta.deprecated)
+  .reduce(
+    (acc, [name]) => ({
+      ...acc,
+      [`jest/${name}`]: 'error',
+    }),
+    {},
+  );
 
 const createConfig = (rules: Record<string, TSESLint.Linter.RuleLevel>) => ({
   plugins: ['jest'],