TEST-UNIFY-3: specs into domain folders + playwright config rewrite (0 collection errors, gate green)#124
Merged
Merged
Conversation
…ht config (Phase 3)
Clean tree: tests/e2e/{smoke,auth,graph,ui,a11y,mobile,responsive,api,admin,reports}/,
tests/diagnostics/{layout,interactions,hierarchy,inspector,physics,perf,ui}/, and
infra specs to tests/integration/. Specs renamed for clarity (login-focus→auth/
passwordless-focus, copy-id→graph/node-inspector-copyid, visual-vlm→reports/
vlm-evaluation, etc.).
Config + refs:
- playwright.config.ts: default project testIgnore → '**/reports/**' (capture-heavy
report specs run in their own projects; integration/ is outside testDir); vlm
testMatch → vlm-evaluation.spec.ts. showcase/matrix/matrix-setup testMatch match
by filename (unchanged).
- repointed test:smoke + test:e2e:visual, the unified sequence manifest, and CI
ci.yml living-graph path to the new locations.
Caught + fixed the plan's #1 risk: moving specs into subfolders changed their depth,
so the Phase-2 '../lib/' imports broke (depth-2 specs need '../../lib/') — the
verification showed 0 tests / collection errors; fixed 37 files; re-verified.
Verified: per-project playwright --list all 0 collection errors (default 117, perf 3,
geometry 18, showcase 5, vlm 1, matrix 109, matrix-setup 1); reports specs excluded
from the default project; THE GATE 5/5; unified smoke profile 212✓.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
🧪 Comprehensive Test Suite
Full-stack smoke gate runs in the CI workflow. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Phase 3 of the clean-tree refactor. Specs grouped into tests/e2e//, tests/diagnostics//, tests/integration/ (with the plan's renames). Rewrote playwright.config.ts (default testIgnore '/reports/', vlm testMatch rename), repointed test:smoke/test:e2e:visual + unified manifest + CI living-graph path. Caught+fixed the depth-change import break (37 files → ../../lib/). Verified: per-project --list 0 collection errors (default 117/perf 3/geometry 18/showcase 5/vlm 1/matrix 109/matrix-setup 1); THE GATE 5/5; unified smoke 212✓.
🤖 Generated with Claude Code