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.

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

Refactor getters and pass resource into call and add codecov #11

Merged

Conversation

peterzhuamazon
Copy link
Member

@peterzhuamazon peterzhuamazon commented Sep 27, 2024

Description

Refactor getters and pass resource into call and add codecov

Issues Resolved

Closes #5
Closes #10
#6

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
@peterzhuamazon peterzhuamazon added the enhancement New feature or request label Sep 27, 2024
@peterzhuamazon peterzhuamazon self-assigned this Sep 27, 2024
@peterzhuamazon peterzhuamazon linked an issue Sep 27, 2024 that may be closed by this pull request
@peterzhuamazon peterzhuamazon marked this pull request as draft September 27, 2024 00:03
@peterzhuamazon
Copy link
Member Author

I will add the test cases soon.

@prudhvigodithi please take a look when you have time.

Thanks!

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
@peterzhuamazon
Copy link
Member Author

Added tests for all the utility, calls.

Will open another PR to test all the service related objects.

Thanks.

@peterzhuamazon peterzhuamazon marked this pull request as ready for review September 27, 2024 22:17
Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
@peterzhuamazon peterzhuamazon changed the title Refactor getter and pass resource into call Refactor getters and pass resource into call and add codecov Sep 27, 2024
Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
@peterzhuamazon peterzhuamazon merged commit bab72ee into opensearch-project:main Sep 30, 2024
6 checks passed
@peterzhuamazon peterzhuamazon deleted the service-add-resource branch September 30, 2024 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

[Enhancement] Add codecov [Enhancement] Proper ways of adding getters to classes
2 participants