[https://nvbugs/6061812][fix] Unblock ruff check

[VALLIS-NERIA]

Summary by CodeRabbit

• Tests
  • Streamlined test documentation formatting for improved readability.

Description

Test Coverage

PR Checklist

Please review the following before submitting your PR:

• PR description clearly explains what and why. If using CodeRabbit's summary, please make sure it makes sense.

• PR Follows TRT-LLM CODING GUIDELINES to the best of your knowledge.

• Test cases are provided for new code paths (see test instructions)

• Any new dependencies have been scanned for license and vulnerabilities

• CODEOWNERS updated if ownership changes

• Documentation updated as needed

• Update tava architecture diagram if there is a significant design change in PR.

• The reviewers assigned automatically/manually are appropriate for the PR.

• Please check this after reviewing the above items as appropriate for this PR.

GitHub Bot Help

To see a list of available CI bot commands, please comment /bot help.

[VALLIS-NERIA]

/bot skip --comment "comments only, run release-check"

[coderabbitai]

📝 Walkthrough

Walkthrough

Test file docstrings in test_executor.py are reformatted, collapsing multi-line explanatory strings into single-line sentences for four test methods. No functional changes or logic modifications are included.

Changes

Cohort / File(s)	Summary
Test Docstring Formatting tests/unittest/llmapi/test_executor.py	Reformatted docstrings for four test methods (test_abort_on_DetokenizedGenerationResultBase, test_PostprocWorker_Output_should_abort, test_handle_response_propagates_should_abort, test_get_params_for_first_rsp_returns_disaggregated_params_once) by collapsing multi-line descriptions into single-line sentences with equivalent meaning.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The PR description is entirely a template with no actual content filled in. All substantive sections (Description, Test Coverage) are empty.	Fill in the Description section explaining the changes made to test docstrings. Complete Test Coverage section listing affected tests. Verify PR Checklist items match the minimal nature of docstring formatting changes.
Title check	⚠️ Warning	The title references fixing a ruff check issue, but the changeset only contains trivial docstring reformatting with no actual fixes to ruff checks or code logic.	Update the title to accurately reflect the actual change: reformatting test docstrings to be single-line, such as 'Reformat test docstrings to single-line format' or 'Clean up test docstring formatting'.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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

[tensorrt-cicd]

PR_Github #43019 [ skip ] triggered by Bot. Commit: 2184c26 Link to invocation

[mzweilz]

LGTM

[tensorrt-cicd]

PR_Github #43019 [ skip ] completed with state SUCCESS. Commit: 2184c26
Skipping testing for commit 2184c26

Link to invocation