test: remove tests directory

[celve]

Summary

Removes the top-level tests/ directory (4 files, 330 lines).

Nothing in the repo wires this directory up: there is no [tool.pytest.ini_options]
or testpaths in pyproject.toml, no conftest.py, and no CI workflow that invokes
pytest. The suite is therefore never executed by any automated gate — it only runs if
a contributor happens to type pytest locally, which makes it silent, unenforced
weight that drifts out of step with the code it nominally covers.

This follows the precedent set by #99 (remove tests directory) and the earlier
test: drop VeOmni/meta-init test suite (validate via e2e experiments): this repo
validates behaviour through end-to-end training and rollout experiments rather than an
unenforced unit-test tree.

Removed:

• tests/types/test_advantages_gae.py (from feat(types): add GAE advantage computation #254)
• tests/benchmarks/test_geneval2_reproduction.py (from feat(benchmark/eval): GenEval2 DPPO reproduction (SD3.5) #236)
• tests/sft/test_agent_sft.py (from feat(sft): support agent conversation fine-tuning #234)
• tests/sde/test_prompt_seed.py

Related Issue

N/A

Test Plan

• SKIP=no-commit-to-branch pre-commit run --all-files --show-diff-on-failure — all 16
  hooks pass (check-ast, ruff check, ruff format, check-recipe-targets, ...).
• Verified no remaining references to the removed modules:
  grep -rn "test_advantages_gae\|test_prompt_seed\|test_geneval2_reproduction\|test_agent_sft" . — no hits.
• Verified nothing imports from the package: no from tests/import tests anywhere,
  and no conftest.py in the tree.
• No pytest invocation exists in .github/workflows/, so no CI job changes behaviour.

Compatibility / Risk

Low. Deletion only — no source, config, recipe, checkpoint, data format, or API change.
The dev extra still declares pytest/pytest-cov, which stays useful for ad-hoc local
testing and for the vendored unirl-reward-service/ sub-project, so pyproject.toml is
untouched.

Reviewer Notes

• Interaction with open PRs. feat(algorithms): add AR PPO with GAE and value loss #259 (tests/algorithms/test_ppo_value_loss.py) and
  feat(agentic): add Sample-native multi-turn rollout and training #214 (7 files under tests/) each add new files under tests/. None of them touch
  the 4 files removed here, so there is no merge conflict either way — but whichever of
  those lands after this PR will recreate a tests/ directory. If the intent is for the
  tree to stay absent, those authors should be asked to drop their test files as part of
  their own review, otherwise the directory returns exactly as it did after test: remove tests directory #99.
• Pre-existing stale reference, deliberately left alone. pyproject.toml:87 says the
  VeOmni shim's import closure is audited by tests/test_compat_import.py. That file was
  already deleted earlier (test: drop VeOmni/meta-init test suite), so the comment was
  dangling before this change and is equally dangling after. Flagging rather than fixing
  it, to keep this diff purely a deletion — happy to correct the comment here if you'd
  prefer it bundled.
• AI assistance was used to prepare this change; the diff is a pure deletion and has been
  reviewed in full.
• Duplicate-work check: no open PR removes tests/.

Checklist

• I reviewed the changed code and removed unrelated/generated artifacts.
• I updated tests, docs, and configs where needed, or explained why not.

[Copilot]

Pull request overview

This PR removes the top-level tests/ directory, aligning the repo with its current validation approach (CI runs pre-commit only, with no pytest execution wired in) and matching the earlier precedent of eliminating an unenforced root test tree.

Changes:

• Delete CPU unit tests for compute_gae_advantages (GAE).
• Delete unit tests covering GenEval2 reproduction helpers and prompt-seed determinism.
• Delete unit tests covering agent SFT preparation/tokenization behavior.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File	Description
tests/types/test_advantages_gae.py	Removes GAE advantage computation unit tests.
tests/benchmarks/test_geneval2_reproduction.py	Removes GenEval2 reproduction helper unit tests.
tests/sft/test_agent_sft.py	Removes agent SFT preparation/tokenization unit tests.
tests/sde/test_prompt_seed.py	Removes prompt-seed determinism unit test.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.