Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit 763a252

Browse files
authored
fix(eslint-plugin): [no-var-requires] report when used in type assertion (typescript-eslint#3074)
1 parent 1bd8f4e commit 763a252

File tree

2 files changed

+11
-0
lines changed

2 files changed

+11
-0
lines changed

packages/eslint-plugin/src/rules/no-var-requires.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,7 @@ export default util.createRule<Options, MessageIds>({
3737
(parent.type === AST_NODE_TYPES.VariableDeclarator ||
3838
parent.type === AST_NODE_TYPES.CallExpression ||
3939
parent.type === AST_NODE_TYPES.TSAsExpression ||
40+
parent.type === AST_NODE_TYPES.TSTypeAssertion ||
4041
parent.type === AST_NODE_TYPES.MemberExpression)
4142
) {
4243
context.report({

packages/eslint-plugin/tests/rules/no-var-requires.test.ts

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -112,6 +112,16 @@ ruleTester.run('no-var-requires', rule, {
112112
},
113113
],
114114
},
115+
{
116+
code: "const foo = <Foo>require('./foo.json');",
117+
errors: [
118+
{
119+
messageId: 'noVarReqs',
120+
line: 1,
121+
column: 18,
122+
},
123+
],
124+
},
115125
{
116126
code: "const foo: Foo = require('./foo.json').default;",
117127
errors: [

0 commit comments

Comments
 (0)