执行摘要

本次 PR 将工具解析器的纯单元测试从 tests/entrypoints/openai/tool_parsers/ 移动至 tests/tool_parsers/ 目录，分离单元测试与集成测试，提升测试组织性。变更涉及文件移动和拆分，对业务逻辑无直接影响，但 review 中暴露出一个测试逻辑缺陷，值得后续关注。

功能与动机

PR body 明确说明动机是“让 CPU-only 工具解析器测试在 tests/tool_parsers/ 下运行（在 misc.yaml 中运行，无需 GPU）”，而集成测试保留在原位置使用 RemoteOpenAIServer。这旨在改进测试结构，使测试更清晰、可维护，并优化 CI 流程。

实现拆解

实现主要包括以下步骤：

• 移动纯单元测试文件：使用 git mv 将 6 个文件（如 pythonic、llama3_json 等）移至 tests/tool_parsers/。
• 拆分混合测试文件：从 test_hermes_tool_parser.py 和 test_granite4_tool_parser.py 中提取单元测试部分，分别创建新文件 tests/tool_parsers/test_hermes_tool_parser.py 和 tests/tool_parsers/test_granite4_tool_parser.py。
• 添加测试隔离：在 tests/tool_parsers/test_gigachat3_tool_parser.py 中添加函数作用域的 default_tokenizer 覆盖，防止 tokenizer 状态泄漏：
  

  @pytest.fixture(scope="function")
  def default_tokenizer() -> TokenizerLike:
      return AutoTokenizer.from_pretrained("gpt2")
  

• 删除冗余配置：移除 tests/entrypoints/openai/tool_parsers/conftest.py，因为不再需要。

评论区精华

review 中，gemini-code-assist[bot] 指出一个关键问题：

“The streaming test for Granite4ToolParser does not correctly simulate a streaming scenario. The previous_text and current_text arguments to extract_tool_calls_streaming are hardcoded to empty strings.”
作者 bbrowning 回复强调 PR 范围仅为移动现有逻辑，而非修改测试。讨论未解决测试逻辑缺陷，但 PR 已获批准合并。

风险与影响

风险：

• 测试逻辑缺陷：test_granite4_tool_parser.py 中的流式测试可能无法正确模拟流式行为，若未来解析器使用 previous_text 和 current_text 参数，测试可能漏检回归。
• 文件移动风险：文件重定位可能导致导入路径错误或 CI 配置问题，但 PR 测试通过（490 个单元测试通过），风险较低。

影响：

• 对用户无直接影响，因为是内部测试重构。
• 对团队：提升测试组织性，单元测试更集中，便于维护和独立运行（CPU-only），可能提高 CI 效率。

关联脉络

从同仓库近期历史 PR 分析，本次 PR 是独立的测试重构，与其他 PR（如 bugfix 或性能优化）无直接关联。它反映了仓库在整理测试基础设施方面的持续努力，可能为后续工具解析器功能演进提供更清晰的测试基础。