Skip to content

Conversation

@DhiraPT
Copy link

@DhiraPT DhiraPT commented Dec 29, 2025

Summary

This PR refactors the testing infrastructure to separate correctness tests from performance benchmarks and establishes a strict directory mirroring strategy for better maintainability.

Key Changes

  • Separation of Concerns: Moved performance/profiling code from tests/ into a dedicated benchmark/ directory (e.g., bench_index.py, bench_store.py).
  • Infrastructure: Added conftest.py with pytest fixtures (e.g., cuda_device) and updated pyproject.toml with markers (@cuda, @distributed, @e2e).
  • Strict Mirroring: Reorganized tests/ to exactly mirror the python/minisgl/ source structure, documented in the new tests/README.md.
  • Coverage: Added unit tests for message/backend and message/utils (replacing the old test_serialize.py).

@DhiraPT DhiraPT marked this pull request as draft December 29, 2025 19:15
@DhiraPT DhiraPT changed the title [Refactor] Test [Refactor] Restructure test suite to match source layout and isolate benchmarks Dec 31, 2025
@DhiraPT DhiraPT marked this pull request as ready for review December 31, 2025 17:47
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