Merge pull request #57 from NA-Wen/killtestprocess_windows

make sure test process can be killed on windows
This commit is contained in:
Thinkwee 2023-09-14 15:39:42 +08:00 committed by GitHub
commit ea3d415b1c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -111,6 +111,8 @@ class ChatEnv:
os.killpg(os.getpgid(process.pid), signal.SIGTERM)
else:
os.kill(process.pid, signal.SIGTERM)
if process.poll() is None:
os.kill(process.pid,signal.CTRL_BREAK_EVENT)
if return_code == 0:
return False, success_info