-
Notifications
You must be signed in to change notification settings - Fork 498
/
Copy pathprefer-assert-strict-equal.ts
86 lines (79 loc) · 3.16 KB
/
prefer-assert-strict-equal.ts
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
// Copyright 2025 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
import {createRule} from './utils/ruleCreator.ts';
const FALSY_ASSERTIONS = new Set(['isFalse', 'isNotOk', 'isNotTrue', 'notOk']);
const TRUTHY_ASSERTIONS = new Set(['isNotFalse', 'isOk', 'isTrue', 'ok']);
export default createRule({
name: 'prefer-assert-strict-equal',
meta: {
type: 'suggestion',
docs: {
description:
'Prefer explicit `assert.strictEqual(x, y)` or `assert.strictNotEqual(x, y)` over generic `assert(x === y)` or `assert.isFalse(x === y)',
category: 'Best Practices',
},
messages: {
useAssertStrictEqual: 'Use `assert.strictEqual` to assert that two values are strictly equal',
useAssertNotStrictEqual: 'Use `assert.notStrictEqual` to assert that two values aren\'t strictly equal',
},
fixable: 'code',
schema: [], // no options
},
defaultOptions: [],
create: function(context) {
function isTruthyAssertion(calleeNode) {
if (calleeNode.type === 'Identifier' && calleeNode.name === 'assert') {
return true;
}
return calleeNode.type === 'MemberExpression' && calleeNode.object.type === 'Identifier' &&
calleeNode.object.name === 'assert' && calleeNode.property.type === 'Identifier' &&
TRUTHY_ASSERTIONS.has(calleeNode.property.name);
}
function isFalsyAssertion(calleeNode) {
return calleeNode.type === 'MemberExpression' && calleeNode.object.type === 'Identifier' &&
calleeNode.object.name === 'assert' && calleeNode.property.type === 'Identifier' &&
FALSY_ASSERTIONS.has(calleeNode.property.name);
}
function reportError(node, calleeText, messageId) {
context.report({
node,
messageId,
fix(fixer) {
const {sourceCode} = context;
const calleeNode = node.callee;
const [argumentNode] = node.arguments;
const argumentText = `${sourceCode.getText(argumentNode.left)}, ${sourceCode.getText(argumentNode.right)}`;
return [
fixer.replaceText(calleeNode, calleeText),
fixer.replaceText(argumentNode, argumentText),
];
}
});
}
return {
CallExpression(node) {
if (node.arguments.length === 0) {
return;
}
const [argumentNode] = node.arguments;
if (argumentNode.type !== 'BinaryExpression') {
return;
}
if (argumentNode.operator === '===') {
if (isTruthyAssertion(node.callee)) {
reportError(node, 'assert.strictEqual', 'useAssertStrictEqual');
} else if (isFalsyAssertion(node.callee)) {
reportError(node, 'assert.notStrictEqual', 'useAssertNotStrictEqual');
}
} else if (argumentNode.operator === '!==') {
if (isFalsyAssertion(node.callee)) {
reportError(node, 'assert.strictEqual', 'useAssertStrictEqual');
} else if (isTruthyAssertion(node.callee)) {
reportError(node, 'assert.notStrictEqual', 'useAssertNotStrictEqual');
}
}
}
};
},
});