Skip to content

Commit 6142747

Browse files
TrottMylesBorins
authored andcommitted
tools: replace custom assert.fail lint rule
Replace custom lint rule for `assert.fail()` function signature errors with a restricted-syntax rule. PR-URL: #12287 Reviewed-By: Teddy Katz <teddy.katz@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com> Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com> Reviewed-By: James M Snell <jasnell@gmail.com>
1 parent d9866ce commit 6142747

File tree

2 files changed

+3
-31
lines changed

2 files changed

+3
-31
lines changed

.eslintrc.yaml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -113,6 +113,9 @@ rules:
113113
}, {
114114
selector: "ThrowStatement > CallExpression[callee.name=/Error$/]",
115115
message: "Use new keyword when throwing an Error."
116+
}, {
117+
selector: "CallExpression[callee.object.name='assert'][callee.property.name='fail'][arguments.length=1]",
118+
message: "assert.fail() message should be third argument"
116119
}]
117120
no-tabs: 2
118121
no-trailing-spaces: 2
@@ -143,7 +146,6 @@ rules:
143146

144147
# Custom rules in tools/eslint-rules
145148
align-multiline-assignment: 2
146-
assert-fail-single-argument: 2
147149
assert-throws-arguments: [2, { requireTwo: false }]
148150
no-useless-regex-char-class-escape: [2, { override: ['[', ']'] }]
149151

tools/eslint-rules/assert-fail-single-argument.js

Lines changed: 0 additions & 30 deletions
This file was deleted.

0 commit comments

Comments
 (0)