forked from ChromeDevTools/devtools-frontend
-
Notifications
You must be signed in to change notification settings - Fork 0
/
Copy pathno-assert-equal.js
68 lines (64 loc) · 1.99 KB
/
no-assert-equal.js
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
// Copyright 2020 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.
/**
* @fileoverview Rule to ban usage of assert.equal
* @author Jack Franklin
*/
'use strict';
// ------------------------------------------------------------------------------
// Rule Definition
// ------------------------------------------------------------------------------
/**
* @type {import('eslint').Rule.RuleModule}
*/
module.exports = {
meta: {
type: 'problem',
docs: {
description: 'Usage of assert.equal',
category: 'Possible Errors',
},
fixable: 'code',
schema: [], // no options
},
create: function (context) {
return {
CallExpression(node) {
if (
node.callee.type === 'MemberExpression' &&
node.callee.object.name === 'assert' &&
node.callee.property.name === 'equal'
) {
context.report({
node,
message:
'assert.equal is non-strict. Use assert.strictEqual or assert.deepEqual to compare objects',
fix(fixer) {
/**
* Get the type of the second argument and try to match it to a assert type
*/
const compareToType = node.arguments.at(1)?.type;
if (
// Match number or string
compareToType === 'Literal' ||
// Match `` string
compareToType === 'TemplateElement'
) {
return fixer.replaceText(node.callee.property, 'strictEqual');
}
if (
// Match any object `{...}`
compareToType === 'ObjectExpression' ||
// Match any array `[...]`
compareToType === 'ArrayExpression'
) {
return fixer.replaceText(node.callee.property, 'deepEqual');
}
},
});
}
},
};
},
};