wehub-resource-sync
101 lines
3.8 KiB
Python
101 lines
3.8 KiB
Python
"""Tests for ``omnigent/inner/copilot_harness.py`` — the ``harness: copilot`` wrap.
|
|
|
|
The wrap reads ``HARNESS_COPILOT_*`` env vars and constructs a
|
|
:class:`CopilotExecutor` lazily. Constructing the executor does NOT import the
|
|
``github-copilot-sdk`` package (that import is deferred to the first turn), so
|
|
these tests run without the optional SDK installed. Mirrors
|
|
``test_cursor_harness.py``.
|
|
"""
|
|
|
|
from __future__ import annotations
|
|
|
|
import json
|
|
|
|
import pytest
|
|
|
|
from omnigent.inner import copilot_harness as ch
|
|
from omnigent.inner.copilot_executor import CopilotExecutor
|
|
|
|
|
|
@pytest.fixture(autouse=True)
|
|
def _clear_harness_env(monkeypatch: pytest.MonkeyPatch) -> None:
|
|
for var in (
|
|
"HARNESS_COPILOT_MODEL",
|
|
"HARNESS_COPILOT_CWD",
|
|
"HARNESS_COPILOT_GITHUB_TOKEN",
|
|
"HARNESS_COPILOT_OS_ENV",
|
|
"HARNESS_COPILOT_SKILLS_FILTER",
|
|
"HARNESS_COPILOT_BUNDLE_DIR",
|
|
"HARNESS_COPILOT_AGENT_NAME",
|
|
"COPILOT_GITHUB_TOKEN",
|
|
"GH_TOKEN",
|
|
"GITHUB_TOKEN",
|
|
):
|
|
monkeypatch.delenv(var, raising=False)
|
|
|
|
|
|
def test_resolve_os_env_default() -> None:
|
|
os_env = ch._resolve_os_env()
|
|
assert os_env.type == "caller_process"
|
|
assert os_env.sandbox is not None and os_env.sandbox.type == "none"
|
|
|
|
|
|
def test_resolve_os_env_from_json(monkeypatch: pytest.MonkeyPatch) -> None:
|
|
monkeypatch.setenv(
|
|
"HARNESS_COPILOT_OS_ENV",
|
|
json.dumps({"type": "caller_process", "cwd": "/tmp/x", "sandbox": {"type": "none"}}),
|
|
)
|
|
os_env = ch._resolve_os_env()
|
|
assert os_env.cwd == "/tmp/x"
|
|
|
|
|
|
def test_resolve_skills_filter_defaults_all(monkeypatch: pytest.MonkeyPatch) -> None:
|
|
assert ch._resolve_skills_filter() == "all"
|
|
monkeypatch.setenv("HARNESS_COPILOT_SKILLS_FILTER", json.dumps(["a", "b"]))
|
|
assert ch._resolve_skills_filter() == ["a", "b"]
|
|
monkeypatch.setenv("HARNESS_COPILOT_SKILLS_FILTER", "not-json")
|
|
assert ch._resolve_skills_filter() == "all"
|
|
|
|
|
|
def test_build_executor_threads_env(monkeypatch: pytest.MonkeyPatch) -> None:
|
|
monkeypatch.setenv("HARNESS_COPILOT_MODEL", "claude-haiku-4.5")
|
|
monkeypatch.setenv("HARNESS_COPILOT_GITHUB_TOKEN", "gho_test")
|
|
monkeypatch.setenv("HARNESS_COPILOT_CWD", "/tmp/work")
|
|
monkeypatch.setenv("HARNESS_COPILOT_AGENT_NAME", "cop")
|
|
executor = ch._build_copilot_executor()
|
|
assert isinstance(executor, CopilotExecutor)
|
|
assert executor._model_override == "claude-haiku-4.5"
|
|
assert executor._github_token == "gho_test"
|
|
assert executor._cwd == "/tmp/work"
|
|
assert executor._agent_name == "cop"
|
|
|
|
|
|
def test_build_executor_threads_os_env_bundle_dir_and_ambient_token(
|
|
monkeypatch: pytest.MonkeyPatch,
|
|
) -> None:
|
|
from pathlib import Path
|
|
|
|
# No HARNESS_COPILOT_CWD / GITHUB_TOKEN: cwd falls back to os_env.cwd and the
|
|
# token falls back to an ambient GH_TOKEN.
|
|
monkeypatch.setenv(
|
|
"HARNESS_COPILOT_OS_ENV",
|
|
json.dumps({"type": "caller_process", "cwd": "/tmp/oe", "sandbox": {"type": "none"}}),
|
|
)
|
|
monkeypatch.setenv("HARNESS_COPILOT_BUNDLE_DIR", "/tmp/bundle")
|
|
monkeypatch.setenv("GH_TOKEN", "gho_ambient")
|
|
executor = ch._build_copilot_executor()
|
|
assert isinstance(executor, CopilotExecutor)
|
|
assert executor._os_env_spec is not None and executor._os_env_spec.cwd == "/tmp/oe"
|
|
assert executor._cwd == "/tmp/oe" # cwd fell back to os_env.cwd
|
|
assert executor._bundle_dir == Path("/tmp/bundle")
|
|
assert executor._github_token == "gho_ambient" # ambient fallback
|
|
|
|
|
|
def test_create_app_exposes_executor_adapter_routes() -> None:
|
|
app = ch.create_app()
|
|
paths = {getattr(r, "path", "") for r in app.routes}
|
|
# The ExecutorAdapter wires the harness HTTP contract — assert the real
|
|
# endpoints exist, not merely that the object has a ``routes`` attribute.
|
|
assert "/health" in paths
|
|
assert any("/v1/sessions/" in p and p.endswith("/events") for p in paths)
|