Skip to content
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

[FEATURE] Improve test coverage for MLHttpClientFactory.java #1379

Open
dhrubo-os opened this issue Sep 25, 2023 · 3 comments
Open

[FEATURE] Improve test coverage for MLHttpClientFactory.java #1379

dhrubo-os opened this issue Sep 25, 2023 · 3 comments
Assignees
@Arnav-Gr0ver
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@dhrubo-os
Copy link
Collaborator

dhrubo-os commented Sep 25, 2023 •
edited
Loading

We need to improve test coverage in our repo.

This issue will target only to improve test coverage for MLHttpClientFactory.java class

From this url we can see where test coverage is missing for this class: https://app.codecov.io/gh/opensearch-project/ml-commons/blob/2.x/ml-algorithms%2Fsrc%2Fmain%2Fjava%2Forg%2Fopensearch%2Fml%2Fengine%2Fhttpclient%2FMLHttpClientFactory.java
@dhrubo-os dhrubo-os added enhancement New feature or request untriaged good first issue Good for newcomers and removed untriaged labels Sep 25, 2023
@Arnav-Gr0ver
Copy link
Contributor

@dhrubo-os Hello Dhrubo! I would like to take up this feature as my first issue

@dhrubo-os
Copy link
Collaborator Author

Sure.

@dhrubo-os
Copy link
Collaborator Author

@Arnav-Gr0ver any update on this?

This was referenced Nov 10, 2023
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Arnav-Gr0ver Arnav-Gr0ver

Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Arnav-Gr0ver @dhrubo-os