Skip to content

test: add unit tests for Google GenAI integration#3092

Merged
anakin87 merged 2 commits into
mainfrom
google-genai-tests
Apr 2, 2026
Merged

test: add unit tests for Google GenAI integration#3092
anakin87 merged 2 commits into
mainfrom
google-genai-tests

Conversation

@anakin87
Copy link
Copy Markdown
Member

@anakin87 anakin87 commented Apr 2, 2026

Related Issues

Proposed Changes:

  • increase unit tests coverage by adding unit tests
  • remove two redundant integration tests

How did you test it?

CI

Checklist

@github-actions github-actions Bot added integration:google-genai type:documentation Improvements or additions to documentation labels Apr 2, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 2, 2026

Coverage report (google_genai)

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  integrations/google_genai/src/haystack_integrations/components/generators/google_genai/chat
  chat_generator.py
  utils.py
Project Total  

This report was generated by python-coverage-comment-action

@anakin87 anakin87 self-assigned this Apr 2, 2026
@anakin87 anakin87 marked this pull request as ready for review April 2, 2026 13:42
@anakin87 anakin87 requested a review from a team as a code owner April 2, 2026 13:42
@anakin87 anakin87 requested review from davidsbatista and removed request for a team April 2, 2026 13:42
Copy link
Copy Markdown
Contributor

@davidsbatista davidsbatista left a comment

Choose a reason for hiding this comment

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

looks good!

@anakin87 anakin87 merged commit 58b00df into main Apr 2, 2026
13 checks passed
@anakin87 anakin87 deleted the google-genai-tests branch April 2, 2026 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

integration:google-genai type:documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants