Skip to content

test: Amazon Sagemaker - add unit tests#3191

Merged
anakin87 merged 2 commits intomainfrom
sagemaker-tests
Apr 21, 2026
Merged

test: Amazon Sagemaker - add unit tests#3191
anakin87 merged 2 commits intomainfrom
sagemaker-tests

Conversation

@anakin87
Copy link
Copy Markdown
Member

@anakin87 anakin87 commented Apr 21, 2026

Related Issues

Proposed Changes:

  • add unit tests for Amazon Sagemaker, focusing on uncovered code paths

How did you test it?

CI

Checklist

@github-actions
Copy link
Copy Markdown
Contributor

Coverage report (amazon_sagemaker)

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  integrations/amazon_sagemaker/src/haystack_integrations/components/generators/amazon_sagemaker
  sagemaker.py
Project Total  

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

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

@sjrl sjrl 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 5e145d4 into main Apr 21, 2026
13 checks passed
@anakin87 anakin87 deleted the sagemaker-tests branch April 21, 2026 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants