The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
litellm	Ready	Preview, Comment	Mar 20, 2026 11:13pm

Merging this PR will not alter performance

✅ 16 untouched benchmarks

_{Comparing joereyna:fix/anthropic-file-content-test-mock (f0e0d98) with main (72c307d)}

Greptile Summary

This PR fixes a test regression in test_afile_content_missing_api_key caused by commit f415b72b, which changed AnthropicFilesHandler.afile_content to call get_auth_header instead of get_api_key for credential resolution. The old mock on get_api_key was silently bypassed, causing the test to make a real HTTP request to api.anthropic.com and receive a 400 instead of the expected ValueError.

Key changes:

• Replaces patch.object(handler.anthropic_model_info, "get_api_key", return_value=None) with patch.object(handler.anthropic_model_info, "get_auth_header", return_value=None) in test_afile_content_missing_api_key, correctly intercepting the gating check in the handler.

Remaining concern:

• A sibling test (test_afile_content_mixed_results, line 306) still carries a dead get_api_key mock that is also never exercised by the handler post-refactor. That test passes incidentally because the credential is supplied as an explicit argument, but the mock should be updated to patch get_auth_header for consistency and future-proofing.

Confidence Score: 4/5

• This PR is safe to merge; the targeted fix is correct. A minor dead-mock cleanup is recommended but not blocking.
• The one-line change is precise and correctly addresses the root cause. The handler's get_auth_header gating check is now properly mocked, ensuring the test no longer makes real outbound HTTP calls. A small residual issue — a dead get_api_key mock in a related test — does not affect correctness but leaves misleading test code.
• tests/test_litellm/llms/anthropic/test_anthropic_files_and_batches.py — line 306 still has a dead get_api_key mock that should be updated.

Sequence Diagram

sequenceDiagram
    participant Test
    participant Handler as afile_content
    participant ModelInfo as AnthropicModelInfo
    participant HTTP as httpx client

    Note over Test,HTTP: After fix — mock intercepts correctly
    Test->>Handler: afile_content(file_id, no credentials)
    Handler->>ModelInfo: get_auth_header() [mocked to return None]
    ModelInfo-->>Handler: None
    Handler-->>Test: raises ValueError ✅

    Note over Test,HTTP: Before fix — mock was bypassed
    Test->>Handler: afile_content(file_id, no credentials)
    Handler->>ModelInfo: get_auth_header() [NOT mocked, get_api_key was]
    ModelInfo-->>Handler: reads env vars
    Handler-->>HTTP: real outbound request to Anthropic — returns 400 ❌

_{Last reviewed commit: "fix(test): mock get_..."}