Skip to content

Add coverage badge and test workflow#19

Merged
roesel merged 3 commits into
mainfrom
claude/add-coverage-badge-qac8Q
Mar 8, 2026
Merged

Add coverage badge and test workflow#19
roesel merged 3 commits into
mainfrom
claude/add-coverage-badge-qac8Q

Conversation

@roesel

@roesel roesel commented Mar 8, 2026

Copy link
Copy Markdown
Owner

Add a GitHub Actions workflow that runs pytest with coverage reporting
and uploads results to Codecov. Add Codecov coverage badge to README.

https://claude.ai/code/session_01Q96kfpc951RY8U2Nspm2CP

claude added 2 commits March 8, 2026 09:40
Add a GitHub Actions workflow that runs pytest with coverage reporting
and uploads results to Codecov. Add Codecov coverage badge to README.

https://claude.ai/code/session_01Q96kfpc951RY8U2Nspm2CP
- Add tests/test_unit.py with 41 unit tests covering tools (parse, s32,
  is_metric, error_check, move_check), commands, devices, error codes,
  and custom exceptions — all runnable without hardware
- Mark existing hardware tests in test_rotator.py with pytest.mark.hardware
- Register the "hardware" marker in pyproject.toml
- Update CI workflow to skip hardware-marked tests (-m "not hardware")

https://claude.ai/code/session_01Q96kfpc951RY8U2Nspm2CP
@codecov-commenter

Copy link
Copy Markdown

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.

ℹ️ You can also turn on project coverage checks and project coverage reporting on Pull Request comment

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

The `X | Y` union syntax in a runtime assignment is not supported
in Python 3.9 (only in annotations with `from __future__ import
annotations`). Use typing.Union/Dict/Tuple instead.

https://claude.ai/code/session_01Q96kfpc951RY8U2Nspm2CP
@roesel
roesel merged commit b7105e1 into main Mar 8, 2026
6 checks passed
@roesel
roesel deleted the claude/add-coverage-badge-qac8Q branch March 8, 2026 10:24
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.

3 participants