Skip to content

Commit 6beca38

Browse files
authored
refactor: Require rule name in createTestingLibraryRule return type and use rule.name in tests (#860)
* deps: update typescript-eslint related packages * refactor: update rule imports to use rule.name instead of RULE_NAME * refactor: remove export of RULE_NAME * refactor: use RULE_NAME constant for rule name in prefer-expect-assertions
1 parent 9636ae8 commit 6beca38

File tree

163 files changed

+376
-488
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

163 files changed

+376
-488
lines changed

package.json

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -41,15 +41,15 @@
4141
"update:eslint-docs": "pnpm build && eslint-doc-generator"
4242
},
4343
"dependencies": {
44-
"@typescript-eslint/scope-manager": "^8.46.1",
45-
"@typescript-eslint/utils": "^8.46.1"
44+
"@typescript-eslint/scope-manager": "^8.51.0",
45+
"@typescript-eslint/utils": "^8.51.0"
4646
},
4747
"devDependencies": {
4848
"@eslint/js": "^9.38.0",
4949
"@types/eslint": "^9.6.1",
5050
"@types/node": "^22.18.10",
51-
"@typescript-eslint/parser": "^8.46.1",
52-
"@typescript-eslint/rule-tester": "^8.46.1",
51+
"@typescript-eslint/parser": "^8.51.0",
52+
"@typescript-eslint/rule-tester": "^8.51.0",
5353
"bumpp": "^10.3.1",
5454
"concurrently": "^9.2.1",
5555
"eslint": "^9.37.0",
@@ -64,7 +64,7 @@
6464
"tsdown": "^0.15.10",
6565
"tsx": "^4.20.6",
6666
"typescript": "^5.9.3",
67-
"typescript-eslint": "^8.46.1",
67+
"typescript-eslint": "^8.51.0",
6868
"vitest": "^3.2.4"
6969
},
7070
"peerDependencies": {

pnpm-lock.yaml

Lines changed: 105 additions & 215 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/rules/consistent-each-for.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ import { TSESTree } from '@typescript-eslint/utils'
22
import { createEslintRule, getAccessorValue } from '../utils'
33
import { parseVitestFnCall } from '../utils/parse-vitest-fn-call'
44

5-
export const RULE_NAME = 'consistent-each-for'
5+
const RULE_NAME = 'consistent-each-for'
66
export type MessageIds = 'consistentMethod'
77

88
type EachOrFor = 'each' | 'for'

src/rules/consistent-test-filename.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import { createEslintRule } from '../utils'
22

3-
export const RULE_NAME = 'consistent-test-filename'
3+
const RULE_NAME = 'consistent-test-filename'
44

55
const defaultPattern = /.*\.test\.[tj]sx?$/
66
const defaultTestsPattern = /.*\.(test|spec)\.[tj]sx?$/

src/rules/consistent-test-it.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import {
66
} from '../utils/parse-vitest-fn-call'
77
import { TestCaseName } from '../utils/types'
88

9-
export const RULE_NAME = 'consistent-test-it'
9+
const RULE_NAME = 'consistent-test-it'
1010
export type MessageIds = 'consistentMethod' | 'consistentMethodWithinDescribe'
1111

1212
const buildFixer =

src/rules/consistent-vitest-vi.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import { createEslintRule, isSupportedAccessor } from '../utils'
33
import { UtilName } from '../utils/types'
44
import { parseVitestFnCall } from '../utils/parse-vitest-fn-call'
55

6-
export const RULE_NAME = 'consistent-vitest-vi'
6+
const RULE_NAME = 'consistent-vitest-vi'
77
export type MESSAGE_ID = 'consistentUtil'
88

99
const getOppositeVitestUtilKeyword = (util: UtilName) =>

src/rules/expect-expect.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import {
66
isTypeOfVitestFnCall,
77
} from '../utils/parse-vitest-fn-call'
88

9-
export const RULE_NAME = 'expect-expect'
9+
const RULE_NAME = 'expect-expect'
1010
export type MESSAGE_ID = 'noAssertions'
1111
type Options = [
1212
{

src/rules/hoisted-apis-on-top.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import { AST_NODE_TYPES, TSESLint, TSESTree } from '@typescript-eslint/utils'
22
import { createEslintRule } from '../utils'
33

4-
export const RULE_NAME = 'hoisted-apis-on-top'
4+
const RULE_NAME = 'hoisted-apis-on-top'
55

66
type MESSAGE_ID =
77
| 'hoistedApisOnTop'

src/rules/max-expects.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import {
55
parseVitestFnCall,
66
} from '../utils/parse-vitest-fn-call'
77

8-
export const RULE_NAME = 'max-expects'
8+
const RULE_NAME = 'max-expects'
99
export type MESSAGE_ID = 'maxExpect'
1010
export type Options = [
1111
{

src/rules/max-nested-describe.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import { TSESTree } from '@typescript-eslint/utils'
22
import { createEslintRule } from '../utils'
33

4-
export const RULE_NAME = 'max-nested-describe'
4+
const RULE_NAME = 'max-nested-describe'
55
export type MESSAGE_ID = 'maxNestedDescribe'
66
export type Options = [
77
{

0 commit comments

Comments
 (0)