Skip to content

[Tests] - Improve coverage for core workflows#1

Merged
maatheusgois-dd merged 4 commits into
mainfrom
maatheusgois-dd/improve-test-coverage
May 15, 2026
Merged

[Tests] - Improve coverage for core workflows#1
maatheusgois-dd merged 4 commits into
mainfrom
maatheusgois-dd/improve-test-coverage

Conversation

@maatheusgois-dd

@maatheusgois-dd maatheusgois-dd commented May 15, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Expands unit coverage for core command wrappers, runtime config, daemon helpers, LLDB sessions, Swift Package commands, simulator/device flows, and UI automation helpers.
  • Replaces the redundant MCP startup smoke test with targeted coverage across the underlying registry and helper layers.
  • Raises per-file statement coverage thresholds for covered core/runtime/helper modules to 80%.

Test plan

  • npm run lint
  • npm run typecheck
  • npm test
  • npx vitest --run --coverage

Notes

  • .github/PULL_REQUEST_TEMPLATE.md is not present in this repository, so this body uses the requested summary/test-plan context directly.
  • Overall coverage is currently 54.08%; CLI/MCP/handler coverage remains follow-up work before a global 80% threshold can be enabled.

Expand unit coverage around command wrappers and runtime helpers so regressions are caught without invoking real developer tooling.

Co-authored-by: Cursor <cursoragent@cursor.com>
@maatheusgois-dd maatheusgois-dd self-assigned this May 15, 2026
Comment thread src/tools/helpers.test.ts Fixed
Comment thread src/tools/helpers.test.ts Fixed
maatheusgois-dd and others added 3 commits May 15, 2026 10:56
Remove unused imports and consume streaming output so static analysis stays clean.

Co-authored-by: Cursor <cursoragent@cursor.com>
Import StreamChunk from the process utility module so CI typecheck can resolve the test type.

Co-authored-by: Cursor <cursoragent@cursor.com>
Raise thresholds for covered core files without failing CI on areas that still need follow-up coverage.

Co-authored-by: Cursor <cursoragent@cursor.com>
@codecov

codecov Bot commented May 15, 2026

Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

ℹ️ You can also turn on project coverage checks and project coverage reporting on Pull Request comment

Thanks for integrating Codecov - We've got you covered ☂️

@maatheusgois-dd
maatheusgois-dd marked this pull request as ready for review May 15, 2026 14:20
@maatheusgois-dd
maatheusgois-dd merged commit 8a48a40 into main May 15, 2026
6 checks passed
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