Skip to content

Focus PR verification and remove low-value tests - #449

Merged
theogravity merged 5 commits into
mainfrom
perf/focused-tests
Oct 26, 2026
Merged

theogravity merged 5 commits into
mainfrom
perf/focused-tests

Conversation

@theogravity

@theogravity theogravity commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Remove low-value copy, SEO metadata, and duplicate assertions while preserving behavior and correctness checks.
  • Update testing guidance to prioritize expected behavior and critical correctness over coverage counts.
  • Add change-aware PR test planning and keep full-suite verification for releases.
  • Fix the installer CLI e2e script to ignore an inherited SUBSHELL_DATA_DIR when it creates an isolated fake home.

Verification

  • Unit tests: 12,543 passed, 5 skipped.
  • Script tests: 335 passed, 3 skipped.
  • Build, type checks, lint, and Rust fmt/Clippy/tests passed.
  • Playwright e2e after fixture fixes: 87 passed, 5 skipped, 0 failed.
  • Compiled CLI integration, including installer and server/node update scenarios, passed on the clean CI runner.

CI follow-up fixes prepare website data before type checking, build the SPA before server tests, isolate server test files, and validate the update backup archive instead of obsolete console text. Workflow-planning tests: 18 passed. All checks passed on commit 5f15feb, including browser/CLI integration, all unit lanes, build/types/lint, Linux/macOS Rust, and CodeQL. Browser tests now identify their launched pane from its actual WebSocket connection to avoid workspace navigation and pane-order races.

@theogravity
theogravity merged commit c44aec2 into main Oct 26, 2026
14 checks passed
@theogravity
theogravity deleted the perf/focused-tests branch October 26, 2026 06:49
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 26, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant