test: improve coverage and exclude demo/bench crates from codecov #40
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add comprehensive integration tests for compression streaming (40 tests) - Add domain ports unit tests (64 tests) - Exclude pjs-bench and pjs-demo from coverage generation - Use --ignore-filename-regex to filter out excluded crates from reports Coverage improvement: 62.55% → 70.03% (excluding demo/bench) Eliminated zero-coverage modules: - stream/compression_integration.rs: 0% → 64.33% - domain/ports/mod.rs: 0% → 100% - domain/ports/writer.rs: 0% → 100%
Codecov Report✅ All modified and coverable lines are covered by tests. @@ Coverage Diff @@
## main #40 +/- ##
==========================================
+ Coverage 62.74% 68.57% +5.82%
==========================================
Files 68 68
Lines 16584 16584
==========================================
+ Hits 10406 11372 +966
+ Misses 6178 5212 -966
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
WASM Bundle Size Report
|
bug-ops
force-pushed
the
test/improve-coverage-exclude-demo-bench
branch
from
bf72871 to
681145a
Compare
WASM Bundle Size Report
|
Add 229 new tests to improve coverage: Parser tests (pjs-core): - parser_value_comprehensive.rs (65 tests): value.rs 30% → 86% - parser_simd_zero_copy_comprehensive.rs (84 tests): simd_zero_copy.rs 35% → 46% Domain tests (pjs-domain): - stream_comprehensive.rs (49 tests): stream.rs 57% → 87% - events_comprehensive.rs (31 tests): events/mod.rs 59% → 88% Coverage: 62.55% → 70.36% (+7.81pp) All 814 tests passing
bug-ops
force-pushed
the
test/improve-coverage-exclude-demo-bench
branch
from
681145a to
8f589d0
Compare
WASM Bundle Size Report
|
Add comprehensive tests for infrastructure/integration layer: - object_pool.rs: 0% → 60.74% (30 tests) - simd_acceleration.rs: 0% → 62.61% (24 tests) - streaming_adapter.rs: 0% → 44.89% (14 tests) - universal_adapter.rs: 0% → 48.60% (10 tests) Fix clippy warnings in existing tests: - Replace drop() with let _ = for non-Drop types - Replace match with if let for single patterns - Replace vec![] with array literals Coverage: 70.36% → 72.79% Total tests: 814 → 892
WASM Bundle Size Report
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
pjs-benchandpjs-demofrom codecov coverage reportsCoverage Improvement
Layer Coverage
New Test Files
pjs-core (7 files, 331 tests)
compression_streaming_integration.rsdomain_ports_test.rsparser_value_comprehensive.rsparser_simd_zero_copy_comprehensive.rsinfrastructure_integration_comprehensive.rspjs-domain (2 files, 80 tests)
stream_comprehensive.rsevents_comprehensive.rsKey Improvements
Previously 0% Coverage → Now Tested
object_pool.rs: 0% → 60.74%simd_acceleration.rs: 0% → 62.61%streaming_adapter.rs: 0% → 44.89%universal_adapter.rs: 0% → 48.60%stream/compression_integration.rs: 0% → 64%domain/ports/*: 0% → 100%Major Improvements
parser/value.rs: 30% → 86% (+56pp)entities/stream.rs: 57% → 87% (+30pp)events/mod.rs: 59% → 88% (+29pp)CI/CD Updates
pjs-benchandpjs-demofrom coverage generation (--exclude)--ignore-filename-regex)cargo llvm-cov nextestcommand syntaxTest Plan