set tmp_path only

Author: ReimarBauer

tests/fixtures.py

@@ -83,6 +83,11 @@ def qtbot(qtbot, fail_if_open_message_boxes_left, close_remaining_widgets):
     qtbot.wait(5000)
 
 
+@pytest.fixture
+def msui_configs(tmp_path):
+    modify_config_file({"mss_dir": tmp_path.name})
+
+
 @pytest.fixture(scope="session")
 def mscolab_session_app():
     """Session-scoped fixture that provides the WSGI app instance for MSColab.
@@ -141,11 +146,6 @@ def reset_mscolab(mscolab_session_app):
         handle_db_reset()
 
 
-@pytest.fixture(scope="session")
-def msui_configs(tmp_path):
-    modify_config_file({"mss_dir": tmp_path.name})
-
-
 @pytest.fixture
 def mscolab_app(mscolab_session_app, reset_mscolab):
     """Fixture that provides the MSColab WSGI app instance and does cleanup actions.
@@ -196,60 +196,20 @@ def _running_eventlet_server(app):
     """Context manager that starts the app in an eventlet server and returns its URL."""
     scheme = "http"
     host = "127.0.0.1"
-
+    socket = eventlet.listen((host, 0))
+    port = socket.getsockname()[1]
+    url = f"{scheme}://{host}:{port}"
+    app.config['URL'] = url
     if "fork" not in multiprocessing.get_all_start_methods():
         pytest.skip("requires the multiprocessing start_method 'fork', which is unavailable on this system")
-
     ctx = multiprocessing.get_context("fork")
-
-    def _serve_in_child(conn, _app, _host, _scheme):
-        # Create the listening socket inside the child to avoid passing GreenSocket/FD across fork.
-        sock = eventlet.listen((_host, 0))
-        port = sock.getsockname()[1]
-        url = f"{_scheme}://{_host}:{port}"
-
-        try:
-            _app.config["URL"] = url
-        except Exception:
-            # If app/config is not writable for some reason, still start server.
-            pass
-
-        conn.send(port)
-        conn.close()
-
-        # Run until terminated by parent.
-        eventlet.wsgi.server(sock, _app)
-
-    parent_conn, child_conn = ctx.Pipe(duplex=False)
-    process = ctx.Process(target=_serve_in_child, args=(child_conn, app, host, scheme), daemon=True)
-
+    process = ctx.Process(target=eventlet.wsgi.server, args=(socket, app), daemon=True)
     try:
         process.start()
-
-        # Wait for the child to report the bound port.
-        if not parent_conn.poll(5):
-            raise RuntimeError("Server did not report its port within 5 seconds")
-        port = parent_conn.recv()
-
-        url = f"{scheme}://{host}:{port}"
-        app.config["URL"] = url
-
-        start_time = time.time()
-        sleep_time = 0.01
         while not is_url_response_ok(urllib.parse.urljoin(url, "index")):
-            if (time.time() - start_time) > 5:
-                raise RuntimeError(f"Server did not start within 5 seconds at {url}")
-            time.sleep(sleep_time)
-            sleep_time *= 2
-            if sleep_time > 1:
-                sleep_time = 1
-
+            time.sleep(0.5)
         yield url
     finally:
-        try:
-            parent_conn.close()
-        except Exception:
-            pass
         process.terminate()
         process.join(10)
         process.close()