Skip to content

Commit 5c11882

Browse files
author
Guillaume Martigny
committed
Add auto-fix of misspelled 'falsey' to 'falsy'
fix avajs#190
1 parent c5a77bb commit 5c11882

File tree

2 files changed

+18
-1
lines changed

2 files changed

+18
-1
lines changed

rules/use-t-well.js

+13-1
Original file line numberDiff line numberDiff line change
@@ -12,13 +12,16 @@ const isCallExpression = node =>
1212
const getMemberStats = members => {
1313
const initial = {
1414
skip: [],
15+
falsey: [],
1516
method: [],
1617
other: []
1718
};
1819

1920
return members.reduce((res, member) => {
2021
if (member === 'skip') {
2122
res.skip.push(member);
23+
} else if (member === 'falsey') {
24+
res.falsey.push(member);
2225
} else if (isMethod(member)) {
2326
res.method.push(member);
2427
} else {
@@ -81,6 +84,14 @@ const create = context => {
8184
node,
8285
message: 'Too many chained uses of `skip`.'
8386
});
87+
} else if (stats.falsey.length > 0) {
88+
context.report({
89+
node,
90+
message: 'Misspelled `falsy` to `falsey`.',
91+
fix: fixer => {
92+
return fixer.replaceTextRange(node.property.range, 'falsy');
93+
}
94+
});
8495
} else if (stats.method.length > 1) {
8596
context.report({
8697
node,
@@ -107,6 +118,7 @@ module.exports = {
107118
meta: {
108119
docs: {
109120
url: util.getDocsUrl(__filename)
110-
}
121+
},
122+
fixable: 'code'
111123
}
112124
};

test/use-t-well.js

+5
Original file line numberDiff line numberDiff line change
@@ -114,6 +114,11 @@ ruleTester.run('use-t-well', rule, {
114114
{
115115
code: testCase('t.deepEqual.skip.skip(a, a);'),
116116
errors: [error('Too many chained uses of `skip`.')]
117+
},
118+
{
119+
code: testCase('t.falsey(a);'),
120+
output: testCase('t.falsy(a);'),
121+
errors: [error('Misspelled `falsy` to `falsey`.')]
117122
}
118123
]
119124
});

0 commit comments

Comments
 (0)