Skip to content

test(daemon): expand src/core unit coverage - #108

Merged
sudhirverma merged 2 commits into
mainfrom
improve-daemon-core-unit-tests
Aug 11, 2026
Merged

test(daemon): expand src/core unit coverage#108
sudhirverma merged 2 commits into
mainfrom
improve-daemon-core-unit-tests

Conversation

@sudhirverma

Copy link
Copy Markdown
Contributor

Summary

  • Expand CoreState unit tests (builder API, credentials, list/chat/health paths) in packages/daemon/src/core/state.test.ts
  • Expand engines.ts coverage for registry accessors, fetchModels provider paths, and streamChat error/stream mapping
  • Add config helper and MemorySecretStore tests in existing core test files

Coverage impact (src/core):

  • Overall: 56% → 83% statements
  • state.ts: 14% → 80%
  • engines.ts: 42% → 71%
  • config.ts: 72% → 94%

Commits

  • test(daemon): expand src/core unit coverage for CoreState, engines, and config — adds/extends core unit tests without new duplicate test files

Test plan

  • npx vitest run src/core — 307 passed
  • npm run lint -w packages/daemon
  • npm test (full workspace)
  • npm run ci:build

…nd config

Improve coverage across core modules by extending existing test files
instead of adding duplicates. CoreState, engines registry/discovery,
streamChat paths, config helpers, and MemorySecretStore are now covered.
@github-actions github-actions Bot added the chore label Aug 10, 2026
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.70%. Comparing base (9d26eb2) to head (68af5df).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #108      +/-   ##
==========================================
+ Coverage   60.12%   63.70%   +3.57%     
==========================================
  Files          37       37              
  Lines        4973     4973              
  Branches     1568     1568              
==========================================
+ Hits         2990     3168     +178     
+ Misses       1462     1262     -200     
- Partials      521      543      +22     
Flag Coverage Δ
daemon 63.70% <ø> (+3.57%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@sudhirverma
sudhirverma merged commit 320ab0a into main Aug 11, 2026
8 checks passed
@sudhirverma
sudhirverma deleted the improve-daemon-core-unit-tests branch August 11, 2026 03:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants