Skip to content

Speed up make check ~5x#459

Open
Seth Fitzsimmons (mojodna) wants to merge 1 commit intodevfrom
speed-up-tests
Open

Speed up make check ~5x#459
Seth Fitzsimmons (mojodna) wants to merge 1 commit intodevfrom
speed-up-tests

Conversation

@mojodna
Copy link
Collaborator

Run test, doctest, lint, and mypy targets in parallel via make -j, and replace the geometry type powerset (127 subsets) with representative subsets (singletons + pairs + full = 28 subsets).

Changes

Makefile: Split each check target into a -only variant (no uv-sync dep) so make -j can run them concurrently. check does a single uv-sync upfront, then fans out.

test_geom.py: Replace powerset enumeration with singletons, pairs, and the full set. Covers individual type acceptance/rejection, pairwise composition, and unconstrained acceptance without combinatorial explosion.

Geometry constraint tests: replace full powerset (127 subsets of 7
geometry types) with singletons + pairs + full set (29 subsets).
GeometryTypeConstraint is a set-membership check — combinatorial
testing beyond pairs exercises no additional code paths.

Makefile: run pytest, doctest, ruff, and mypy concurrently via
make -j after a single upfront uv-sync. Added -only variants
(no uv-sync dep) for the parallel invocation; standalone targets
preserved with their original uv-sync dependencies.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant