Skip to content

test: cover remote daemon env forwarding#350

Open
honor2030 wants to merge 1 commit into
browser-use:mainfrom
honor2030:fix/remote-daemon-env-regression
Open

test: cover remote daemon env forwarding#350
honor2030 wants to merge 1 commit into
browser-use:mainfrom
honor2030:fix/remote-daemon-env-regression

Conversation

@honor2030
Copy link
Copy Markdown

@honor2030 honor2030 commented May 13, 2026

Summary

Refs #181.

Test Plan

  • uv run --with pytest --with pillow python -m pytest tests/unit/test_admin.py -q
  • uv run --with pytest --with pillow python -m pytest tests -q
  • uv run python -m compileall src tests
  • uv pip check --python .venv/bin/python
  • uv run --with ruff python -m ruff check --select B tests/unit/test_admin.py

Summary by cubic

Add a regression test to ensure start_remote_daemon forwards the resolved remote CDP websocket and browser ID to ensure_daemon via env vars. Confirms BU_CDP_WS and BU_BROWSER_ID are set so the daemon doesn’t fall back to local Chrome discovery (guards against #181).

Written for commit 559bc1b. Summary will update on new commits.

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant