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

[FEATURE] Standardize, setup, report, and increase test coverage #73

Open
Tracked by #30
Xtansia opened this issue Sep 14, 2022 · 3 comments
Open
Tracked by #30

[FEATURE] Standardize, setup, report, and increase test coverage #73

Xtansia opened this issue Sep 14, 2022 · 3 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@Xtansia
Copy link
Collaborator

Xtansia commented Sep 14, 2022

Setup code coverage, once reporting from CI/CD analyze the test gap and fill it.

See the meta issue opensearch-project/opensearch-clients#30 for details.

@Xtansia
Copy link
Collaborator Author

Xtansia commented Sep 16, 2022

codecov.io is already in use by other clients and opensearch projects for reporting, so that is essentially settled.

To do the actual coverage collection, it appears the main options are directly with the compiler https://doc.rust-lang.org/rustc/instrument-coverage.html or with https://github.com/xd009642/tarpaulin.

It appears to me that the compiler's support is the new best practice, and using something like grcov to get lcov output to pass to codecov

@harshavamsi
Copy link
Contributor

@Xtansia this can be closed.

@Xtansia
Copy link
Collaborator Author

Xtansia commented Oct 4, 2022

@harshavamsi This issue is not complete, only the report part has actually been completed

@Xtansia Xtansia added the good first issue Good for newcomers label Sep 18, 2023
@Xtansia Xtansia removed their assignment Sep 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants