Skip to content

Commit 617d1cc

Browse files
committed
Update: utils.getTestInfo filter elements equal to null (fixes eslint-community#37).
1 parent 0a81434 commit 617d1cc

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

lib/utils.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -179,8 +179,8 @@ module.exports = {
179179
const invalidProperty = run.properties.find(prop => module.exports.getKeyName(prop) === 'invalid');
180180

181181
return {
182-
valid: validProperty && validProperty.value.type === 'ArrayExpression' ? validProperty.value.elements : null,
183-
invalid: invalidProperty && invalidProperty.value.type === 'ArrayExpression' ? invalidProperty.value.elements : null,
182+
valid: validProperty && validProperty.value.type === 'ArrayExpression' ? validProperty.value.elements.filter(Boolean) : [],
183+
invalid: invalidProperty && invalidProperty.value.type === 'ArrayExpression' ? invalidProperty.value.elements.filter(Boolean) : [],
184184
};
185185
});
186186
},

tests/lib/utils.js

+1
Original file line numberDiff line numberDiff line change
@@ -213,6 +213,7 @@ describe('utils', () => {
213213
'var foo = new RuleTester(); foo.run(bar, baz, { valid: [foo], invalid: [bar] })': { valid: 1, invalid: 1 },
214214
'var foo = new (require("eslint")).RuleTester; foo.run(bar, baz, { valid: [], invalid: [] })': { valid: 0, invalid: 0 },
215215
'var foo = new bar.RuleTester; foo.run(bar, baz, { valid: [], invalid: [bar, baz] })': { valid: 0, invalid: 2 },
216+
'var foo = new bar.RuleTester; foo.run(bar, baz, { valid: [,], invalid: [bar, , baz] })': { valid: 0, invalid: 2 },
216217
};
217218

218219
Object.keys(CASES).forEach(testSource => {

0 commit comments

Comments
 (0)