Files
AgentLab/backend/tests/test_run_timing.py
T
ThearchyHelios 03c18c4d78 界面文案整改:口语改书面、术语统一、不再露出实现细节;修掉几处说错的提示;新增文案检查防回潮
一、文案(前端约 1600 处,后端传到界面的约 880 处)
- 术语统一:工作流层面用「运行」(继续运行、重新运行、试运行),节点和工具层面用「执行」,不再用「跑」;
  总时长 / 执行时长 / 等待审批;服务端;助手;Agent;token;工作流;固定版本;模型接入;系统提示;批准 / 驳回;受管级别
- 口语改书面:取不到、没取回来、读不懂、连不上、多半、照样、兜底、揉成这类说法改成「无法 / 未能 / 失败 / 请」;去掉拟人写法
- 不再露出实现细节:界面上不写 checkpoint、同一拍、哈希(凭证处除外),不写改版历史(「以前…」「老数据」);
  报错、校验、门禁、自动修复里裸写的配置键名和枚举值换成界面上的中文标签,由新的 backend/app/engine/labels.py 统一提供;
  发布弹窗里结论句的选项不再是 off / require_citation
- 同一件事只留一种说法:生成阶段文案、出具档位说明、画布锁定提示各合成一份
- 去掉界面上的工业场景示例(MES、ERP、「张工」「王工」)和真实产品名,换成通用示例

二、说错的提示(都有测试证明)
- 数据库测连接失败时的专用提示以前从不出现(按报错文字全等比较,永远比不上):改为按错误类型判断,不再拼出重复的前缀
- 「设置 → 供应商」改指「设置 → 模型接入」;「在设置里建数据源」改指「数据」页
- 缺少 API Key、模型接入已停用的报错不再夹英文,写明去哪里填
- 两处原因说错(「这种引用在后续版本支持」「这一版还不支持」)改成真实原因
- 节点说明里不准确的说法改正:代码节点在 microVM 或系统沙箱中执行;报告撰写、口径卡的说明不再说过头
- 导出的证据 CSV 里两列同名,第三列改为「片段编号」

三、前后端对齐
- 前端按后端原文识别日志、报错的地方,新旧两种写法都认得(历史运行里存的是旧文字),逐条补了新旧原文的核对
- 后端交回模型的原指令放进单独的 for_model 字段,前端在解析响应时去掉,不会显示到界面上

四、检查
- 检查脚本里按原文匹配的断言改成新文案;改完会变成永远成立的反向断言逐条修正,并做了变异验证
- 新增 check-copy(静态扫描前端界面字符串和后端上界面的文案):再出现「跑」「后端」「多半」「令牌」、
  界面上的 Copilot、裸写的配置键名、半角标点等就报错;确需保留的写进 scripts/copy-allow.json 并注明理由
2026-09-30 08:05:15 +08:00

305 lines
13 KiB
Python
Raw Blame History

