-
Notifications
You must be signed in to change notification settings - Fork 652
Expand file tree
/
Copy pathprefer-assert-is-ok.ts
More file actions
103 lines (90 loc) · 3.73 KB
/
prefer-assert-is-ok.ts
File metadata and controls
103 lines (90 loc) · 3.73 KB
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
// Copyright 2024 The Chromium Authors
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
import type {TSESTree} from '@typescript-eslint/utils';
import {createRule} from './utils/ruleCreator.ts';
const FALSY_ASSERTIONS = new Set(['isFalse', 'isNotOk', 'isNotTrue', 'notOk']);
const TRUTHY_ASSERTIONS = new Set(['isNotFalse', 'isOk', 'isTrue', 'ok']);
type MessageIds =
'useAssertIsOk'|'useAssertIsOkInsteadOfNegation'|'useAssertIsNotOk'|'useAssertIsNotOkInsteadOfNegation';
function isAssertOk(node: TSESTree.Expression) {
return node.type === 'MemberExpression' && node.object.type === 'Identifier' && node.object.name === 'assert' &&
node.property.type === 'Identifier' && node.property.name === 'ok';
}
function isAssertNotOk(node: TSESTree.Expression) {
return node.type === 'MemberExpression' && node.object.type === 'Identifier' && node.object.name === 'assert' &&
node.property.type === 'Identifier' && node.property.name === 'notOk';
}
function isTruthyAssertion(node: TSESTree.Expression) {
if (node.type === 'Identifier' && node.name === 'assert') {
return true;
}
return node.type === 'MemberExpression' && node.object.type === 'Identifier' && node.object.name === 'assert' &&
node.property.type === 'Identifier' && TRUTHY_ASSERTIONS.has(node.property.name);
}
function isFalsyAssertion(node: TSESTree.Expression) {
return node.type === 'MemberExpression' && node.object.type === 'Identifier' && node.object.name === 'assert' &&
node.property.type === 'Identifier' && FALSY_ASSERTIONS.has(node.property.name);
}
export default createRule<unknown[], MessageIds>({
name: 'prefer-assert-is-ok',
meta: {
type: 'suggestion',
docs: {
description: 'Prefer `assert.isOk` and `assert.isNotOk` over `assert.ok` and `assert.notOk`.',
category: 'Best Practices',
},
messages: {
useAssertIsOk: 'Use `assert.isOk(e)` or `assert(e)` instead of `assert.ok(e)`',
useAssertIsOkInsteadOfNegation: 'Use `assert.isOk(e)` instead of `assert.isNotOk(!e)`',
useAssertIsNotOk: 'Use `assert.isNotOk(e)` or `assert(!e)` instead of `assert.notOk(e)`',
useAssertIsNotOkInsteadOfNegation: 'Use `assert.isNotOk(e)` instead of `assert.isOk(!e)`',
},
fixable: 'code',
schema: [], // no options
},
defaultOptions: [],
create: function(context) {
function reportError(
node: TSESTree.CallExpression,
calleeText: string,
firstArgNode: TSESTree.CallExpressionArgument,
messageId: MessageIds,
) {
context.report({
node,
messageId,
fix(fixer) {
const {sourceCode} = context;
return [
fixer.replaceText(node.callee, calleeText),
fixer.replaceText(node.arguments[0], sourceCode.getText(firstArgNode)),
];
}
});
}
return {
CallExpression(node) {
if (node.arguments.length < 1) {
return;
}
const [argumentNode] = node.arguments;
if (argumentNode.type === 'UnaryExpression' && argumentNode.operator === '!') {
if (isTruthyAssertion(node.callee)) {
reportError(node, 'assert.isNotOk', argumentNode.argument, 'useAssertIsNotOkInsteadOfNegation');
return;
}
if (isFalsyAssertion(node.callee)) {
reportError(node, 'assert.isOk', argumentNode.argument, 'useAssertIsOkInsteadOfNegation');
return;
}
}
if (isAssertOk(node.callee)) {
reportError(node, 'assert.isOk', argumentNode, 'useAssertIsOk');
} else if (isAssertNotOk(node.callee)) {
reportError(node, 'assert.isNotOk', argumentNode, 'useAssertIsNotOk');
}
}
};
},
});