Skip to content

Use tarpaulin config and test error displays#35

Merged
DennisLent merged 1 commit intomainfrom
codex/update-tarpaulin-config-for-coverage
Aug 20, 2025
Merged

Use tarpaulin config and test error displays#35
DennisLent merged 1 commit intomainfrom
codex/update-tarpaulin-config-for-coverage

Conversation

@DennisLent
Copy link
Owner

Summary

  • configure tarpaulin to skip GUI, Python, and CLI entrypoint files
  • add comprehensive display tests for GameError variants
  • point CI coverage job at shared config without redundant crate exclusion

Testing

  • cargo fmt --all
  • cargo clippy --all-targets --all-features -- -D warnings
  • cargo test -p rusty_runways_core
  • cargo tarpaulin --workspace --out Xml --skip-clean

https://chatgpt.com/codex/tasks/task_e_68a5f61fc4288320b663cbcfbba24548

@codecov
Copy link

codecov bot commented Aug 20, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 60.63%. Comparing base (1edeca2) to head (7f744dd).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main      #35       +/-   ##
===========================================
+ Coverage   36.53%   60.63%   +24.09%     
===========================================
  Files          22       17        -5     
  Lines        1497      912      -585     
===========================================
+ Hits          547      553        +6     
+ Misses        950      359      -591     
Flag Coverage Δ
rust 60.63% <ø> (+24.09%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@DennisLent DennisLent merged commit fa7db11 into main Aug 20, 2025
10 checks passed
@DennisLent DennisLent deleted the codex/update-tarpaulin-config-for-coverage branch August 20, 2025 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant