Skip to content
This repository was archived by the owner on Nov 19, 2025. It is now read-only.

Add test and coverage steps to Rust CI pipeline#57

Merged
RonTuretzky merged 2 commits into
devfrom
hudsonhrh/add-test-coverage-ci
Oct 8, 2025
Merged

Add test and coverage steps to Rust CI pipeline#57
RonTuretzky merged 2 commits into
devfrom
hudsonhrh/add-test-coverage-ci

Conversation

@hudsonhrh
Copy link
Copy Markdown
Contributor

Summary

  • Add test job that runs all workspace tests with cargo test
  • Add coverage job that generates code coverage reports using cargo-tarpaulin
  • Integrate with Codecov to make coverage results visible in CI and PRs

Test plan

  • Verify CI pipeline runs successfully
  • Verify test job executes all tests
  • Verify coverage job generates and uploads coverage reports
  • Check Codecov integration displays results in PR

Resolves #36

🤖 Generated with Claude Code

Update the CI build pipeline to include:
- Test job that runs all workspace tests with cargo test
- Coverage job that generates code coverage reports using cargo-tarpaulin
- Codecov integration to make coverage results visible in CI and PRs

Resolves #36

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
@hudsonhrh hudsonhrh marked this pull request as ready for review October 6, 2025 18:59
Comment thread .github/workflows/rust-ci.yml Outdated
- name: Generate coverage report
run: cargo tarpaulin --all-features --workspace --out xml --output-dir ./coverage

- name: Upload coverage to Codecov
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This step fails with:

error - 2025-10-06 18:52:25,639 -- Upload failed: {"message":"Token required - not valid tokenless upload"}

I think it's ok to just remove it personally

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removed here 3b81f85

Copy link
Copy Markdown
Contributor

@bagelface bagelface left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see comment

Codecov upload fails without a token. Coverage report is still generated
and visible in CI logs.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
@hudsonhrh hudsonhrh requested a review from bagelface October 7, 2025 19:04
Copy link
Copy Markdown
Contributor

@bagelface bagelface left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@RonTuretzky RonTuretzky merged commit 7aa353f into dev Oct 8, 2025
6 checks passed
@RonTuretzky RonTuretzky deleted the hudsonhrh/add-test-coverage-ci branch October 8, 2025 01:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add 'test' and 'coverage' steps to build pipeline

3 participants