Files
group_fqcd_jr/tests/unit/service/test_run_query_service.py
T
lzf_0626 6516ccb385 feat: 第二版——接口契约对齐 docs/05,修复静默故障与数据库基线
相对第一版 46fc976 的完整变更。组员迁移对照表见 docs/20。

一、对外契约对齐 docs/05(破坏性,共 4 处,组员需按 docs/20 调整)
1) 配置发布端点改为文档规定的复数资源名:submit→validations、
   approve→reviews(需 body decision)、activate→activations、
   rollback→rollbacks;第一版这 4 个动词式路径 docs/05 从未定义过。
2) 错误码由 8 个笼统码改为 15 个具体语义码(FORBIDDEN→AGENT_PERMISSION_DENIED、
   UNAUTHORIZED→AUTHENTICATION_REQUIRED、CONFLICT→RESOURCE_VERSION_CONFLICT、
   RESOURCE_NOT_FOUND→RUN_NOT_FOUND/SESSION_NOT_FOUND 等),
   输入类错误状态码 400→422。
3) POST /api/v1/agent-runs 与 GET /api/v1/agent-runs/{run_id} 统一为
   {data, meta} 信封(data 内字段名与语义未变)。
4) 错误响应体统一为 {error:{code,message,retryable,field_errors}, meta:{trace_id}},
   不再返回 FastAPI 默认的 {"detail": ...}。

二、数据库基线与约束
新增 39 张表的基线迁移(链根)与联合唯一键纠偏(4 张表、删 8 增 4,幂等收敛);
撤下 config_release 的双人复核 CHECK(应用层已允许自审,审核节点保留,
自审如实写入 reviewer_id);记忆 active key 生成列与唯一键;
activate 开始记录 supersedes_release_id 使版本链可追溯。
docs/00 基线未修改,未重命名或删除任何表与字段。

三、修复会静默出错或无报错的缺陷
- 跑完集成测试后平台会静默失去生效配置:清理只删自己创建的版本,却没有恢复被它
  顶成 superseded 的原生效版本,且审计一并删除因而完全无痕,表现为所有工具被拒
  但没有任何报错。已修清理逻辑并加恢复。
- Worker 单轮异常导致进程退出;记忆抽取调用方的“事务已开始”异常;
  召回缓存丢失 degraded 标记;连接时区未生效导致 created_at/updated_at 差 8 小时;
  .env 与 os.getenv 密钥来源分裂导致“没有可用的已批准模型端点”。
- 记忆信号识别漏判与跨键误命中;SSE 未带 Accept 的协商行为。

四、功能补齐
记忆链路 P1/P2/P3(抽取、受控词表、召回与缓存、生命周期级联及投影事件)、
fin_* 场内交易只读 ORM 层、agent_intent_config 状态流转并在运行期真正生效、
限流(Redis 固定窗口、故障一律放行)、游标校验、trace_id 中间件、
示例业务 Agent fund_query_demo 与一键端到端验证脚本,以及审计/指纹/迁移状态工具。

五、文档与验证
新增 docs/19(业务 Agent 接入实操)、docs/20(第一版迁移指南)与 docs/evidence 证据;
docs/01/02/06/08/09/17 同步实现现状。

验证结果:ruff 通过、mypy 103 文件无错、unit+contract 447 passed、
integration 29 passed、acceptance_check --production 7 PASS、
demo_agent_e2e 9/9 PASS(含失败关闭反证)。
2026-09-10 15:55:54 +08:00

181 lines
6.2 KiB
Python

