Skip to content

Add integration tests, lib refactor, MIT license, and CI#5

Open
Abhishek21g wants to merge 1 commit into
CognitionAI:mainfrom
Abhishek21g:fix/integration-tests-and-license
Open

Add integration tests, lib refactor, MIT license, and CI#5
Abhishek21g wants to merge 1 commit into
CognitionAI:mainfrom
Abhishek21g:fix/integration-tests-and-license

Conversation

@Abhishek21g

@Abhishek21g Abhishek21g commented Jul 11, 2026

Copy link
Copy Markdown

Problem

Fix

  • Extract BDiffHeader, format_size, and block_padding_size() into src/lib.rs with unit tests
  • Deduplicate padding via shared helper
  • Add Linux integration tests (tests/roundtrip.rs) for sparse compactify + base diff roundtrips
  • Add GitHub Actions CI (cargo test, cargo clippy -D warnings)
  • Add MIT LICENSE (matches sibling CognitionAI OSS repos)

Verification

cargo test --lib          # portable unit tests
cargo test                # full suite on Linux (ext4/xfs + fiemap)
cargo clippy -- -D warnings

Integration tests require Linux (fiemap, copy_file_range).

Fixes #3 (license). Does not duplicate open PR #2 (README blog link).


Devin Review

Status Commit
⚪ Not started

Run Devin Review

💡 Connect your GitHub account to enable automatic code reviews.

Open in Devin Review (Staging)

- Extract BDiffHeader/format helpers into testable lib module
- Deduplicate block padding calculation via block_padding_size()
- Add Linux roundtrip integration tests (create/apply)
- Add GitHub Actions CI (cargo test + clippy)
- Add MIT LICENSE (addresses CognitionAI#3)

Verified: cargo test --lib (unit); full suite requires Linux + fiemap.
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.

Please add an open source license?

1 participant