Skip to content

Commit 23ec6f0

Browse files
authored
Fix CI tests related to .kill(0) (#1212)
1 parent 9a2cb79 commit 23ec6f0

File tree

1 file changed

+7
-10
lines changed

1 file changed

+7
-10
lines changed

test/return/result.js

Lines changed: 7 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -111,16 +111,13 @@ test('result.isTerminated is false if not killed', async t => {
111111
t.false(isTerminated);
112112
});
113113

114-
// Remove the condition after fixing the bug at https://github.com/sindresorhus/execa/issues/1193
115-
if (!isWindows) {
116-
test('result.isTerminated is false if not killed and subprocess.kill() was called', async t => {
117-
const subprocess = execa('noop.js');
118-
subprocess.kill(0);
119-
t.true(subprocess.killed);
120-
const {isTerminated} = await subprocess;
121-
t.false(isTerminated);
122-
});
123-
}
114+
test('result.isTerminated is false if not killed and subprocess.kill() was called', async t => {
115+
const subprocess = execa('noop.js');
116+
subprocess.kill(0);
117+
t.true(subprocess.killed);
118+
const {isTerminated} = await subprocess;
119+
t.false(isTerminated);
120+
});
124121

125122
test('result.isTerminated is false if not killed, in sync mode', t => {
126123
const {isTerminated} = execaSync('noop.js');

0 commit comments

Comments
 (0)