Skip to content

test(bytestring): expand test coverage - #947

Merged
robjtede merged 1 commit into
mainfrom
bytestring-test-coverage
Aug 26, 2026
Merged

test(bytestring): expand test coverage#947
robjtede merged 1 commit into
mainfrom
bytestring-test-coverage

Conversation

@robjtede

Copy link
Copy Markdown
Member

Summary

  • add coverage for ByteString::into_bytes
  • exercise empty, Unicode, and invalid UTF-8 conversion paths
  • cover owned and borrowed byte-array conversions

Tests

  • cargo llvm-cov --package bytestring --all-features --summary-only — 100% regions, functions, and lines
  • cargo llvm-cov --package bytestring --no-default-features --summary-only — 100% regions, functions, and lines
  • cargo test --package bytestring --all-features
  • cargo +nightly fmt --package bytestring -- --check

The repository-wide just check remains blocked by existing formatting errors in deny.toml and actix-tls/Cargo.toml.

@robjtede
robjtede force-pushed the bytestring-test-coverage branch from ee72f11 to 464d73d Compare August 26, 2026 11:38
@robjtede
robjtede merged commit 3de9367 into main Aug 26, 2026
18 checks passed
@robjtede
robjtede deleted the bytestring-test-coverage branch August 26, 2026 23:49
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