github/deer-flow
1
0
Fork 0
mirror of https://github.com/bytedance/deer-flow.git synced 2026-10-26 13:16:24 +00:00
Code Issues Packages Projects Releases Wiki Activity

test(lark): make blocking I/O CLI fixture work on Windows (#5826)

Browse Source 
* test(lark): make blocking I/O CLI fixture work on Windows

* docs(tests): document portable Lark CLI fixtures

---------

Co-authored-by: Zhangchenhao-111 <285911435+Zhangchenhao-111@users.noreply.github.com>
This commit is contained in:
haochen 2026-09-26 08:17:28 +08:00 committed by GitHub
parent 74f6dce489
commit fc9fb2dea2
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 43 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
backend/tests/AGENTS.md
View File

@ -2,6 +2,14 @@
Backend tests must preserve the runtime invariants they exercise without changing production execution topology.
## Lark CLI blocking-I/O fixtures
`blocking_io/test_integrations_router.py` uses a real local CLI stub: a `.cmd`
script on Windows and an executable shell script on POSIX. Resolve it through
the production PATH lookup, including a directory containing spaces. Seed fake
app credentials so auth completion reaches the CLI instead of returning early,
and keep fixture filesystem work behind `asyncio.to_thread`.
## Shared sandbox search contracts
`test_sandbox_search_contract.py` runs shared `ls`/`glob`/`grep` scenarios through

36
backend/tests/blocking_io/test_integrations_router.py
View File

@ -38,6 +38,26 @@ def _build_lark_archive(archive: Path) -> None:
def _write_stub_lark_cli(path: Path) -> None:
path.parent.mkdir(parents=True, exist_ok=True)
if os.name == "nt":
path.write_text(
"""@echo off
if "%~1" == "--version" (
echo v1.0.65
exit /b 0
)
if "%~1" == "auth" if "%~2" == "status" (
echo {"identities":{"user":{"userName":"Alice"}}}
exit /b 0
)
echo {}
exit /b 0
""",
encoding="utf-8",
)
return
path.write_text(
"""#!/bin/sh
if [ "$1" = "--version" ]; then
@ -119,11 +139,22 @@ async def test_lark_install_route_does_not_block_event_loop(tmp_path: Path, monk
async def test_lark_auth_complete_route_does_not_block_event_loop(tmp_path: Path, monkeypatch: pytest.MonkeyPatch) -> None:
_reset_paths(tmp_path, monkeypatch)
config = await _config(tmp_path)
cli_path = tmp_path / "bin" / "lark-cli"
cli_name = "lark-cli.cmd" if os.name == "nt" else "lark-cli"
cli_path = tmp_path / "cli bin" / cli_name
await asyncio.to_thread(_write_stub_lark_cli, cli_path)
monkeypatch.setenv("PATH", f"{cli_path.parent}{os.pathsep}{os.environ.get('PATH', '')}")
monkeypatch.setattr(integrations, "get_effective_user_id", lambda: "loop-user")
resolved_cli_path = await asyncio.to_thread(lark_cli._resolve_lark_cli_path)
assert resolved_cli_path is not None
assert Path(resolved_cli_path) == cli_path
await asyncio.to_thread(lark_cli.ensure_lark_cli_credential_tree, "loop-user")
config_dir = await asyncio.to_thread(lark_cli.lark_cli_config_dir, "loop-user")
await asyncio.to_thread(
(config_dir / "config.json").write_text,
'{"apps":[{"appId":"test-app","appSecret":"test-secret"}]}',
encoding="utf-8",
)
generation = await asyncio.to_thread(_advance_lark_flow, "loop-user")
response = await integrations.complete_lark_browser_auth(
@ -134,3 +165,6 @@ async def test_lark_auth_complete_route_does_not_block_event_loop(tmp_path: Path
assert response.status.cli.available is True
assert response.status.cli.version == "v1.0.65"
assert response.success is True
assert response.status.auth.status == "authenticated"
assert response.status.auth.user == "Alice"