Skip to content

Python: Improve test coverage #10681

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 19 commits into from
Mar 12, 2025
Merged

Python: Improve test coverage #10681

merged 19 commits into from
Mar 12, 2025

Conversation

rracanicci
Copy link
Contributor

Motivation and Context

Adding more tests to improve overall coverage.

Description

  • adding tests for bedrock agent channel
  • adding tests for bedrock model provider utils
  • adding tests for mistral ai chat completion
  • tests for bing search
  • tests for google search

Contribution Checklist

@rracanicci rracanicci requested a review from a team as a code owner February 25, 2025 17:55
@markwallace-microsoft markwallace-microsoft added the python Pull requests for the Python Semantic Kernel label Feb 25, 2025
@github-actions github-actions bot changed the title Improve test coverage Python: Improve test coverage Feb 25, 2025
@markwallace-microsoft
Copy link
Member

markwallace-microsoft commented Feb 25, 2025 •
edited
Loading

Python Test Coverage

Python Test Coverage Report •
FileStmtsMissCoverMissing
TOTAL19779219689% 
report-only-changed-files is enabled. No files were changed during this commit :)

Python Unit Test Overview

Tests Skipped Failures Errors Time
3290 5 💤 0 ❌ 0 🔥 1m 43s ⏱️

TaoChenOSU
TaoChenOSU reviewed Feb 25, 2025
View reviewed changes
moonbox3
moonbox3 reviewed Feb 28, 2025
View reviewed changes
Copy link
Contributor

@moonbox3 moonbox3 left a comment

Choose a reason for hiding this comment

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

Thanks for working on this, @rracanicci.

@rracanicci rracanicci requested a review from a team as a code owner March 5, 2025 16:00
@markwallace-microsoft markwallace-microsoft added .NET Issue or Pull requests regarding .NET code kernel Issues or pull requests impacting the core kernel kernel.core documentation memory labels Mar 5, 2025
@github-actions github-actions bot changed the title Python: Improve test coverage .Net: Python: Improve test coverage Mar 5, 2025
@rracanicci rracanicci force-pushed the feature/improve-test-coverage branch from 50cd0c6 to 4fb3101 Compare March 5, 2025 16:03
@markwallace-microsoft markwallace-microsoft removed .NET Issue or Pull requests regarding .NET code kernel Issues or pull requests impacting the core kernel kernel.core documentation memory labels Mar 5, 2025
@rracanicci rracanicci changed the title .Net: Python: Improve test coverage Python: Improve test coverage Mar 5, 2025
@rracanicci rracanicci force-pushed the feature/improve-test-coverage branch 2 times, most recently from 19bb678 to a815b23 Compare March 7, 2025 10:28
@rracanicci rracanicci force-pushed the feature/improve-test-coverage branch from 4476f53 to 8f62a96 Compare March 7, 2025 12:00
@moonbox3 moonbox3 enabled auto-merge March 12, 2025 07:45
@moonbox3 moonbox3 added this pull request to the merge queue Mar 12, 2025
Merged via the queue into microsoft:main with commit 63f140e Mar 12, 2025
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eavanvalkenburg eavanvalkenburg eavanvalkenburg left review comments

@TaoChenOSU TaoChenOSU TaoChenOSU approved these changes

@moonbox3 moonbox3 moonbox3 approved these changes

Assignees
No one assigned
Labels
python Pull requests for the Python Semantic Kernel
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@rracanicci @markwallace-microsoft @TaoChenOSU @eavanvalkenburg @moonbox3