Skip to content

tests: create dedicated test folder with unit and integration tests - #620

Merged
RyanPCo merged 1 commit into
mainfrom
ryanco/tests-dir
Sep 20, 2026
Merged

RyanPCo merged 1 commit into
mainfrom
ryanco/tests-dir

Conversation

@RyanPCo

@RyanPCo RyanPCo commented Sep 12, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Moves every test_*.py out of egomimic/ into tests/{unit,integration} and makes pytest from the repo root collect cleanly (24 collection errors → 0).

  • pyproject: [tool.pytest.ini_options] with testpaths=["tests"] and norecursedirs so a bare pytest no longer sweeps external/, the venv, or the Stanford hardware scripts. Registers the integration marker.
  • tests/unit/: test_dataset_filter (moved) and test_action_chunk_transforms (moved and ported to the post-embodiment classes #234 API: builders now come from embodiment.{eva,human}, is_quat= → mode=, cmd_gripper keys, trailing NumpyToTensor).
  • tests/integration/: real-episode comparison, pi05, mink solver, s3/sql scripts, mecka zero-row scanner. tests/conftest.py marks everything under that dir by path and skips it unless --integration is passed. pytest.importorskip guards openpi / mujoco / mink / the removed S3RLDBDataset. Hardcoded personal and lab paths are replaced by EGOVERSE_TEST_{EVA,ARIA,SCALE}_ZARR and EGOVERSE_TEST_LEROBOT_CACHE.
  • Deletes egomimic/test_zarr.py (a PACE-only script that raised FileNotFoundError at import).
  • AGENTS.md gains a Tests section; MINK_SOLVER_README.md points at the new path.

Follow-up once #infra-fixes lands: move test_pi05_norm_rot6d and test_wrist6d_roundtrip (added there) into tests/unit/.

Test plan

Invocation Result
bare pytest from root 22 passed, 3 skipped, 0 collection errors
pytest tests/unit 22 passed
pytest --integration 23 passed, 2 skipped (openpi not checked out; S3RLDBDataset no longer exists)
pre-commit (ruff + ruff-format) passed

🤖 Generated with Claude Code

https://claude.ai/code/session_01N254Pmk9beYNooGou6T29R

RyanPCo commented Sep 12, 2026 •

Copy link
Copy Markdown
Contributor Author

@RyanPCo
RyanPCo force-pushed the ryanco/critique-docs branch from 2d43631 to 6cfda33 Compare September 12, 2026 21:28
@RyanPCo RyanPCo changed the title tests: move test files out of egomimic/ into tests/{unit,integration} tests: create dedicated test folder with unit and integration tests Sep 14, 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