From ce765796cd1916dcb1c81cb1a66db37745c1215c Mon Sep 17 00:00:00 2001 From: Gareth Jones Date: Sun, 11 Apr 2021 10:25:17 +1200 Subject: [PATCH] docs(no-focused-tests): remove references to `ftest` method (#816) There's no such prefixed method for `test`; only `fit` & `fdescribe` --- docs/rules/no-focused-tests.md | 6 +----- src/rules/__tests__/no-focused-tests.test.ts | 16 ---------------- 2 files changed, 1 insertion(+), 21 deletions(-) diff --git a/docs/rules/no-focused-tests.md b/docs/rules/no-focused-tests.md index a728fb1f0..22f00bd99 100644 --- a/docs/rules/no-focused-tests.md +++ b/docs/rules/no-focused-tests.md @@ -12,7 +12,7 @@ whenever you are using the exclusivity feature. ## Rule Details This rule looks for every `describe.only`, `it.only`, `test.only`, `fdescribe`, -`fit` and `ftest` occurrences within the source code. Of course there are some +and `fit` occurrences within the source code. Of course there are some edge-cases which can’t be detected by this rule e.g.: ```js @@ -31,13 +31,9 @@ test.only('foo', () => {}); test['only']('bar', () => {}); fdescribe('foo', () => {}); fit('foo', () => {}); -ftest('bar', () => {}); fit.each` table `(); -ftest.each` - table -`(); ``` These patterns would not be considered warnings: diff --git a/src/rules/__tests__/no-focused-tests.test.ts b/src/rules/__tests__/no-focused-tests.test.ts index 0bc9ae7a3..c6229e770 100644 --- a/src/rules/__tests__/no-focused-tests.test.ts +++ b/src/rules/__tests__/no-focused-tests.test.ts @@ -317,21 +317,5 @@ ruleTester.run('no-focused-tests', rule, { }, ], }, - { - code: 'ftest.each`table`()', - errors: [ - { - messageId: 'focusedTest', - column: 1, - line: 1, - suggestions: [ - { - messageId: 'suggestRemoveFocus', - output: 'test.each`table`()', - }, - ], - }, - ], - }, ], });