# PR #48220 完整报告

- 仓库：`vllm-project/vllm`
- 标题：[Misc] Remove dead code in ViT functionality test
- 合并时间：2026-07-10 19:17
- 原文链接：http://prhub.com.cn/vllm-project/vllm/pull/48220

---

## 执行摘要

PR #48220 移除了一个始终被跳过的 ViT 后端功能测试文件 `tests/models/multimodal/generation/test_vit_backend_functionality.py`，因为已有 `test_vit_cudagraph.py` 提供了更全面的覆盖。这是一个低风险的代码清理，无功能影响。

## 功能与动机

PR 作者在描述中说明该测试文件 "always skipped"，且已有 `test_vit_cudagraph.py` 覆盖了 encoder CG 的相关测试。删除死代码可以减少维护成本，避免误导开发者。

## 实现拆解

1. **删除测试文件**：完全移除 `tests/models/multimodal/generation/test_vit_backend_functionality.py`。该文件约 438 行，包含：
 - 多个模型配置（dots_ocr、ernie45_vl、glm4_1v、glm_ocr、keye_vl、ovis2_5、qwen2_5_vl）
 - Prompt 构建函数（build_dots_ocr_prompt、build_processor_prompt、build_ovis_prompt、build_qwen2_5_video_prompt）
 - 通用测试运行函数（run_llm_generate_test、run_llm_chat_test、run_video_test）
 - 主测试函数 test_vit_backend_functionality（被 @pytest.mark.skip 跳过）

2. **无其他文件变更**：该文件未被外部导入，删除无需调整其他模块。

### 文件完全删除，无遗留代码。其核心内容已由 `tests/models/multimodal/generation/test_vit_cudagraph.py` 覆盖。

## 评论区精华

无有效讨论。review 直接由 DarkLight1337 批准。

## 风险与影响

- **风险**：极低。被删文件从未执行，且无外部依赖。
- **影响**：减少代码量，提升测试套件清洁度，对用户无影响。

## 关联脉络

PR 源于对测试覆盖的审计，与本仓库近期其他清理工作（如 #43896 修复模型别名）一致，体现了维护代码健康的常规活动。