# PR #22571 完整报告

- 仓库：`sgl-project/sglang`
- 标题：Remove redundant test_page_size.py
- 合并时间：2026-04-11 11:35
- 原文链接：http://prhub.com.cn/sgl-project/sglang/pull/22571

---

## 执行摘要
此 PR 删除了冗余的测试文件 `test/registered/core/test_page_size.py`，该文件仅使用 `--page-size 4 --chunked-prefill-size 128` 参数运行 MMLU 基准测试，其功能已被 `test_eagle_infer_b.py::TestEAGLEServerPageSize` 完全覆盖。这是对测试套件的清理优化，旨在消除重复测试、简化维护并提升 CI 效率，风险较低但需注意测试覆盖完整性。

## 功能与动机
移除冗余测试文件以简化测试套件。根据 PR body，`test_page_size.py` 仅运行 MMLU 测试，使用参数 `--page-size 4 --chunked-prefill-size 128`，而相同的测试场景已在 `test_eagle_infer_b.py::TestEAGLEServerPageSize` 中实现，后者提供了更全面的测试覆盖。因此，删除此文件可减少维护开销和 CI 执行时间。

## 实现拆解
- **删除文件**：`test/registered/core/test_page_size.py`，该文件包含测试类 `TestPageSize`，其关键逻辑如下：
  ```python
  class TestPageSize(CustomTestCase, MMLUMixin):
      @classmethod
      def setUpClass(cls):
          os.environ["SGLANG_DEBUG_MEMORY_POOL"] = "1"
          cls.process = popen_launch_server(
              cls.model,
              cls.base_url,
              other_args=["--page-size", 4, "--chunked-prefill-size", 128],
          )
  ```
- **无新增代码**：仅删除操作，未修改其他文件。

## 评论区精华
无 review 评论，讨论仅限于 PR body 中的简要说明和测试计划验证。

## 风险与影响
- **风险**：
 1. 测试覆盖缺口：如果 `TestEAGLEServerPageSize` 未完全复制 `test_page_size.py` 的所有设置（如环境变量 `SGLANG_DEBUG_MEMORY_POOL=1`），可能导致特定场景测试丢失。
 2. CI 影响：删除的文件注册了 CUDA 和 AMD CI 的 `est_time` 值（77 秒和 60 秒），可能影响 CI 负载均衡算法，需确认相关配置是否已调整。
- **影响**：对用户和系统无直接影响，仅简化测试维护；可能减少 CI 执行时间，提升团队效率。

## 关联脉络
- 与近期 PR #22565 和 #22557 相关，它们都涉及 CI 测试时间估算（`est_time`）优化。本 PR 删除的测试文件包含 `est_time` 注册，在清理冗余测试时需考虑对 CI 负载均衡的潜在影响。
- 这反映了仓库在持续优化测试套件，通过删除重复测试来提高 CI 效率和可维护性，是测试基础设施演进的一部分。