Skip to content

fix: enforce coverage thresholds + add node/web integration tests - #23

Merged
Lambertyan merged 2 commits into
masterfrom
fix-coverage-reports
Jul 3, 2026
Merged

fix: enforce coverage thresholds + add node/web integration tests#23
Lambertyan merged 2 commits into
masterfrom
fix-coverage-reports

Conversation

@Lambertyan

Copy link
Copy Markdown
Contributor

Summary

  • Remove all continue-on-error bandaids from CI workflows
  • Add vitest coverage thresholds (>=95% lines/functions/statements, >=90% branches)
  • Add integration tests for embed-code-node (7 tests) and embed-code-web (5 tests) with real ONNX model
  • Add ort-backend unit tests (5 tests)
  • Extend coverage to embed-code-web/src
  • Exclude interface files from coverage
  • Add file:// protocol support in WebEmbedder.create()
  • Overall coverage: Lines 97.11%, Statements 96.52%, Functions 98.48%, Branches 93.33%

- Remove all continue-on-error from ci.yml, release.yml
- Add vitest coverage thresholds (>=95% lines/functions/statements, >=90% branches)
- Add ci-coverage-check.js to integration-test job
- Add 13 unit tests to close coverage gaps (batch-processor, pooling, pre-tokenizer, vocab-loader, tokenizer)
- Fix release.yml benchmark output paths
- Exclude docs/ from prettier
@Lambertyan
Lambertyan force-pushed the fix-coverage-reports branch from aeb916d to dac1631 Compare July 3, 2026 11:11
…th real ONNX model

- Add 7 tests for NodeEmbedder (create, embed, embedBatch, dispose, modelInfo)
- Add 5 tests for WebEmbedder (embed, embedBatch, dispose, modelInfo) via WASM
- Add 5 tests for ort-backend (createTensor, dispose)
- Extend vitest.config.ts coverage include to embed-code-web/src
- Exclude embedder-interface.ts and ort-backend-interface.ts from coverage
- Add file:// protocol support in WebEmbedder.create() for Node.js testing
- Extend descriptor tests to cover weights field and missing weights/onnx branches
- Overall coverage: Lines 97.11%, Statements 96.52%, Functions 98.48%, Branches 93.33%
@Lambertyan
Lambertyan force-pushed the fix-coverage-reports branch from dac1631 to 5add8d1 Compare July 3, 2026 11:12
@Lambertyan
Lambertyan merged commit dab9a44 into master Jul 3, 2026
7 checks passed
@Lambertyan
Lambertyan deleted the fix-coverage-reports branch July 3, 2026 11:18
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