[https://nvbugs/6418006][fix] Mirror the test_medusa pattern from commit b08ce26ac1 — replace test_whisper's…

[trtllm-agent]

Summary

• Root cause: PR [TRTLLM-13781][chore] Remove legacy TensorRT examples #15763 deleted examples/models/core/whisper/, but build_model.sh's whisper branch still pushd's into it and invokes convert_checkpoint.py, aborting with CalledProcessError under set -e.
• Fix: Mirror the test_medusa pattern from commit b08ce26 — replace test_whisper's body with an assertion that the removed convert_checkpoint.py has not been reintroduced. Keeps pytest node id resolvable while removing the broken build/test.sh invocation.
• Automated fix generated by repair-bot

Test plan

• Verify fix on the same GPU type as the original failure
• Check for regressions in related tests

Links

• Bug: https://nvbugs/6418006

Summary by CodeRabbit

• Tests
  • Updated Whisper integration coverage to validate the expected file layout and fail clearly if a removed script reappears.
  • Simplified the test setup by removing unused build and execution steps from this check.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 450bed07-1653-4bc7-9bea-c8a080ed1474

📥 Commits

Reviewing files that changed from the base of the PR and between a0c406f and b325280.

📒 Files selected for processing (1)

• tests/integration/defs/triton_server/test_triton.py

---

📝 Walkthrough

Walkthrough

The test_whisper function in the Triton server integration tests was rewritten to remove model building and execution logic, replacing it with an assertion verifying that convert_checkpoint.py does not exist under llm_root. The function signature was simplified accordingly.

Changes

Whisper Test Change

Layer / File(s)	Summary
Guard assertion replaces build/run logic tests/integration/defs/triton_server/test_triton.py	test_whisper no longer builds the model or runs test.sh; it now asserts convert_checkpoint.py is absent under llm_root, and the function signature drops model_path and engine_dir parameters.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Poem

A rabbit hops through test files old,
Removes the build, the scripts once bold,
Now checks a path that should be bare,
No convert script waiting there,
A tidy hop, a simpler tale to tell.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title uses a valid bug tag and fix type and clearly points to the Whisper test replacement.
Description check	✅ Passed	The description includes Summary, Test plan, and Links, and it explains the issue and fix clearly.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

_{Comment @coderabbitai help to get the list of available commands.}