Skip to content

test(desktop): prune duplicate Electron journeys - #1936

Merged
jackwener merged 1 commit into
mainfrom
codex/slim-test-suite
Aug 3, 2026
Merged

test(desktop): prune duplicate Electron journeys#1936
jackwener merged 1 commit into
mainfrom
codex/slim-test-suite

Conversation

@jackwener

Copy link
Copy Markdown
Member

Summary

  • remove five low-signal Electron specs whose behavior is already owned by source contracts, Storybook, alignment smoke, or service/unit tests
  • collapse provider, floating-layer, settings, bot, quote, send, and sidebar coverage to representative cross-boundary journeys and failure paths
  • remove the orphaned Playwright permission fixture and stale references to deleted specs

Measured impact

  • Electron E2E: 91 to 58 tests, about 36% fewer app launches
  • E2E source: 4,920 to 2,866 lines, about 42% smaller
  • CI-like one-worker E2E: historical 6m19s to 2.8m on this worktree, about 56% faster

Verification

  • npm run build
  • npm run typecheck
  • npm run lint
  • npm run format:check
  • npm run test:scripts
  • 223 targeted replacement-layer unit/contract tests
  • full Electron E2E with 4 workers: 58 passed in 1.8m
  • full Electron E2E with CI=1: 58 passed in 2.8m

@jackwener
jackwener merged commit 5e782d9 into main Aug 3, 2026
1 check passed
@jackwener
jackwener deleted the codex/slim-test-suite branch August 3, 2026 05:56
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