Skip to content

[Code Quality] Add test cases for MCP experimental feature #3743

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
Closed
rithin-pullela-aws opened this issue Apr 17, 2025 · 2 comments · Fixed by #3787
Closed

[Code Quality] Add test cases for MCP experimental feature #3743

rithin-pullela-aws opened this issue Apr 17, 2025 · 2 comments · Fixed by #3787
Assignees
@rithin-pullela-aws
Labels
infrastructure

Comments

@rithin-pullela-aws
Copy link
Contributor

MCP is currently an experimental feature and hence it does not have UT and IT coverage. When it goes GA, we are targeting about 80% of test code coverage an ensure IT and UT for every feature.

Run ./gradlew opensearch-ml-plugin:jacocoTestCoverageVerification on local to see if the code coverage is satisfactory
@rithin-pullela-aws rithin-pullela-aws mentioned this issue Apr 17, 2025
Merged
5 tasks
@rithin-pullela-aws
Copy link
Contributor Author

Can someone please assign this issue to me?

@dhrubo-os dhrubo-os self-assigned this Apr 17, 2025
@dhrubo-os
Copy link
Collaborator

Thanks for creating this issue, assigned it to you.

@b4sjoo b4sjoo moved this to Untriaged in ml-commons projects Apr 22, 2025
@b4sjoo b4sjoo moved this from Untriaged to On-deck in ml-commons projects Apr 22, 2025
@rithin-pullela-aws rithin-pullela-aws mentioned this issue Apr 28, 2025
Merged
5 tasks
@github-project-automation github-project-automation bot moved this from On-deck to Done in ml-commons projects May 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rithin-pullela-aws rithin-pullela-aws

Labels
infrastructure
Projects
ml-commons projects
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add Unit Tests for MCP feature rithin-pullela-aws/ml-commons
3 participants
@dhrubo-os @b4sjoo @rithin-pullela-aws