Skip to content

chore: use memory store in tests - #2634

Open
ilan-gold wants to merge 4 commits into
mainfrom
ig/memory_store_for_tests
Open

chore: use memory store in tests#2634
ilan-gold wants to merge 4 commits into
mainfrom
ig/memory_store_for_tests

Conversation

@ilan-gold

@ilan-gold ilan-gold commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

I try to resolve the problem "what if our zarr tests didn't take so long and we also didn't write SO much data during testing" by moving to memory stores where possible - there is still a test explicitly for local storage in tests/test_readwrite.py::test_readwrite_equivalent_h5ad_zarr (or where appropriate to not change) but otherwise, this should provide a big speedup!

  • Closes #
  • Tests added
  • Release note not necessary because: testing change

@ilan-gold ilan-gold added this to the 0.13.4 milestone Sep 4, 2026
@ilan-gold ilan-gold changed the title perf: use memory store in tests chore: use memory store in tests Sep 4, 2026
@codecov

codecov Bot commented Sep 4, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 95.23810% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 85.83%. Comparing base (d75540e) to head (425e98e).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/anndata/tests/helpers.py 88.88% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2634      +/-   ##
==========================================
- Coverage   87.66%   85.83%   -1.83%     
==========================================
  Files          49       49              
  Lines        8121     8133      +12     
==========================================
- Hits         7119     6981     -138     
- Misses       1002     1152     +150     
Files with missing lines Coverage Δ
src/anndata/experimental/merge.py 88.70% <100.00%> (+0.37%) ⬆️
src/anndata/tests/helpers.py 83.78% <88.88%> (-9.00%) ⬇️

... and 7 files with indirect coverage changes

@ilan-gold

Copy link
Copy Markdown
Contributor Author

Not too shabby! https://github.com/scverse/anndata/actions/runs/33877901352/job/101039244161 for example took 13min but now it runs in 9min!

@ilan-gold
ilan-gold marked this pull request as ready for review September 4, 2026 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant