Skip to content

Commit 79e89ad

Browse files
LitoMoreSBoudrias
authored andcommitted
Fix failing test (#155)
1 parent c8faa84 commit 79e89ad

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
"node": ">=8"
1414
},
1515
"scripts": {
16-
"test": "xo && ava --timeout=20s"
16+
"test": "xo && ava --timeout=20s -s"
1717
},
1818
"files": [
1919
"index.js",

test/notify.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ function Control(shouldNotifyInNpmScript) {
2121
const setupTest = isNpmReturnValue => {
2222
['..', 'is-npm'].forEach(clearModule);
2323
process.stdout.isTTY = true;
24-
mock('is-npm', isNpmReturnValue || false);
24+
mock('is-npm', {isNpm: isNpmReturnValue || false});
2525
const updateNotifier = require('..');
2626
util.inherits(Control, updateNotifier.UpdateNotifier);
2727
};
@@ -70,7 +70,7 @@ test('shouldNotifyInNpmScript should default to false', t => {
7070
t.not(stripAnsi(errorLogs).indexOf('Update available'), -1);
7171
});
7272

73-
test.failing('suppress output when running as npm script', t => {
73+
test('suppress output when running as npm script', t => {
7474
setupTest(true);
7575
const notifier = new Control();
7676
notifier.notify({defer: false});

0 commit comments

Comments
 (0)