test(handler): fix test assertions and helper visibility - #335
Merged
Merged
Conversation
mroczect
commented
Aug 20, 2026
Owner
- test(handler): fix empty slice assertion
- test(handler): make test helpers public and add lint attributes
- test(handler): use first() instead of get(0)
mroczect
added a commit
that referenced
this pull request
Aug 21, 2026
…ocumentation (#341) * refactor(docs): refactor all doc, and make everything robust (#331) * chore(fmt): format all code * chore(sha512): update Cargo.lock for zeroize * feat(sha512): add zeroize dependency * feat(sha512): enable no_std support and clean up * refactor(sha512): use Zeroize and conditional no_std * feat(sha512): implement Zeroize for sha384 * feat(sha512): implement Zeroize and Drop for sha512 * fix(sha512): improve safety in utility functions * chore: update clippy lint priorities and package metadata * ci: improve Makefile targets and add strict clippy * fix(sha512): correct authors field and downgrade zeroize * test(sha512): fix benchmark unused results and add zeroize * test(sha512): fix benchmark unused results * feat(sha512): allow indexing slicing and improve safety * feat(sha512): allow indexing slicing and improve safety * refactor(sha512): add Debug impls, improve safety, and clean up * refactor(sha512): add Debug impl and improve safety * refactor(sha512): add Debug impl and improve safety * refactor(sha512): improve safety and use let-else * test(sha512): cover HKDF edge cases (#332) * test(sha512): cover HKDF edge cases * test(sha512): cover HMAC key handling * test(sha512): add SHA-384 vectors * test(sha512): add SHA-512 vectors * test(sha512): cover utility helpers * test(sha512): remove legacy integration tests * test(sha512): add integration test helper * test(sha512): add public API vectors * chore(workspace): update sha512 lock version (#333) * chore(workspace): update sha512 lock version * chore(libvctrl): update dependency versions * chore(core): update dependency versions * chore(plumbing): update libvctrl version * chore(sha512): bump crate version * test(workspace): add benchmarks and tests, improve validation and safety (#334) * chore(workspace): update Cargo.lock for handler benchmarks * chore(handler): add criterion dev-dependency and benchmark target * style(handler): remove unnecessary blank lines * style(handler): remove unnecessary blank lines * style(handler): remove unnecessary blank line * refactor(handler): use alloc and reorder imports * refactor(handler): add extern crate and test imports * style(handler): remove unnecessary blank lines * style(handler): reorder imports * style(handler): reorder imports * style(handler): reorder imports * refactor(handler): add Clone bound to Entry and clean up * style(handler): remove unnecessary blank lines * style(handler): reorder imports * style(handler): reorder imports * style(handler): remove unnecessary blank lines * style(handler): remove unnecessary blank lines * style(handler): reorder imports * refactor(handler): use HashSet for parent deduplication * refactor(handler): use specific imports and iter types * refactor(handler): reorder imports and use wrapping_add * style(handler): remove unnecessary blank line * fix(handler): improve duplicate detection in Tree * style(handler): remove unnecessary blank lines * fix(handler): validate ref name components more strictly * bench(handler): add handler benchmarks * test(handler): add blob tests * test(handler): add commit tests * test(handler): add commit_meta tests * test(handler): add common test utilities * test(handler): add delta tests * test(handler): add entry_kind tests * test(handler): add errors tests * test(handler): add hash tests * test(handler): add criterion import to hash_validation * test(handler): add criterion import to type_validation * test(handler): add tag_reflog tests * test(handler): add traits_index tests * test(handler): add tree tests * test(handler): add user_id tests * test(handler): add validation tests * test(handler): fix test assertions and helper visibility (#335) * test(handler): fix empty slice assertion * test(handler): make test helpers public and add lint attributes * test(handler): use first() instead of get(0) * test(core): add tests, refactor codec, and improve coverage (#336) * test(core): add comprehensive tests for binary decoder * test(core): add tests for binary encoder and fix clippy * test(core): add tests for sha512 hasher * style(core): add alloc extern and allow lint * test(core): add tests for blob builder * test(core): add tests for commit builder * test(core): add tests for tag builder * test(core): add tests for tree and tree entry builders * test(core): add tests for memory store * test(core): add tests for memory ref store * test(core): remove obsolete codec_test * test(core): remove obsolete store_test * test(core): add builder API integration tests * test(core): add codec roundtrip integration tests * test(core): add common test utilities * test(core): remove inline tests and obsolete integration tests (#338) * test(core): remove inline tests from binary_decoder * test(core): remove inline tests from binary_encoder * test(core): remove inline tests from sha512 * test(core): remove inline tests from blob * test(core): remove inline tests from commit * test(core): remove inline tests from tag * test(core): remove inline tests from tree * test(core): remove inline tests from memory store * test(core): remove inline tests from ref_store * test(core): remove builder_api integration tests * test(core): remove codec_roundtrip integration tests * test(core): remove common test utilities * test(core): add comprehensive unit and integration tests (#339) * test(core): rewrite binary_decoder tests with roundtrip checks * test(core): add binary_encoder tests and roundtrip checks * test(core): add sha512 hasher tests with known vectors * test(core): add blob builder tests * test(core): add commit builder tests * test(core): add tag builder tests * test(core): add tree and tree entry builder tests * test(core): add memory store tests * test(core): add memory ref store tests * test(core): add common test utilities * test(core): add builder integration tests * test(core): add codec integration roundtrip tests * test(core): add hasher integration test * test(core): add store integration tests * fix(plumbing): improve cat_file safety and error handling (#340) * fix(plumbing): improve cat_file safety and error handling * style(plumbing): add alloc extern * test(plumbing): update cat_file integration tests * ci(workflow): pin rust 1.96 and disable autocrlf (#343) * Delete a * test(root): add public API integration tests (#344) * style(root): remove unnecessary blank lines * test(root): add public API integration tests * chore: bump workspace crate versions (#345) * chore: update Cargo.lock for version bumps * chore(root): bump version to 2.2.0 * chore(core): bump version to 3.2.0 * chore(handler): bump version to 5.2.0 * chore(plumbing): bump version to 0.3.0 * chore(sha512): bump version to 3.2.0
mroczect
added a commit
that referenced
this pull request
Aug 21, 2026
* refactor(docs): refactor all doc, and make everything robust (#331) * chore(fmt): format all code * chore(sha512): update Cargo.lock for zeroize * feat(sha512): add zeroize dependency * feat(sha512): enable no_std support and clean up * refactor(sha512): use Zeroize and conditional no_std * feat(sha512): implement Zeroize for sha384 * feat(sha512): implement Zeroize and Drop for sha512 * fix(sha512): improve safety in utility functions * chore: update clippy lint priorities and package metadata * ci: improve Makefile targets and add strict clippy * fix(sha512): correct authors field and downgrade zeroize * test(sha512): fix benchmark unused results and add zeroize * test(sha512): fix benchmark unused results * feat(sha512): allow indexing slicing and improve safety * feat(sha512): allow indexing slicing and improve safety * refactor(sha512): add Debug impls, improve safety, and clean up * refactor(sha512): add Debug impl and improve safety * refactor(sha512): add Debug impl and improve safety * refactor(sha512): improve safety and use let-else * test(sha512): cover HKDF edge cases (#332) * test(sha512): cover HKDF edge cases * test(sha512): cover HMAC key handling * test(sha512): add SHA-384 vectors * test(sha512): add SHA-512 vectors * test(sha512): cover utility helpers * test(sha512): remove legacy integration tests * test(sha512): add integration test helper * test(sha512): add public API vectors * chore(workspace): update sha512 lock version (#333) * chore(workspace): update sha512 lock version * chore(libvctrl): update dependency versions * chore(core): update dependency versions * chore(plumbing): update libvctrl version * chore(sha512): bump crate version * test(workspace): add benchmarks and tests, improve validation and safety (#334) * chore(workspace): update Cargo.lock for handler benchmarks * chore(handler): add criterion dev-dependency and benchmark target * style(handler): remove unnecessary blank lines * style(handler): remove unnecessary blank lines * style(handler): remove unnecessary blank line * refactor(handler): use alloc and reorder imports * refactor(handler): add extern crate and test imports * style(handler): remove unnecessary blank lines * style(handler): reorder imports * style(handler): reorder imports * style(handler): reorder imports * refactor(handler): add Clone bound to Entry and clean up * style(handler): remove unnecessary blank lines * style(handler): reorder imports * style(handler): reorder imports * style(handler): remove unnecessary blank lines * style(handler): remove unnecessary blank lines * style(handler): reorder imports * refactor(handler): use HashSet for parent deduplication * refactor(handler): use specific imports and iter types * refactor(handler): reorder imports and use wrapping_add * style(handler): remove unnecessary blank line * fix(handler): improve duplicate detection in Tree * style(handler): remove unnecessary blank lines * fix(handler): validate ref name components more strictly * bench(handler): add handler benchmarks * test(handler): add blob tests * test(handler): add commit tests * test(handler): add commit_meta tests * test(handler): add common test utilities * test(handler): add delta tests * test(handler): add entry_kind tests * test(handler): add errors tests * test(handler): add hash tests * test(handler): add criterion import to hash_validation * test(handler): add criterion import to type_validation * test(handler): add tag_reflog tests * test(handler): add traits_index tests * test(handler): add tree tests * test(handler): add user_id tests * test(handler): add validation tests * test(handler): fix test assertions and helper visibility (#335) * test(handler): fix empty slice assertion * test(handler): make test helpers public and add lint attributes * test(handler): use first() instead of get(0) * test(core): add tests, refactor codec, and improve coverage (#336) * test(core): add comprehensive tests for binary decoder * test(core): add tests for binary encoder and fix clippy * test(core): add tests for sha512 hasher * style(core): add alloc extern and allow lint * test(core): add tests for blob builder * test(core): add tests for commit builder * test(core): add tests for tag builder * test(core): add tests for tree and tree entry builders * test(core): add tests for memory store * test(core): add tests for memory ref store * test(core): remove obsolete codec_test * test(core): remove obsolete store_test * test(core): add builder API integration tests * test(core): add codec roundtrip integration tests * test(core): add common test utilities * test(core): remove inline tests and obsolete integration tests (#338) * test(core): remove inline tests from binary_decoder * test(core): remove inline tests from binary_encoder * test(core): remove inline tests from sha512 * test(core): remove inline tests from blob * test(core): remove inline tests from commit * test(core): remove inline tests from tag * test(core): remove inline tests from tree * test(core): remove inline tests from memory store * test(core): remove inline tests from ref_store * test(core): remove builder_api integration tests * test(core): remove codec_roundtrip integration tests * test(core): remove common test utilities * test(core): add comprehensive unit and integration tests (#339) * test(core): rewrite binary_decoder tests with roundtrip checks * test(core): add binary_encoder tests and roundtrip checks * test(core): add sha512 hasher tests with known vectors * test(core): add blob builder tests * test(core): add commit builder tests * test(core): add tag builder tests * test(core): add tree and tree entry builder tests * test(core): add memory store tests * test(core): add memory ref store tests * test(core): add common test utilities * test(core): add builder integration tests * test(core): add codec integration roundtrip tests * test(core): add hasher integration test * test(core): add store integration tests * fix(plumbing): improve cat_file safety and error handling (#340) * fix(plumbing): improve cat_file safety and error handling * style(plumbing): add alloc extern * test(plumbing): update cat_file integration tests * ci(workflow): pin rust 1.96 and disable autocrlf (#343) * Delete a * test(root): add public API integration tests (#344) * style(root): remove unnecessary blank lines * test(root): add public API integration tests * chore: bump workspace crate versions (#345) * chore: update Cargo.lock for version bumps * chore(root): bump version to 2.2.0 * chore(core): bump version to 3.2.0 * chore(handler): bump version to 5.2.0 * chore(plumbing): bump version to 0.3.0 * chore(sha512): bump version to 3.2.0 * docs: update workspace README * docs(root): update facade README * docs(core): update core README * docs(handler): update handler README * docs(sha512): update sha512 README * docs: remove obsolete CONVENTIONS.md
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.