lsm / neokai / 23421610737

88%

dev: 84%

Pull #749

github

test: address E2E review feedback for TaskViewV2 tests

- P0: Remove 4 hollow slide-out tests that always took early return path
  (blockCount === 0) without exercising claimed open/close behavior. Replace
  with 4 concrete tests: panel off-screen (translate-x-full class), backdrop
  transparent (opacity-0 class), accessibility attrs, close button aria-label.
  Add NOTE comment explaining FK constraint prevents agent-turn seeding.

- P1: Fix vacuous toggle text assertions. Both V1 and V2 states contain 'V1'
  and 'V2' in the button. Use '←' (V2 active) / '→' (V1 active) as the
  discriminating directional arrow characters.

- P1: Move createRoomAndTask/createRoomTaskAndGroup calls from test bodies
  into beforeEach blocks per CLAUDE.md E2E infrastructure rule.

- P2: Replace toBeInViewport() with toHaveClass(/translate-x-full/) for
  closed-state assertions — directly tests the CSS mechanism, not viewport
  intersection which is viewport-size sensitive.

- P3: Move slide-out panel setup to beforeEach; reduce boilerplate in
  test bodies.

5377 of 6809 branches covered (78.97%)

Branch coverage included in aggregate %.

43321 of 48789 relevant lines covered (88.79%)

233.76 hits per line