This file contains ambiguous Unicode characters
This file contains Unicode characters that might be confused with other characters. If you think that this is intentional, you can safely ignore this warning. Use the Escape button to reveal them.
"""耗时口径:经过审批恢复或接着跑的运行,耗时不能只算最后一段。
真实运行 2609b1c7:事件从头到尾约 87 秒,列表和页脚却写 22ms——每次 _drive
都把 started_at 改成"这一段的开始",_finalize 又拿这一段的耗时覆盖 usage.duration_ms。
审批恢复和接着跑正是治理的主路径,所以凡是走过这条路的运行,耗时全是错的。
现在的口径:
- started_at 只在第一次开始时写;
- duration_ms(= active_ms)是各段执行时长之和;
- wall_ms 是从第一次开始到结束的墙钟;wait_ms 是等人审批的总时长;
- 三者随 run.finished / run.failed / run.cancelled 的 timing 一起发出。
"""
from __future__ import annotations
import asyncio
import time
from datetime import datetime
import pytest
from sqlalchemy import select
from app.db.base import SessionLocal
from app.db.models import Run, RunEvent
from app.engine import compiler
from app.engine.context import NodeError
from app.engine.runner import run_manager
from app.engine.schema import NodeType
SEGMENT = 0.3
@pytest.fixture(autouse=True)
async def engine_up():
await run_manager.setup()
yield
await run_manager.shutdown()
def node(nid, ntype, **config):
return {"id": nid, "type": ntype, "position": {"x": 0, "y": 0},
"data": {"label": nid, "config": config}}
def chain(*nodes):
return {"nodes": list(nodes),
"edges": [{"source": a["id"], "target": b["id"]} for a, b in zip(nodes, nodes[1:])]}
@pytest.fixture
def slow_transforms(monkeypatch):
"""id 以 slow 开头的整形节点各卡 SEGMENT 秒:让每一段执行都有可量的长度。"""
original = compiler.RUNNERS[NodeType.TRANSFORM]
async def runner(state, ctx):
if ctx.node.id.startswith("slow"):
await asyncio.sleep(SEGMENT)
return await original(state, ctx)
monkeypatch.setitem(compiler.RUNNERS, NodeType.TRANSFORM, runner)
async def _wait(run_id: str, statuses: tuple[str, ...], timeout: float = 20.0) -> Run:
deadline = time.monotonic() + timeout
while time.monotonic() < deadline:
async with SessionLocal() as session:
row = await session.get(Run, run_id)
if row and row.status in statuses:
return row
await asyncio.sleep(0.03)
raise AssertionError(f"等超时了:{run_id}")
async def _events(run_id: str) -> list[RunEvent]:
async with SessionLocal() as session:
return list((await session.execute(
select(RunEvent).where(RunEvent.run_id == run_id).order_by(RunEvent.seq)
)).scalars())
def _same_instant(a: datetime, b: datetime) -> bool:
return abs((a - b).total_seconds()) < 1e-3
APPROVAL_GRAPH = chain(
node("start", "input", fields=[{"name": "question"}]),
node("slow1", "transform", mode="template", template="第一段"),
node("gate", "human", mode="approve", title="看一眼"),
node("slow2", "transform", mode="template", template="第二段"),
node("out", "output", fields=[{"name": "结果", "value": "{{ nodes.slow2 }}"}]),
)
WAIT = 0.6
async def test_resumed_run_keeps_its_start_and_sums_every_segment(slow_transforms):
run = await run_manager.start(graph=APPROVAL_GRAPH, input_payload={"question": "q"})
paused = await _wait(run.id, ("interrupted", "failed"))
assert paused.status == "interrupted", paused.error
first_start = paused.started_at
assert first_start is not None
await asyncio.sleep(WAIT) # 人去倒了杯水
await run_manager.resume(run.id, {"approved": True})
done = await _wait(run.id, ("succeeded", "failed"))
assert done.status == "succeeded", done.error
assert _same_instant(done.started_at, first_start), \
f"恢复把 started_at 从 {first_start} 改成了 {done.started_at}"
usage = done.usage
# 两段各至少 SEGMENT 秒。只算最后一段的话这里只有一个 SEGMENT
assert usage["duration_ms"] >= 2 * SEGMENT * 1000 * 0.9, usage
assert usage["active_ms"] == usage["duration_ms"]
assert WAIT * 1000 * 0.8 <= usage["wait_ms"] < usage["wall_ms"], usage
assert usage["wall_ms"] >= usage["active_ms"] + usage["wait_ms"] - 50, usage
wall = (done.finished_at - done.started_at).total_seconds() * 1000
assert abs(usage["wall_ms"] - wall) < 100, (usage, wall)
finished = next(e for e in await _events(run.id) if e.type == "run.finished")
assert finished.data["timing"] == {
"wall_ms": usage["wall_ms"], "active_ms": usage["active_ms"], "wait_ms": usage["wait_ms"],
}
async def test_bookkeeping_after_the_pause_is_not_counted_twice(slow_transforms, monkeypatch):
"""停在审批上之后的收尾(写审批记录、发 run.interrupted)不算执行时长。
以前这一段的执行一直计到 _drive 的 finally,等待又从 run.interrupted 算起:收尾那一截两边都算,
active + wait 比墙钟还长。平时只差几毫秒,机器一忙就超出容差(全量并行测试里偶发)。
这里把收尾拖慢 0.4 秒,不靠机器忙不忙碰运气。
"""
original = run_manager._handle_updates
async def slow(run_id, chunk):
hit = await original(run_id, chunk)
if hit:
await asyncio.sleep(0.4) # 审批卡已经发出去了,引擎也不在执行
return hit
monkeypatch.setattr(run_manager, "_handle_updates", slow)
run = await run_manager.start(graph=APPROVAL_GRAPH, input_payload={"question": "q"})
paused = await _wait(run.id, ("interrupted", "failed"))
assert paused.status == "interrupted", paused.error
await asyncio.sleep(WAIT)
await run_manager.resume(run.id, {"approved": True})
done = await _wait(run.id, ("succeeded", "failed"))
assert done.status == "succeeded", done.error
usage = done.usage
# 执行、等待、墙钟严格对得上账:取整各丢不到 1 毫秒
assert usage["wall_ms"] >= usage["active_ms"] + usage["wait_ms"] - 2, usage
# 0.4 秒的收尾没有算进执行(两段各 SEGMENT 秒,其余开销远小于 0.4 秒)
assert usage["active_ms"] < 2 * SEGMENT * 1000 + 400, usage
@pytest.fixture
def fails_once(monkeypatch):
original = compiler.RUNNERS[NodeType.TRANSFORM]
calls = {"n": 0}
async def runner(state, ctx):
if ctx.node.id.startswith("slow"):
await asyncio.sleep(SEGMENT)
if ctx.node.id == "flaky":
calls["n"] += 1
if calls["n"] == 1:
raise NodeError(ctx.node.id, "第一次故意失败")
return await original(state, ctx)
monkeypatch.setitem(compiler.RUNNERS, NodeType.TRANSFORM, runner)
return calls
async def test_continued_run_keeps_its_start_and_sums_every_segment(fails_once):
graph = chain(
node("start", "input", fields=[{"name": "question"}]),
node("slow1", "transform", mode="template", template="前面这步"),
node("flaky", "transform", mode="template", template="会挂一次"),
node("out", "output", fields=[{"name": "结果", "value": "{{ nodes.flaky }}"}]),
)
run = await run_manager.start(graph=graph, input_payload={"question": "q"})
failed = await _wait(run.id, ("failed", "succeeded"))
assert failed.status == "failed"
first_start = failed.started_at
assert failed.usage["duration_ms"] >= SEGMENT * 1000 * 0.9
failed_ev = next(e for e in await _events(run.id) if e.type == "run.failed")
assert set(failed_ev.data["timing"]) == {"wall_ms", "active_ms", "wait_ms"}
assert failed_ev.data["node_id"] == "flaky"
await run_manager.continue_failed(run.id)
done = await _wait(run.id, ("succeeded", "failed"))
assert done.status == "succeeded", done.error
assert _same_instant(done.started_at, first_start)
# 第二段只跑了 flaky 和 out,很快。只算最后一段的话会远小于 SEGMENT
assert done.usage["duration_ms"] >= SEGMENT * 1000 * 0.9, done.usage
# 失败到接着跑之间不是在等审批
assert done.usage["wait_ms"] == 0, done.usage
async def test_cancel_reports_timing_too(slow_transforms):
graph = chain(
node("start", "input", fields=[{"name": "question"}]),
node("slow1", "transform", mode="template", template="x"),
node("slow2", "transform", mode="template", template="y"),
)
run = await run_manager.start(graph=graph, input_payload={"question": "q"})
await asyncio.sleep(SEGMENT / 2)
assert await run_manager.cancel(run.id)
await _wait(run.id, ("cancelled",))
cancelled = next(e for e in await _events(run.id) if e.type == "run.cancelled")
assert set(cancelled.data["timing"]) == {"wall_ms", "active_ms", "wait_ms"}
assert cancelled.data["timing"]["active_ms"] > 0
async def test_long_output_says_it_was_clipped():
"""事件里的 output 为了控制体积会截断——截了就得说,消费方才知道要回源取全文。"""
long_text = "很长的结论。" * 1500
graph = chain(
node("start", "input", fields=[{"name": "question"}]),
node("out", "output", fields=[{"name": "answer", "value": "{{ input.question }}"}]),
)
run = await run_manager.start(graph=graph, input_payload={"question": long_text})
done = await _wait(run.id, ("succeeded", "failed"))
assert done.status == "succeeded", done.error
finished = next(e for e in await _events(run.id) if e.type == "run.finished")
assert finished.data.get("output_truncated") is True
assert len(finished.data["output"]["answer"]) < len(long_text)
assert done.output["answer"] == long_text, "库里的是完整版本"
short = await run_manager.start(graph=graph, input_payload={"question": "短的"})
await _wait(short.id, ("succeeded",))
finished = next(e for e in await _events(short.id) if e.type == "run.finished")
assert not finished.data.get("output_truncated")
async def test_event_ts_is_when_it_happened_not_when_it_was_forwarded():
async with SessionLocal() as session:
run = Run(workflow_name="ts", status="running", graph={}, input={})
session.add(run)
await session.commit()
run_id = run.id
produced_at = time.time() - 5
await run_manager._handle_custom(run_id, {
"__agentlab__": True, "type": "log", "node_id": "x", "ts": produced_at,
"data": {"message": "早先发生的事"},
})
[event] = await _events(run_id)
assert abs(event.ts - produced_at) < 1e-6
async def test_node_started_says_which_iteration_and_whether_it_was_a_replay(slow_transforms):
graph = {
"nodes": [
node("start", "input", fields=[{"name": "question"}]),
node("lp", "loop", mode="foreach", items='["a", "b", "c"]', max_iterations=5),
node("body", "transform", mode="template", template="{{ vars.item }}"),
node("gate", "human", mode="approve", title="收尾前看一眼"),
node("out", "output", fields=[{"name": "结果", "value": "ok"}]),
],
"edges": [
{"source": "start", "target": "lp"},
{"source": "lp", "target": "body", "sourceHandle": "body"},
{"source": "body", "target": "lp"},
{"source": "lp", "target": "gate", "sourceHandle": "done"},
{"source": "gate", "target": "out"},
],
}
run = await run_manager.start(graph=graph, input_payload={"question": "q"})
await _wait(run.id, ("interrupted",))
await run_manager.resume(run.id, {"approved": True})
await _wait(run.id, ("succeeded",))
events = await _events(run.id)
body = [e.data.get("iteration") for e in events
if e.type == "node.started" and e.node_id == "body"]
assert body == [1, 2, 3]
gate = [e.data for e in events if e.type == "node.started" and e.node_id == "gate"]
assert len(gate) == 2
assert not gate[0].get("resumed")
assert gate[1].get("resumed") is True, "恢复后重放的那一次要标出来"
out = [e.data for e in events if e.type == "node.started" and e.node_id == "out"]
assert not out[0].get("resumed"), "恢复之后新走到的节点不是重放"
async def test_node_skipped_says_which_node_by_name_not_just_id():
"""被 skip_if 跳过的节点也得带上名称和类型:界面按名字写「跳过「整理结果」」,
以前只带 reason,右栏只能退回写节点 id。"""
graph = chain(
node("start", "input", fields=[{"name": "question"}]),
{"id": "bg", "type": "transform", "position": {"x": 0, "y": 0},
"data": {"label": "整理结果",
"config": {"mode": "template", "template": "x", "skip_if": "true"}}},
node("out", "output", fields=[{"name": "结果", "value": "ok"}]),
)
run = await run_manager.start(graph=graph, input_payload={"question": "q"})
done = await _wait(run.id, ("succeeded", "failed"))
assert done.status == "succeeded", done.error
[skipped] = [e for e in await _events(run.id) if e.type == "node.skipped"]
assert skipped.node_id == "bg"
assert skipped.data.get("label") == "整理结果"
assert skipped.data.get("node_type") == "transform"
assert "满足跳过条件" in skipped.data.get("reason", "")