[FIX] tests: catch remain processes

In some case a process could remain aive at the end of a tests
In addition to possible race condition, this can also cause a program to
remain stuck at the end of the tests.

This commit proposes to
- catch all remaining processes at the end of a base case.
- log a message if a process is found at the shutdown of the server.

closes odoo#232424

X-original-commit: 5f7f138
Signed-off-by: Xavier Dollé (xdo) <xdo@odoo.com>

Author: Xavier-Do

odoo/service/server.py

@@ -653,6 +653,11 @@ def stop(self):
             _logger.info("Logging uses the database, stop logging then close DB connections")
             log_ctx = mute_logger('')
 
+        current_process = psutil.Process()
+        children = current_process.children(recursive=False)
+        for child in children:
+            _logger.info('A child process was found, pid is %s, process may hang', child)
+
         with log_ctx:
             sql_db.close_all()
             _logger.debug('--')

odoo/tests/common.py

@@ -19,6 +19,7 @@
 import pathlib
 import platform
 import pprint
+import psutil
 import re
 import shutil
 import signal
@@ -382,6 +383,15 @@ def run(self, result: OdooTestResult) -> None:
 
     @classmethod
     def setUpClass(cls):
+        def check_remaining_processes():
+            current_process = psutil.Process()
+            children = current_process.children(recursive=False)
+            for child in children:
+                _logger.warning('A child process was found, terminating it: %s', child)
+                child.terminate()
+            psutil.wait_procs(children, timeout=10)  # mainly to avoid a zombie process that would be logged again at the end.
+        cls.addClassCleanup(check_remaining_processes)
+
         def check_remaining_patchers():
             for patcher in _patch._active_patches:
                 _logger.warning("A patcher (targeting %s.%s) was remaining active at the end of %s, disabling it...", patcher.target, patcher.attribute, cls.__name__)