Skip to content

Commit

Permalink
Merge pull request #100 from vim-denops/term-host-in-test
Browse files Browse the repository at this point in the history
🌿 Properly terminate Vim/Neovim after `test()`
  • Loading branch information
lambdalisue committed Aug 14, 2021
2 parents 3fe0390 + ea23165 commit add6971
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 0 deletions.
2 changes: 2 additions & 0 deletions denops/@denops/test/runner.ts
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,8 @@ export function run(
cmd: [...args, ...cmds],
env: options.env,
stdin: "piped",
stdout: options.verbose ? "inherit" : "null",
stderr: options.verbose ? "inherit" : "null",
});
return proc;
}
Expand Down
2 changes: 2 additions & 0 deletions denops/@denops/test/tester.ts
Original file line number Diff line number Diff line change
Expand Up @@ -70,6 +70,8 @@ async function withDenops(
);
} finally {
proc.stdin?.close();
proc.kill(Deno.Signal.SIGTERM);
await proc.status();
proc.close();
conn.close();
listener.close();
Expand Down

0 comments on commit add6971

Please sign in to comment.