Files
Octop/tests/unit/providers/test_codex_oauth.py
347dee56f4 refactor(infra): group agents modules and clarify package ownership (#1164)
* refactor(agents): group thin modules into domain subpackages

Move plugin helpers, memory, settings stores, persona/MBTI, and avatar
bootstrap code under plugins/, memory/, settings/, persona/, and experts/
while keeping top-level import shims so existing call sites stay stable.

Co-authored-by: Cursor <cursoragent@cursor.com>

* refactor(agents): drop compatibility shims after subpackage move

Call sites already import the new package paths; remove the top-level
re-export stubs so the agents tree only shows real modules.

Co-authored-by: Cursor <cursoragent@cursor.com>

* refactor(agents): move thread helpers into threads/

Group artifact shaping, fork, and context breakdown under
agents/threads so top-level only keeps lifecycle/config adapters.

Co-authored-by: Cursor <cursoragent@cursor.com>

* refactor(agents): fold profile/runtime/tool catalog into settings/

Move the remaining config-surface helpers under settings/ and drop the
skill_package id-list aliases in favor of dump/parse_id_list.

Co-authored-by: Cursor <cursoragent@cursor.com>

* refactor(agents): group workspace_dir and execute_env under workspace/

Keep path resolution and harness execute-env injection together as one
workspace package; leave only manager and conversation_mode at top level.

Co-authored-by: Cursor <cursoragent@cursor.com>

* refactor(infra): fix history/SkillHub ownership and drop thin providers/

Move history_projection under history/, share SkillHub host/limits and
route expert skill downloads through skills.skillhub_market, and fold
Codex OAuth helpers into agents.providers to remove the tiny providers/
package.

Co-authored-by: Cursor <cursoragent@cursor.com>

* docs: sync AGENTS.md and guides with infra/agents layout

Reflect subpackages (settings/workspace/threads/persona, history/, skills/)
and drop stale runtime.py / mbti_profiles paths after the agents reorg.

Co-authored-by: Cursor <cursoragent@cursor.com>

---------

Co-authored-by: jubaoliang <jubaoliang@tencent.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
2026-09-25 22:32:41 +08:00

133 lines
4.2 KiB
Python

"""Unit tests for Codex OAuth helpers."""
from __future__ import annotations
import json
import urllib.error
import urllib.request
from unittest import mock
from urllib.parse import urlparse
import pytest
from octop.infra.agents.providers.codex_apply import CODEX_MODELS, CODEX_PROVIDER_NAME
from octop.infra.agents.providers.codex_oauth import (
CodexOAuthDeviceCodeError,
build_codex_headers,
exchange_device_code,
poll_device_token,
request_device_code,
)
class _Resp:
def __init__(self, payload: bytes) -> None:
self._payload = payload
def read(self) -> bytes:
return self._payload
def __enter__(self) -> _Resp:
return self
def __exit__(self, *args: object) -> None:
return None
def test_request_device_code_parses_response() -> None:
payload = json.dumps(
{"device_auth_id": "dev-1", "user_code": "ABCD-1234", "interval": 5}
).encode()
def fake_urlopen(req: urllib.request.Request, timeout: float = 30) -> _Resp:
assert "deviceauth/usercode" in req.full_url
return _Resp(payload)
with mock.patch("urllib.request.urlopen", fake_urlopen):
info = request_device_code()
assert info["device_auth_id"] == "dev-1"
assert info["user_code"] == "ABCD-1234"
assert info["interval_s"] == 5
assert urlparse(info["verification_url"]).hostname == "auth.openai.com"
def test_request_device_code_normalizes_network_failures() -> None:
def fake_urlopen(req: urllib.request.Request, timeout: float = 30) -> _Resp:
raise urllib.error.URLError("DNS lookup failed")
with (
mock.patch("urllib.request.urlopen", fake_urlopen),
pytest.raises(CodexOAuthDeviceCodeError) as raised,
):
request_device_code()
assert raised.value.reason == "network_error"
assert raised.value.upstream_status is None
assert "DNS lookup failed" not in str(raised.value)
def test_request_device_code_preserves_upstream_http_status() -> None:
def fake_urlopen(req: urllib.request.Request, timeout: float = 30) -> _Resp:
raise urllib.error.HTTPError(req.full_url, 503, "unavailable", {}, None) # type: ignore[arg-type]
with (
mock.patch("urllib.request.urlopen", fake_urlopen),
pytest.raises(CodexOAuthDeviceCodeError) as raised,
):
request_device_code()
assert raised.value.reason == "upstream_http_error"
assert raised.value.upstream_status == 503
def test_poll_device_token_returns_none_while_pending() -> None:
def fake_urlopen(req: urllib.request.Request, timeout: float = 30) -> _Resp:
raise urllib.error.HTTPError(req.full_url, 403, "pending", {}, None) # type: ignore[arg-type]
with mock.patch("urllib.request.urlopen", fake_urlopen):
result = poll_device_token("dev-1", "ABCD-1234")
assert result is None
def test_poll_device_token_returns_code_on_success() -> None:
payload = json.dumps({"authorization_code": "auth-code", "code_verifier": "verifier"}).encode()
def fake_urlopen(req: urllib.request.Request, timeout: float = 30) -> _Resp:
assert "deviceauth/token" in req.full_url
return _Resp(payload)
with mock.patch("urllib.request.urlopen", fake_urlopen):
result = poll_device_token("dev-1", "ABCD-1234")
assert result == ("auth-code", "verifier")
def test_exchange_device_code_builds_credentials() -> None:
payload = json.dumps(
{"access_token": "access-tok", "refresh_token": "refresh-tok", "expires_in": 3600}
).encode()
def fake_urlopen(req: urllib.request.Request, timeout: float = 30) -> _Resp:
assert req.full_url.endswith("/oauth/token")
return _Resp(payload)
with mock.patch("urllib.request.urlopen", fake_urlopen):
cred = exchange_device_code("auth-code", "verifier")
assert cred["access"] == "access-tok"
assert cred["refresh"] == "refresh-tok"
def test_build_codex_headers_includes_account_id() -> None:
headers = build_codex_headers("acct-123")
assert headers["originator"] == "openclaw"
assert headers["chatgpt-account-id"] == "acct-123"
assert "User-Agent" in headers
def test_codex_provider_constants() -> None:
assert CODEX_PROVIDER_NAME == "openai-codex"
assert any(m["id"] == "gpt-5.4" for m in CODEX_MODELS)