Skip to content

Expand test coverage and add Codecov reporting#5

Merged
snabb merged 4 commits into
mainfrom
add-transaction-tests
Apr 18, 2026
Merged

Expand test coverage and add Codecov reporting#5
snabb merged 4 commits into
mainfrom
add-transaction-tests

Conversation

@snabb
Copy link
Copy Markdown
Owner

@snabb snabb commented Apr 18, 2026

Summary

  • Add 16 new blocking I2C transaction tests covering all previously untested driver methods: configure_flat, configure_orientation, configure_tap, configure_significant_motion, configure_tilt, step detector/counter enable, configure_step_counter, reset_step_counter, set_interrupt_latching, read_interrupt_status, set_fifo_config, fifo_fill_level, flush_fifo, read_fifo_words, read_temperature_celsius, and read_accel/read_gyro
  • Mirror all 24 blocking I2C tests in the async I2C test suite so both transports have equivalent coverage
  • Add a coverage CI job using cargo-llvm-cov that uploads LCOV reports to Codecov via CODECOV_TOKEN repository secret
  • Add Codecov badge to README

Test plan

  • All 73 tests pass locally (cargo test)
  • Verify Codecov badge appears after first CI run (requires CODECOV_TOKEN secret to be set in repository settings)

🤖 Generated with Claude Code

snabb and others added 3 commits April 18, 2026 23:05
Add 16 new blocking I2C tests covering configure_flat, configure_orientation,
configure_tap, configure_significant_motion, configure_tilt, step detector/counter
enable, configure_step_counter, reset_step_counter, set_interrupt_latching,
read_interrupt_status, set_fifo_config, fifo_fill_level, flush_fifo,
read_fifo_words, read_temperature_celsius, and read_accel/read_gyro.

Mirror all 24 blocking I2C tests in the async I2C test suite so both
transports have equivalent coverage.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Add a coverage job that generates an LCOV report with cargo-llvm-cov
and uploads it to Codecov using CODECOV_TOKEN from repository secrets.
Add the Codecov badge to README.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 18, 2026

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 ☂️

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@snabb snabb merged commit 57fa724 into main Apr 18, 2026
4 checks passed
@snabb snabb deleted the add-transaction-tests branch April 18, 2026 20:18
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.

1 participant