Skip to content

Commit

Permalink
docs(no-focused-tests): remove references to ftest method (#816)
Browse files Browse the repository at this point in the history
There's no such prefixed method for `test`; only `fit` & `fdescribe`
  • Loading branch information
G-Rath authored Apr 10, 2021
1 parent 72fe0c6 commit ce76579
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 21 deletions.
6 changes: 1 addition & 5 deletions docs/rules/no-focused-tests.md
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand All @@ -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:
Expand Down
16 changes: 0 additions & 16 deletions src/rules/__tests__/no-focused-tests.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -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`()',
},
],
},
],
},
],
});

0 comments on commit ce76579

Please sign in to comment.