执行摘要
此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中的简要说明和测试计划验证。
风险与影响
- 风险:
- 测试覆盖缺口:如果
TestEAGLEServerPageSize未完全复制test_page_size.py的所有设置(如环境变量SGLANG_DEBUG_MEMORY_POOL=1),可能导致特定场景测试丢失。
- CI影响:删除的文件注册了CUDA和AMD CI的
est_time值(77秒和60秒),可能影响CI负载均衡算法,需确认相关配置是否已调整。
- 影响:对用户和系统无直接影响,仅简化测试维护;可能减少CI执行时间,提升团队效率。
关联脉络
- 与近期PR #22565和#22557相关,它们都涉及CI测试时间估算(
est_time)优化。本PR删除的测试文件包含est_time注册,在清理冗余测试时需考虑对CI负载均衡的潜在影响。
- 这反映了仓库在持续优化测试套件,通过删除重复测试来提高CI效率和可维护性,是测试基础设施演进的一部分。
参与讨论