Skip to content

Commit 86f4b4d

Browse files
authored
Merge pull request #46 from nodaguti/stop-showing-empty-message
Fix an issue where messages without text property are shown as `undefined [plugin-name]`
2 parents 390f475 + 35d50d0 commit 86f4b4d

File tree

2 files changed

+30
-1
lines changed

2 files changed

+30
-1
lines changed

lib/formatter.js

+3-1
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,9 @@ module.exports = function(opts) {
1010
var positionless = options.positionless || 'first';
1111

1212
return function(input) {
13-
var messages = input.messages;
13+
var messages = input.messages.filter(function(message) {
14+
return typeof message.text === 'string';
15+
});
1416
var source = input.source;
1517

1618
if (!messages.length) return '';

test/formatter.js

+27
Original file line numberDiff line numberDiff line change
@@ -256,3 +256,30 @@ test('defaultFormatter with real sourcemaps', function(t) {
256256
);
257257
t.end();
258258
});
259+
260+
var textlessMessages = [
261+
{
262+
type: 'warning',
263+
plugin: 'foo',
264+
},
265+
{
266+
type: 'dependency',
267+
plugin: 'bar',
268+
file: 'bar file',
269+
},
270+
];
271+
272+
var textlessMessagesOutput = '';
273+
274+
test('defaultFormatter with messages without text property', function(t) {
275+
t.equal(
276+
stripColor(defaultFormatter({
277+
messages: textlessMessages,
278+
source: '<input css 1>',
279+
})),
280+
textlessMessagesOutput,
281+
'basic'
282+
);
283+
284+
t.end();
285+
});

0 commit comments

Comments
 (0)