Skip to content

Conversation

@bug-ops
Copy link
Owner

@bug-ops bug-ops commented Dec 4, 2025

Summary

  • Add comprehensive tests for remaining coverage gaps
  • Fix test assertions to match actual implementation behavior

Test Files Added/Modified

File Tests Module
object_pool_comprehensive.rs 52 object_pool.rs (50% → 60%+)
streaming_comprehensive.rs (pjs-wasm) 52 streaming.rs (54% → 60%+)
Extended optimization_service_comprehensive.rs +20 optimization_service.rs (55% → 70%+)
Extended parser_simd_zero_copy_comprehensive.rs +24 simd_zero_copy.rs (43% → 60%+)

Fixes

  • Fixed Priority constant values (CRITICAL=100, HIGH=80, MEDIUM=50, LOW=25)
  • Fixed SecurityConfig default max_depth (64, not 32)
  • Fixed zero size limit behavior (ignored, uses default)
  • Fixed ObjectPool zero capacity test (should_panic)
  • Fixed Frame::patch tests to use valid non-empty patches

Test plan

  • All 2026 tests pass locally
  • Zero clippy warnings
  • Code formatted with nightly
  • CI tests pass on Ubuntu, macOS, Windows

@github-actions
Copy link

github-actions bot commented Dec 4, 2025

WASM Bundle Size Report

Target Raw (KB) Gzipped (KB) Status
web 155 70 ✓ PASS
nodejs 156 70 ✓ PASS
bundler 155 70 ✓ PASS

@codecov-commenter
Copy link

codecov-commenter commented Dec 4, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #48      +/-   ##
==========================================
+ Coverage   76.48%   77.18%   +0.70%     
==========================================
  Files          68       68              
  Lines       16584    16584              
==========================================
+ Hits        12684    12801     +117     
+ Misses       3900     3783     -117     
Flag Coverage Δ
unittests 77.18% <ø> (+0.70%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 3 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a48decb...357a3e2. Read the comment docs.

🚀 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.

Add comprehensive tests for remaining coverage gaps:

- object_pool_comprehensive.rs (52 tests)
  - ObjectPool creation and lifecycle
  - PooledObject operations
  - Global pools (cow_hashmap, string_vec, etc.)
  - Concurrent access patterns

- streaming_comprehensive.rs (pjs-wasm, 52 tests)
  - SecurityConfig configuration
  - Input validation
  - JsonData conversions
  - Priority value tests

- Extended optimization_service_comprehensive.rs
  - Calculate optimization metrics scenarios
  - Resource utilization with compression
  - Multiple optimization passes

- Extended parser_simd_zero_copy_comprehensive.rs
  - Additional error handling
  - Buffer management edge cases
  - Deep nesting scenarios

Total tests: 2026 (all passing)
@bug-ops bug-ops force-pushed the test/remaining-coverage-gaps branch from 3481e30 to 357a3e2 Compare December 4, 2025 22:56
@github-actions
Copy link

github-actions bot commented Dec 4, 2025

WASM Bundle Size Report

Target Raw (KB) Gzipped (KB) Status
web 155 70 ✓ PASS
nodejs 156 70 ✓ PASS
bundler 155 70 ✓ PASS

@bug-ops bug-ops merged commit 22789b1 into main Dec 4, 2025
29 checks passed
@bug-ops bug-ops deleted the test/remaining-coverage-gaps branch December 4, 2025 23:02
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.

3 participants