From 2cf1a288b1913f1cbeaea2918d21929f390ac1de Mon Sep 17 00:00:00 2001 From: ehmicky Date: Thu, 27 Jun 2019 10:00:00 +0200 Subject: [PATCH] Improve tests for the `timeout` option --- test/kill.js | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/test/kill.js b/test/kill.js index 67e726ab19..e331c1829b 100644 --- a/test/kill.js +++ b/test/kill.js @@ -80,7 +80,7 @@ test('execa() returns a promise with kill()', t => { }); test('timeout kills the process if it times out', async t => { - const {killed, timedOut} = await t.throwsAsync(execa('forever', {timeout: 1, message: TIMEOUT_REGEXP})); + const {killed, timedOut} = await t.throwsAsync(execa('forever', {timeout: 1}), TIMEOUT_REGEXP); t.false(killed); t.true(timedOut); }); @@ -98,12 +98,17 @@ test('timeout does not kill the process if it does not time out', async t => { t.false(timedOut); }); -test('timedOut is false if no timeout was set', async t => { +test('timedOut is false if timeout is undefined', async t => { const {timedOut} = await execa('noop'); t.false(timedOut); }); -test('timedOut will be false if no timeout was set and zero exit code in sync mode', t => { +test('timedOut is false if timeout is 0', async t => { + const {timedOut} = await execa('noop', {timeout: 0}); + t.false(timedOut); +}); + +test('timedOut is false if timeout is undefined and exit code is 0 in sync mode', t => { const {timedOut} = execa.sync('noop'); t.false(timedOut); });