Skip to content

Add import smoke test and drop removed loaders from __all__ - #38

Closed
leifdenby wants to merge 1 commit into
mlwp-tools:refactor/decouple_loading_validationfrom
leifdenby:test/import-smoke-test
Closed

leifdenby wants to merge 1 commit into
mlwp-tools:refactor/decouple_loading_validationfrom
leifdenby:test/import-smoke-test

Conversation

@leifdenby

@leifdenby leifdenby commented Oct 27, 2026 •

Copy link
Copy Markdown
Member

Fixes the failing Run Unit Tests check on #27.

After the properties test was removed, tests/ no longer existed in the repo, so pytest tests exited with code 4 (file or directory not found).

  • Adds tests/test_import.py, which checks that mxalign imports and that every name in mxalign.__all__ exists.
  • Removes "loaders" from mxalign.__all__. The loaders module was removed in refactor: outsource loading and validation #27, so from mxalign import * would raise an error. The new test caught this.

🤖 Generated with Claude Code

The tests/ directory was empty after removing the properties test, so
`pytest tests` failed with exit code 4 in CI. The new test also caught
that `loaders` was still listed in mxalign.__all__ after its removal.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@leifdenby

Copy link
Copy Markdown
Member Author

Hadn't noticed that you already fixed this :)

@leifdenby leifdenby closed this Oct 27, 2026
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