mirror of
https://github.com/arcan1s/ahriman.git
synced 2025-04-24 07:17:17 +00:00
do not terminate child process explicitly
This feature sometimes causes the main process termination. Since the child process has been already terminated (or going to) there is no need to terminate it manually
This commit is contained in:
parent
be6cf5b8af
commit
d99091a3b4
@ -172,7 +172,6 @@ class Spawn(Thread, LazyLogging):
|
||||
process = self.active.pop(process_id, None)
|
||||
|
||||
if process is not None:
|
||||
process.terminate() # make sure lol
|
||||
process.join()
|
||||
|
||||
def stop(self) -> None:
|
||||
|
@ -140,9 +140,7 @@ def test_run_pop(spawner: Spawn) -> None:
|
||||
|
||||
spawner.run()
|
||||
|
||||
first.terminate.assert_called_once_with()
|
||||
first.join.assert_called_once_with()
|
||||
second.terminate.assert_called_once_with()
|
||||
second.join.assert_called_once_with()
|
||||
assert not spawner.active
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user