fix: remove dead callable() check in parametrize test

[andrewwhitecdw]

fix: remove dead callable() check in parametrize test

Root cause

The test asserted assert callable(mutate) even though every parametrized case supplies a lambda. The assertion added no value and was flagged as dead code.

Fix

Drop the assert and call mutate(request) directly. All 18 parametrized cases still pass.

-    assert callable(mutate)
     mutate(request)

Testing

• pytest tests/test_client.py::test_call_rejects_unsupported_normalized_fields -v: 18 passed
• ruff check .: All checks passed

Contributor guidelines

• DCO sign-off: included.
• Squash: one commit.

Summary by CodeRabbit

• Tests
  • Removed a redundant test assertion from the unsupported-fields test case.

[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: 4bd2cf06-3727-432f-8162-cf15749a50d8

📥 Commits

Reviewing files that changed from the base of the PR and between fb3fc30 and 5c29202.

📒 Files selected for processing (1)

• examples/experimental/litellm/tests/test_client.py

💤 Files with no reviewable changes (1)

• examples/experimental/litellm/tests/test_client.py

---

Walkthrough

The unsupported-fields parameterized test now applies mutate directly. It no longer performs a redundant callable assertion.

Changes

Test Assertion Cleanup

Layer / File(s)	Summary
Direct mutation application examples/experimental/litellm/tests/test_client.py	The test removes the preliminary callable(mutate) assertion and applies the mutation directly.

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

Poem

I nibbled one assertion away,
And left the test direct today.
The mutation runs with less delay,
While carrots cheer the cleaner way.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and concisely describes the removal of the redundant callable() check from the parametrized test.
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.

---

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

[ayushag-nv]

I think there is no need for this PR. It is just asserting whether mutate is callable or not. I would keep it as it is.