Prhub

#3195 test(agentic): run the sandbox tests on any provider, not just e2b

原始 PR 作者 nblintao 合并时间 2026-09-12 22:55 文件变更 5 提交数 2 评论 3 代码增减 +195 / -68

执行摘要

沙箱测试改由凭据注册表驱动,支持任意 provider

实现拆解

  1. 核心源码路径
    - miles/rollout/agentic/credentials.py(core-logic):源码主路径;涉及符号 _key_file_has_value, credential_available;包含 控制流调整、配置键调整、异常路径调整;+19/-4;关键符号:_key_file_has_value, credential_available
    - scripts/sandbox_smoke/run.py(core-logic):源码主路径;涉及符号 main;包含 配置键调整、符号定义调整;+2/-2;关键符号:main

  2. 测试与验证
    - tests/e2e/agentic/test_harbor_rollout.py(rename-or-move):测试配套;涉及符号 preflight, harbor_worker_env, probe_e2b_endpoint, execute;包含 文件搬迁、测试覆盖调整、导入关系调整;+76/-62;关键符号:preflight, harbor_worker_env, probe_e2b_endpoint, execute
    - tests/e2e/agentic/test_sandbox_golden.py(test-coverage):测试配套;涉及符号 smoke, test_golden_episode;包含 测试覆盖调整、导入关系调整、控制流调整;+68/-0;关键符号:smoke, test_golden_episode
    - tests/fast/rollout/agentic/test_credentials.py(test-coverage):测试配套;涉及符号 test_credential_available_accepts_either_supply, test_credential_available_needs_every_var_of_a_multi_var_provider;包含 测试覆盖调整、配置键调整、符号定义调整;+30/-0;关键符号:test_credential_available_accepts_either_supply, test_credential_available_needs_every_var_of_a_multi_var_provider

文件 模块 状态 重要度
tests/e2e/agentic/test_harbor_rollout.py harbor/rollout renamed 7.87
miles/rollout/agentic/credentials.py miles/rollout modified 7.19
tests/e2e/agentic/test_sandbox_golden.py sandbox/golden added 6.85
tests/fast/rollout/agentic/test_credentials.py credentials modified 6.08
scripts/sandbox_smoke/run.py scripts/sandbox_smoke modified 5.77

关键符号

preflight harbor_worker_env probe_e2b_endpoint execute _key_file_has_value credential_available smoke test_golden_episode test_credential_available_accepts_either_supply test_credential_available_needs_every_var_of_a_multi_var_provider

分析完成后,这里会展示 LLM 生成的相对完整源码片段和详细注释。

评论区精华

没有提炼出高价值讨论线程

当前评论区没有形成足够清晰的争议点或结论,后续有更多讨论时会体现在这里。

风险与影响

关联 Issue

未识别关联 Issue

当前没有检测到明确关联的 Issue 链接,后续同步到相关引用后会出现在这里。

完整报告

参与讨论