Skip to content

Align unit tests with production modules - #81

Merged
dceoy merged 9 commits into
mainfrom
refactor/test-module-unit-mapping
Sep 2, 2026
Merged

dceoy merged 9 commits into
mainfrom
refactor/test-module-unit-mapping

Conversation

@dceoy

@dceoy dceoy commented Sep 1, 2026

Copy link
Copy Markdown
Owner

Summary

  • Mirror top-level and router production modules under tests/unit/.
  • Add the constants module test and an automated bijection contract.
  • Move CLI and application-wide prefix/OpenAPI checks to integration/contract scopes.
  • Merge lifespan coverage into tests/unit/test_main.py while preserving coverage.

Verification

  • pytest: 247 passed
  • Coverage: 100% (configured threshold preserved)
  • Ruff: passed
  • Pyright: 0 errors
  • Prettier, actionlint, yamllint, zizmor, and Checkov: passed

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Your trial has ended. Reactivate Greptile to resume code reviews.

@dceoy dceoy left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Review pass for 7826a14. Two actionable contract gaps were found and anchored inline.

Comment thread tests/contracts/test_unit_test_bijection.py Outdated
Comment thread tests/contracts/test_unit_test_bijection.py Outdated

@dceoy dceoy left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

No new actionable findings were found in this pass;

@dceoy dceoy self-assigned this Sep 2, 2026
@dceoy
dceoy merged commit 64ebb89 into main Sep 2, 2026
8 checks passed
@dceoy
dceoy deleted the refactor/test-module-unit-mapping branch September 2, 2026 02:40
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.

2 participants