test: make time clock QA phase-aware

[Clarion1631]

What changed

• Repaired W4.2 at e2e/qa-timeclock-expenses.spec.ts:24.
• The test now selects the seeded project, waits for its phase control, selects its loaded phase, and only then clicks Clock In.
• It proves a successful exact-path clock-in POST, Status: Clocked In, a three-second active interval, successful clock-out PUT, and Status: Clocked Out.
• It captures the created exact time-entry ID and deletes only that test record in afterEach, including after assertion failures after clock-in.

Why the test was wrong

The phase-only feature is correct. Its UI deliberately disables Clock In until a phase is selected. The old W4.2 test at e2e/qa-timeclock-expenses.spec.ts:24 selected only a project and clicked that disabled button. It timed out rather than exercising clock-in.

Verification

• npx eslint e2e/qa-timeclock-expenses.spec.ts — PASS
• git diff --check — PASS
• Isolated local PostgreSQL Docker E2E: npx playwright test e2e/qa-timeclock-expenses.spec.ts --project=chromium — 9 passed (41.3s)
• Exact W4.2 cleanup proof: 6 passed (26.8s); TimeEntry count stayed 2 before and after.
• GitHub required checks for this commit: Migrations, Build + Bundle Size, Playwright E2E Tests, and Vercel — SUCCESS.

Independent review

REVIEW: VERDICT: SHIP
Verdict artifact: C:\Users\jat00\AppData\Local\Temp\t_d9e260ce-review-verdict.md

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
probuild	Ready	Preview	Aug 29, 2026 5:39am