diff options
Diffstat (limited to 'deps/npm/test/tap/install-actions.js')
| -rw-r--r-- | deps/npm/test/tap/install-actions.js | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/deps/npm/test/tap/install-actions.js b/deps/npm/test/tap/install-actions.js index 824a31c1be..6ca6e33534 100644 --- a/deps/npm/test/tap/install-actions.js +++ b/deps/npm/test/tap/install-actions.js @@ -56,8 +56,10 @@ test('->optdep:a->dep:b', function (t) { moduleB.parent = tree t.plan(3) - actions.postinstall('/', moduleA, mockLog, function (er) { - t.is(er && er.code, 'ELIFECYCLE', 'Lifecycle failed') + return actions.postinstall('/', moduleA, mockLog).then(() => { + throw new Error('was not supposed to succeed') + }, (err) => { + t.is(err && err.code, 'ELIFECYCLE', 'Lifecycle failed') t.ok(moduleA.failed, 'moduleA (optional dep) is marked failed') t.ok(moduleB.failed, 'moduleB (direct dep of moduleA) is marked as failed') t.end() @@ -108,8 +110,10 @@ test('->dep:b,->optdep:a->dep:b', function (t) { moduleB.parent = tree t.plan(3) - actions.postinstall('/', moduleA, mockLog, function (er) { - t.ok(er && er.code === 'ELIFECYCLE', 'Lifecycle failed') + return actions.postinstall('/', moduleA, mockLog).then(() => { + throw new Error('was not supposed to succeed') + }, (err) => { + t.ok(err && err.code === 'ELIFECYCLE', 'Lifecycle failed') t.ok(moduleA.failed, 'moduleA (optional dep) is marked failed') t.ok(!moduleB.failed, 'moduleB (direct dep of moduleA) is marked as failed') t.end() |