fix(unbound-method): don't suppress errors from base rule

BREAKING CHANGE
jest-community · Aug 27, 2022 · 5f07860 · 5f07860
1 parent 5508c95
commit 5f07860
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 30 deletions.
diff --git a/src/rules/__tests__/unbound-method.test.ts b/src/rules/__tests__/unbound-method.test.ts
@@ -182,25 +182,6 @@ describe('error handling', () => {
       },
     );
   });
-
-  describe('when "project" is not set', () => {
-    const ruleTester = new ESLintUtils.RuleTester({
-      parser: '@typescript-eslint/parser',
-      parserOptions: {
-        sourceType: 'module',
-        tsconfigRootDir: rootPath,
-      },
-    });
-
-    ruleTester.run(
-      'unbound-method jest edition without "project" property',
-      requireRule(false),
-      {
-        valid: validTestCases.concat(invalidTestCases.map(({ code }) => code)),
-        invalid: [],
-      },
-    );
-  });
 });
 
 ruleTester.run('unbound-method jest edition', requireRule(false), {

diff --git a/src/rules/unbound-method.ts b/src/rules/unbound-method.ts
@@ -33,16 +33,6 @@ const baseRule = (() => {
   }
 })();
 
-const tryCreateBaseRule = (
-  context: Readonly<TSESLint.RuleContext<MessageIds, Options>>,
-) => {
-  try {
-    return baseRule?.create(context);
-  } catch {
-    return null;
-  }
-};
-
 interface Config {
   ignoreStatic: boolean;
 }
@@ -75,7 +65,7 @@ export default createRule<Options, MessageIds>({
     },
   },
   create(context) {
-    const baseSelectors = tryCreateBaseRule(context);
+    const baseSelectors = baseRule?.create(context);
 
     if (!baseSelectors) {
       return {};