Skip to content

Add test coverage for packages/special/__init__.py#1926

Merged
rolandwalker merged 1 commit into
mainfrom
RW/add-test-coverage-special-init-py
Jun 9, 2026
Merged

Add test coverage for packages/special/__init__.py#1926
rolandwalker merged 1 commit into
mainfrom
RW/add-test-coverage-special-init-py

Conversation

@rolandwalker

Copy link
Copy Markdown
Contributor

Description

Add test coverage for packages/special/__init__.py, since it now contains substantive logic.

Checklist

  • I added this contribution to the changelog.md file.
  • I added my name to the AUTHORS file (or it's already there).
  • To lint and format the code, I ran
    uv run ruff check && uv run ruff format && uv run mypy --install-types .

@rolandwalker rolandwalker self-assigned this Jun 8, 2026
@rolandwalker rolandwalker force-pushed the RW/add-test-coverage-special-init-py branch from bc27d1b to 0ddd657 Compare June 8, 2026 09:54
@rolandwalker

rolandwalker commented Jun 8, 2026

Copy link
Copy Markdown
Contributor Author

Nice. I think the tests can pass in some order but not others. Randomizing the test order (as we did some time ago) catches this.

last image

@rolandwalker rolandwalker force-pushed the RW/add-test-coverage-special-init-py branch from 0ddd657 to 16b238a Compare June 8, 2026 10:33
@rolandwalker

Copy link
Copy Markdown
Contributor Author

the tests can pass in some order but not others

Fixed.

@rolandwalker rolandwalker requested review from amjith and scottnemes June 8, 2026 10:35
since it now contains substantive logic
@rolandwalker rolandwalker force-pushed the RW/add-test-coverage-special-init-py branch from 16b238a to 14f6812 Compare June 9, 2026 10:59
@rolandwalker rolandwalker merged commit 5d90ae0 into main Jun 9, 2026
11 checks passed
@rolandwalker rolandwalker deleted the RW/add-test-coverage-special-init-py branch June 9, 2026 11:31
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