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

[indexer-test-coverage] add test coverage ci #592

Merged
merged 2 commits into from
Nov 20, 2024

Conversation

yuunlimm
Copy link
Contributor

@yuunlimm yuunlimm commented Nov 6, 2024

Description

Introducing a CI workflow to track and report test coverage using cargo llvm-cov and integrates with Codecov for visualization. The workflow runs daily or on PRs, generates an lcov report, and uploads it to Codecov. It also updates the README.md to include a Codecov badge for coverage metrics. This aims to improve code quality by providing continuous visibility into test coverage and highlighting untested areas.

Test Plan

https://app.codecov.io/github/aptos-labs/aptos-indexer-processors/tree/11-06-_indexer-test-coverage_add_test_coverage_ci

we can find our coverage here.

Next Step

  • will need to investigate if we can specify the directory we would like to track as a indexer coverage

Copy link
Contributor Author

yuunlimm commented Nov 6, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

@yuunlimm yuunlimm force-pushed the 11-06-_indexer-test-coverage_add_test_coverage_ci branch 7 times, most recently from 729b99a to d37d65a Compare November 7, 2024 01:47
@yuunlimm yuunlimm force-pushed the 11-06-_indexer-test-coverage_add_test_coverage_ci branch from d37d65a to d6f57f0 Compare November 19, 2024 23:53
@yuunlimm yuunlimm force-pushed the 11-06-_indexer-test-coverage_add_test_coverage_ci branch 4 times, most recently from 034d379 to 31edeb1 Compare November 19, 2024 23:58
Copy link

codecov bot commented Nov 20, 2024

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@yuunlimm yuunlimm force-pushed the 11-06-_indexer-test-coverage_add_test_coverage_ci branch from 31edeb1 to bde6363 Compare November 20, 2024 00:13
@yuunlimm yuunlimm force-pushed the 11-06-_indexer-test-coverage_add_test_coverage_ci branch from bde6363 to 7b0c9f3 Compare November 20, 2024 00:21
@yuunlimm yuunlimm marked this pull request as ready for review November 20, 2024 00:48
@yuunlimm yuunlimm requested a review from a team November 20, 2024 01:12
@yuunlimm yuunlimm merged commit 1af386d into main Nov 20, 2024
11 checks passed
@yuunlimm yuunlimm deleted the 11-06-_indexer-test-coverage_add_test_coverage_ci branch November 20, 2024 18:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants