Skip to content

perf: make test suite snappier and clean up some testing things#171

Merged
beckermr merged 19 commits intomainfrom
parallel-tests
Feb 8, 2026
Merged

perf: make test suite snappier and clean up some testing things#171
beckermr merged 19 commits intomainfrom
parallel-tests

Conversation

@beckermr
Copy link
Collaborator

@beckermr beckermr commented Feb 8, 2026

This PR makes the test suite snappier by:

  • using pytest-split to split the tests over four jobs
  • removing extra benchmarks

I've also fixed a few deprecations, bugs, and minor oddities in the tests to make life easier. This includes

  • updating the pre-commit hooks to use the correct hook for ruff-check
  • changed codspeed mode to simulation to match latest API
  • changing imports in the tests to allow them to be run without import errors due to not having galsim's test utils module
  • adding a call to make the output directory in our text fixture if it does not exist
  • removed premade output directory in favor of making it on the fly

closes #166

@codspeed-hq
Copy link

codspeed-hq bot commented Feb 8, 2026

Merging this PR will improve performance by 35.48%

⚡ 1 improved benchmark
✅ 15 untouched benchmarks

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Simulation test_benchmark_spergel_calcfluxrad[run] 1.5 ms 1.1 ms +35.48%

Comparing parallel-tests (eb21f94) with main (cfa1bfd)

Open in CodSpeed

@beckermr beckermr changed the title test: use pytest-split for faster tests perf: make test suite snappier Feb 8, 2026
@beckermr beckermr changed the title perf: make test suite snappier perf: make test suite snappier and add sdist to pypi Feb 8, 2026
@beckermr beckermr changed the title perf: make test suite snappier and add sdist to pypi perf: make test suite snappier Feb 8, 2026
@beckermr beckermr changed the title perf: make test suite snappier perf: make test suite snappier and clean up some testing things Feb 8, 2026
@beckermr beckermr marked this pull request as ready for review February 8, 2026 12:29
@beckermr beckermr merged commit db55542 into main Feb 8, 2026
8 checks passed
@beckermr beckermr deleted the parallel-tests branch February 8, 2026 12:43
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.

adopt pytest-xdist for faster tests

1 participant