Add unit tests for encryption functionality and integration tests#3
Merged
zombocoder merged 20 commits intomainfrom Sep 6, 2025
Merged
Add unit tests for encryption functionality and integration tests#3zombocoder merged 20 commits intomainfrom
zombocoder merged 20 commits intomainfrom
Conversation
- Added `test_encrypt.c` to implement unit tests for encryption support, key management, data encryption/decryption, and error handling. - Introduced `test_encrypt_integration.c` for integration tests focusing on encryption context lifecycle, key derivation edge cases, and large data encryption. - Updated `CMakeLists.txt` to include the new encryption test files and link against libsodium if enabled. - Temporarily disabled integration tests in `test_main.c` due to API mismatches.
…encryption key handling in bfc_writer.c
…mments for clarity
…r encryption functions
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.
This pull request adds encryption support to the project using libsodium (ChaCha20-Poly1305 AEAD with Argon2id key derivation), updates build and CI/CD pipelines to support this feature, and enhances documentation and packaging to reflect the new capabilities. The changes ensure encryption is available as an optional feature, with comprehensive testing and usage instructions.
Encryption Feature Integration:
-DBFC_WITH_SODIUM=ONinCMakeLists.txtand documented in theREADME.md. [1] [2] [3] [4] [5] [6] [7] [8]README.mdto highlight encryption, password/key file modes, and security considerations.Build System and CI/CD Updates:
CMakeLists.txt,Makefile) and CI/CD workflows (ci.yml,release.yml) to installlibsodiumand enable encryption-related build flags and tests across Linux and macOS. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10]Packaging Improvements:
libsodiumas a dependency.Benchmarks and Optional Dependency Handling:
benchmark_encrypt) and updated benchmark build logic to handle optional encryption and compression dependencies cleanly.These changes collectively introduce robust, configurable encryption support, ensure it's well-tested, and make it easy for users and downstream integrators to enable and use encryption features.