Skip to content

chore: Remove 4 redundant Meta Llama integration tests#2646

Merged
vblagoje merged 2 commits into
mainfrom
meta_llama_less_integration_tests
Jan 5, 2026
Merged

chore: Remove 4 redundant Meta Llama integration tests#2646
vblagoje merged 2 commits into
mainfrom
meta_llama_less_integration_tests

Conversation

@vblagoje
Copy link
Copy Markdown
Member

@vblagoje vblagoje commented Jan 2, 2026

Why:

Reduces number of LLMs calls for meta_llama integration tests by removing 4 redundant tests that duplicate functionality already covered by other integration tests. This should hopefully stop CI rate limit failures we get from Meta.

What:

  • Removed test_live_run_async - duplicates basic async functionality covered by other async tests
  • Removed test_live_run_with_response_format_json_schema - redundant with Pydantic response format test
  • Removed test_live_run_with_tools - subset of test_live_run_with_tools_and_response
  • Removed test_live_run_with_tools_streaming_async - duplicates sync version and other async tool tests

How can it be used:

N/A

How did you test it:

  • Verified remaining tests still provide comprehensive coverage
  • Confirmed no functionality gaps in test suite
  • All removed tests were validated as redundant with existing coverage

Notes for the reviewer:

This is a straightforward cleanup - each removed test's functionality is still covered by one or more remaining tests. The reduction in test count (14 → 10) maintains quality while improving CI efficiency.

@github-actions github-actions Bot added integration:meta-llama type:documentation Improvements or additions to documentation labels Jan 2, 2026
@vblagoje vblagoje marked this pull request as ready for review January 2, 2026 10:21
@vblagoje vblagoje requested a review from a team as a code owner January 2, 2026 10:21
@vblagoje vblagoje requested review from mpangrazzi and removed request for a team January 2, 2026 10:21
@vblagoje vblagoje changed the title chore: Remove 4 redundant integration tests chore: Remove 4 redundant Meta Llama integration tests Jan 2, 2026
@vblagoje
Copy link
Copy Markdown
Member Author

vblagoje commented Jan 2, 2026

@mpangrazzi this is hopefully a fix for our rate limiting CI Meta Llama failures. Let's remove 4 redundant integration tests and see if we don't get rate limited as often

Copy link
Copy Markdown
Contributor

@mpangrazzi mpangrazzi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@vblagoje vblagoje merged commit 690e767 into main Jan 5, 2026
10 checks passed
@vblagoje vblagoje deleted the meta_llama_less_integration_tests branch January 5, 2026 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

integration:meta-llama type:documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants