Skip to content

Commit 72b1970

Browse files
Roman3349veritem
andauthored
fix: fix default value propagation (#866)
* fix: fix default value propagation Signed-off-by: Roman Ondráček <mail@romanondracek.cz> * fix(utils): fix TypeScript errors in parse-vitest-fn-call Signed-off-by: Roman Ondráček <mail@romanondracek.cz> * update package-lock yml file --------- Signed-off-by: Roman Ondráček <mail@romanondracek.cz> Co-authored-by: Verite Mugabo <mugaboverite@gmail.com>
1 parent 472ba3d commit 72b1970

File tree

79 files changed

+228
-192
lines changed

Some content is hidden

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

79 files changed

+228
-192
lines changed

package.json

Lines changed: 4 additions & 4 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.51.0",
45-
"@typescript-eslint/utils": "^8.51.0"
44+
"@typescript-eslint/scope-manager": "^8.55.0",
45+
"@typescript-eslint/utils": "^8.55.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.51.0",
52-
"@typescript-eslint/rule-tester": "^8.51.0",
51+
"@typescript-eslint/parser": "^8.55.0",
52+
"@typescript-eslint/rule-tester": "^8.55.0",
5353
"bumpp": "^10.3.1",
5454
"concurrently": "^9.2.1",
5555
"eslint": "^9.37.0",

pnpm-lock.yaml

Lines changed: 132 additions & 15 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: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,6 @@ export default createEslintRule<[Partial<Options>], MessageIds>({
5454
],
5555
defaultOptions: [{}],
5656
},
57-
defaultOptions: [{}],
5857
create(context, [options]) {
5958
return {
6059
CallExpression(node: TSESTree.CallExpression) {

src/rules/consistent-test-filename.ts

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -33,23 +33,21 @@ export default createEslintRule<
3333
pattern: {
3434
type: 'string',
3535
format: 'regex',
36-
default: defaultPattern.source,
3736
},
3837
allTestPattern: {
3938
type: 'string',
4039
format: 'regex',
41-
default: defaultTestsPattern.source,
4240
},
4341
},
4442
},
4543
],
44+
defaultOptions: [
45+
{
46+
pattern: defaultPattern.source,
47+
allTestPattern: defaultTestsPattern.source,
48+
},
49+
],
4650
},
47-
defaultOptions: [
48-
{
49-
pattern: defaultPattern.source,
50-
allTestPattern: defaultTestsPattern.source,
51-
},
52-
],
5351

5452
create: (context, options) => {
5553
const { pattern: patternRaw, allTestPattern: allTestPatternRaw } =

src/rules/consistent-test-it.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,8 +78,8 @@ export default createEslintRule<
7878
additionalProperties: false,
7979
},
8080
],
81+
defaultOptions: [{}],
8182
},
82-
defaultOptions: [{}],
8383
create(context, options) {
8484
const { fn, withinDescribe } = options[0]
8585
const testFnKeyWork = fn || TestCaseName.test

src/rules/consistent-vitest-vi.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,14 +29,13 @@ export default createEslintRule<[Partial<{ fn: UtilName }>], MESSAGE_ID>({
2929
fn: {
3030
type: 'string',
3131
enum: [UtilName.vi, UtilName.vitest],
32-
default: UtilName.vi,
3332
},
3433
},
3534
additionalProperties: false,
3635
},
3736
],
37+
defaultOptions: [{ fn: UtilName.vi }],
3838
},
39-
defaultOptions: [{ fn: UtilName.vi }],
4039
create(context, options) {
4140
const utilKeyword = options[0].fn!
4241
const oppositeUtilKeyword = getOppositeVitestUtilKeyword(utilKeyword)

src/rules/expect-expect.ts

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -49,12 +49,6 @@ export default createEslintRule<Options, MESSAGE_ID>({
4949
noAssertions: 'Test has no assertions',
5050
},
5151
},
52-
defaultOptions: [
53-
{
54-
assertFunctionNames: ['expect', 'assert'],
55-
additionalTestBlockFunctions: [],
56-
},
57-
],
5852
create(
5953
context,
6054
[{ assertFunctionNames = ['expect'], additionalTestBlockFunctions = [] }],

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

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,6 @@ export default createEslintRule<[], MESSAGE_ID>({
2727
},
2828
schema: [],
2929
},
30-
defaultOptions: [],
3130
create(context) {
3231
let lastImportEnd: null | number = null
3332
const nodesToReport: Array<TSESTree.CallExpression> = []

0 commit comments

Comments
 (0)