From 559bc1bdadf239d9aa64fb55a3db9adb521f082e Mon Sep 17 00:00:00 2001 From: honor2030 <20864931+honor2030@users.noreply.github.com> Date: Wed, 13 May 2026 17:09:27 +0900 Subject: [PATCH] test: cover remote daemon env forwarding --- tests/unit/test_admin.py | 22 ++++++++++++++++++++++ 1 file changed, 22 insertions(+) diff --git a/tests/unit/test_admin.py b/tests/unit/test_admin.py index 6826b399..ce03d08a 100644 --- a/tests/unit/test_admin.py +++ b/tests/unit/test_admin.py @@ -264,6 +264,28 @@ def fake_browser_use(path, method, body=None): ] +def test_start_remote_daemon_forwards_remote_cdp_env_to_daemon(monkeypatch): + ensure_calls = [] + browser = {"id": "browser-123", "cdpUrl": "https://cdp.example", "liveUrl": "https://live.example"} + + monkeypatch.setattr(admin, "daemon_alive", lambda name: False) + monkeypatch.setattr(admin, "_browser_use", lambda path, method, body=None: browser) + monkeypatch.setattr(admin, "_cdp_ws_from_url", lambda url: "ws://example.test/devtools/browser/1") + monkeypatch.setattr(admin, "ensure_daemon", lambda **kwargs: ensure_calls.append(kwargs)) + monkeypatch.setattr(admin, "_show_live_url", lambda url: None) + + assert admin.start_remote_daemon("work", proxyCountryCode="us") == browser + assert ensure_calls == [ + { + "name": "work", + "env": { + "BU_CDP_WS": "ws://example.test/devtools/browser/1", + "BU_BROWSER_ID": "browser-123", + }, + } + ] + + # --- restart_daemon: PID-reuse safety --- def test_restart_daemon_does_not_signal_when_daemon_unreachable(monkeypatch, tmp_path):