/
prefer-replace-all.js
89 lines (86 loc) · 1.75 KB
/
prefer-replace-all.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
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
import test from 'ava';
import avaRuleTester from 'eslint-ava-rule-tester';
const rule = require('../rules/prefer-replace-all');
const ruleTester = avaRuleTester(test, {
env: {
es6: true
}
});
const error = {
ruleId: 'prefer-replace-all',
message: 'Prefer `String#replaceAll()` over `String#replace()`.'
};
ruleTester.run('prefer-replace-all', rule, {
valid: [
// No global flag
'foo.replace(/a/, bar)',
// Special characters
'foo.replace(/[a]/g, bar)',
'foo.replace(/a?/g, bar)',
'foo.replace(/.*/g, bar)',
'foo.replace(/\\W/g, bar)',
// Extra flag
'foo.replace(/a/gi, bar)',
// Not regex literal
'foo.replace(\'string\', bar)',
// Not 2 arguments
'foo.replace(/a/g)',
'foo.replace(/\\\\./g)',
// New
'new foo.replace(/a/g, bar)',
// Function call
'replace(/a/g, bar)',
// Not call
'foo.replace',
// Not replace
'foo.methodNotReplace(/a/g, bar);',
// `replace` is not Identifier
'foo[\'replace\'](/a/g, bar)'
],
invalid: [
{
code: 'foo.replace(/a/g, bar)',
output: 'foo.replaceAll(\'a\', bar)',
errors: [error]
},
// Comments
{
code: `
foo/* comment 1 */
.replace/* comment 2 */(
/* comment 3 */
/a/g // comment 4
,
bar
)
`,
output: `
foo/* comment 1 */
.replaceAll/* comment 2 */(
/* comment 3 */
'a' // comment 4
,
bar
)
`,
errors: [error]
},
// Quotes
{
code: 'foo.replace(/"\'/g, \'\\\'\')',
output: 'foo.replaceAll(\'"\\\'\', \'\\\'\')',
errors: [error]
},
// Escaped symbols
{
code: 'foo.replace(/\\./g, bar)',
output: 'foo.replaceAll(\'.\', bar)',
errors: [error]
},
{
code: 'foo.replace(/\\\\\\./g, bar)',
output: 'foo.replaceAll(\'\\.\', bar)',
errors: [error]
}
]
});