Skip to content

Fix stale test count badge (183 -> 191) - #39

Merged
OnePunchMonk merged 1 commit into
mainfrom
fix/test-count-badge
Sep 22, 2026
Merged

OnePunchMonk merged 1 commit into
mainfrom
fix/test-count-badge

Conversation

@OnePunchMonk

@OnePunchMonk OnePunchMonk commented Sep 22, 2026 •

Copy link
Copy Markdown
Owner

This PR fixes a stale test count badge in README.md.

The badge previously showed 183 passed tests. That number was out of date.

I ran the full test suite with pytest tests/ -v --cov=src. The result was 191 passed, 0 failed, 0 skipped.

This PR updates the README badge from 183 to 191 to match the verified result.

This resolves #21, which asked to verify the test suite. The issue title referenced 63 tests, but that number was also stale. The actual verified count is 191.

Verified by running pytest tests/ -v --cov=src: 191 passed, 0 failed,
0 skipped/xfailed. README badge said 183; issue #21 claimed 63. Neither
matched actual test count.

Copy link
Copy Markdown
Owner Author

The "Test Suite (Python 3.10)" check is failing on this PR, but it is not caused by this change.

The failure is a ruff lint error in tests/test_tools.py:

tests/test_tools.py:6:1: I001 Import block is un-sorted or un-formatted

This PR only touches README.md (the test count badge) and does not modify tests/test_tools.py. I confirmed the same ruff check failure reproduces identically against main, so it predates this PR.

No existing fix PR is open for this, so I'm not pulling one in. I'm not fixing it here either, since it's unrelated to this PR's scope. Leaving this PR as is; the other checks (Python 3.11, 3.12, security, build, etc.) are unaffected by this failure.


Generated by Claude Code

@OnePunchMonk
OnePunchMonk merged commit 082aa5a into main Sep 22, 2026
8 of 11 checks passed
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.

Verify test suite: confirm 63 tests passing

1 participant