"""`RunQueryService` 契约测试(全部使用替身,不连数据库)。
关注三条对客户端与安全都重要的语义:
1. 运行不存在与越权访问**返回同一个 404**——不能靠错误码区分"存在但无权";
2. 未成功的运行**不返回任何结果内容**(`result` 为 None),避免未提交结果外泄;
3. 快照里的时间统一以 `...Z` 结尾(UTC),以及 `watch` 的心跳与终态退出行为。
"""
from dataclasses import dataclass, replace
from datetime import datetime
from decimal import Decimal
from types import SimpleNamespace
from typing import Any
import pytest
from app.core.contracts import RequestContext
from app.core.errors import RunNotFoundError
from app.service.run_query_service import RunQueryService, RunSnapshot
CONTEXT = RequestContext(user_id="9001", trace_id="trace-1")
NOW = datetime(2026, 9, 10, 12, 0, 0)
@dataclass
class FakeRun:
run_id: str = "run-1"
trace_id: str = "trace-1"
status: str = "running"
agent_type: str = "customer_service"
session_id: str = "session-1"
error_code: str | None = None
created_at: datetime = NOW
completed_at: datetime | None = None
@dataclass
class FakeMessage:
content: str = "稳健型"
tool_calls: Any = None
intent: str | None = "general"
confidence: Any = None
source_references: Any = None
class FakeRepository:
def __init__(self, rows: tuple[Any, Any] | None) -> None:
self.rows = rows
async def run_result(self, run_id: str, user_id: int) -> tuple[Any, Any] | None:
del run_id, user_id
return self.rows
class FakeSession:
async def __aenter__(self) -> "FakeSession":
return self
async def __aexit__(self, *exc: object) -> bool:
return False
def patch_repository(monkeypatch: pytest.MonkeyPatch, rows: tuple[Any, Any] | None) -> None:
monkeypatch.setattr("app.service.run_query_service.SessionFactory", FakeSession)
monkeypatch.setattr(
"app.service.run_query_service.ConversationRepository",
lambda _session: FakeRepository(rows),
)
async def test_unknown_run_raises_not_found(monkeypatch: pytest.MonkeyPatch) -> None:
"""不存在与越权必须是同一个错误,否则可以据此探测资源是否存在。"""
patch_repository(monkeypatch, None)
with pytest.raises(RunNotFoundError) as excinfo:
await RunQueryService().get("run-x", CONTEXT)
assert excinfo.value.code == "RUN_NOT_FOUND"
assert excinfo.value.status_code == 404
async def test_running_run_does_not_expose_result(monkeypatch: pytest.MonkeyPatch) -> None:
patch_repository(monkeypatch, (FakeRun(status="running"), None))
snapshot = await RunQueryService().get("run-1", CONTEXT)
assert snapshot.status == "running"
assert snapshot.result is None
assert snapshot.completed_at is None
assert snapshot.created_at.endswith("Z")
async def test_succeeded_run_exposes_result_with_string_confidence(
monkeypatch: pytest.MonkeyPatch,
) -> None:
message = FakeMessage(
content="稳健型",
tool_calls={"calls": []},
confidence=Decimal("0.9000"),
source_references=[{"source_type": "memory"}],
)
patch_repository(monkeypatch, (FakeRun(status="succeeded", completed_at=NOW), message))
snapshot = await RunQueryService().get("run-1", CONTEXT)
assert snapshot.result is not None
assert snapshot.result["content"] == "稳健型"
# DECIMAL(5,4) 经驱动回来是 Decimal,接口层统一转字符串避免浮点精度歧义。
assert snapshot.result["confidence"] == "0.9000"
assert snapshot.result["source_references"] == [{"source_type": "memory"}]
assert snapshot.completed_at is not None and snapshot.completed_at.endswith("Z")
async def test_failed_run_does_not_expose_result(monkeypatch: pytest.MonkeyPatch) -> None:
"""即使库里有结果消息,失败运行也不得把它当作成功结果返回。"""
patch_repository(
monkeypatch, (FakeRun(status="failed", error_code="AGENT_INTERNAL_ERROR"), FakeMessage())
)
snapshot = await RunQueryService().get("run-1", CONTEXT)
assert snapshot.result is None
assert snapshot.error_code == "AGENT_INTERNAL_ERROR"
def terminal_snapshot(status: str = "succeeded") -> RunSnapshot:
return RunSnapshot(
run_id="run-1",
trace_id="trace-1",
status=status,
agent_type="customer_service",
session_id="session-1",
result={"content": "ok"},
error_code=None,
created_at="2026-09-10T12:00:00Z",
completed_at="2026-09-10T12:00:01Z",
)
async def test_watch_stops_immediately_on_terminal_snapshot(
monkeypatch: pytest.MonkeyPatch,
) -> None:
"""终态快照只发一次且不再轮询——断流重连也走这条路径。"""
calls = {"count": 0}
async def never_called(self: RunQueryService, run_id: str, context: RequestContext) -> Any:
calls["count"] += 1
raise AssertionError("终态不应继续轮询数据库")
monkeypatch.setattr(RunQueryService, "get", never_called)
initial = terminal_snapshot()
events = [snapshot async for snapshot in RunQueryService().watch(initial, CONTEXT)]
assert events == [initial]
assert calls["count"] == 0
async def test_watch_emits_heartbeat_between_polls(monkeypatch: pytest.MonkeyPatch) -> None:
monkeypatch.setattr(
"app.service.run_query_service.get_settings",
lambda: SimpleNamespace(sse_max_connection_seconds=30, sse_heartbeat_seconds=0),
)
polls = {"count": 0}
async def fake_get(self: RunQueryService, run_id: str, context: RequestContext) -> RunSnapshot:
del self, run_id, context
polls["count"] += 1
return replace(terminal_snapshot(), status="succeeded")
monkeypatch.setattr(RunQueryService, "get", fake_get)
initial = replace(terminal_snapshot(), status="running", result=None, completed_at=None)
events = [snapshot async for snapshot in RunQueryService().watch(initial, CONTEXT)]
# 序列:非终态快照 → 心跳(None) → 轮询得到的终态快照
assert [event is None for event in events] == [False, True, False]
assert events[0] is initial
assert events[2] is not None and events[2].status == "succeeded"
assert polls["count"] == 1