Skip to content

Add tests data related to containers (EWF, VDI, VHD, VHDX and VMDK)#1495

Merged
Schamper merged 27 commits intofox-it:mainfrom
william-billaud:add_test_ewf
Feb 11, 2026
Merged

Add tests data related to containers (EWF, VDI, VHD, VHDX and VMDK)#1495
Schamper merged 27 commits intofox-it:mainfrom
william-billaud:add_test_ewf

Conversation

@william-billaud
Copy link
Contributor

@william-billaud william-billaud commented Jan 14, 2026

Add tests related to multiple containers format. I do not have any mac to generate tests data for mac related containers format.

Add ~1.5mo of git lfs data.

For each test, I have tried to cover most path/functions available from the container (seek, read etc...)

Bump dissect.evidence dev version.

Related to fox-it/dissect.evidence#47

@william-billaud william-billaud marked this pull request as draft January 14, 2026 12:37
@william-billaud
Copy link
Contributor Author

william-billaud commented Jan 14, 2026

It look like their is more untested container format, I will add test for them to in this PR (or in dedicated PR, as you wish).

william-billaud and others added 2 commits January 14, 2026 13:38
Co-authored-by: Erik Schamper <1254028+Schamper@users.noreply.github.com>
Co-authored-by: Erik Schamper <1254028+Schamper@users.noreply.github.com>
@Schamper
Copy link
Member

It look like their is more untested container format, I will add test for them to in this PR (or in dedicated PR, as you wish).

This PR is fine, just rename the PR title.

@william-billaud william-billaud changed the title Add tests related to ewf Add tests data related to containers (EWF, VDI, VHD, VHDX and VMDK) Jan 14, 2026
@william-billaud william-billaud marked this pull request as ready for review January 14, 2026 14:16
@william-billaud
Copy link
Contributor Author

@Schamper any updates/needs on this PR?
I just saw PR #1531 and this PR add code coverage related to changed code.

@Schamper
Copy link
Member

@Schamper any updates/needs on this PR? I just saw PR #1531 and this PR add code coverage related to changed code.

Just small text changes. Thanks!

william-billaud and others added 3 commits February 10, 2026 17:17
Co-authored-by: Erik Schamper <1254028+Schamper@users.noreply.github.com>
def test_ewf_container_splitted() -> None:
"""Test that EWF containers are properly opened when container is split among multiple files.

```
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you do the same everywhere else?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, missed this one.

I will write 1 000 times I will not submit review just before leaving work as a reminder.

@codecov
Copy link

codecov bot commented Feb 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.91%. Comparing base (3208a8f) to head (d1b174a).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1495      +/-   ##
==========================================
+ Coverage   80.80%   80.91%   +0.10%     
==========================================
  Files         396      396              
  Lines       34827    34827              
==========================================
+ Hits        28142    28180      +38     
+ Misses       6685     6647      -38     
Flag Coverage Δ
unittests 80.91% <ø> (+0.10%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Schamper Schamper merged commit d9a1481 into fox-it:main Feb 11, 2026
16 of 21 checks passed
@codspeed-hq
Copy link

codspeed-hq bot commented Feb 11, 2026

Merging this PR will not alter performance

✅ 11 untouched benchmarks


Comparing william-billaud:add_test_ewf (cf26576) with main (3208a8f)

Open in CodSpeed

@william-billaud william-billaud deleted the add_test_ewf branch February 11, 2026 12:08
Matthijsy pushed a commit to Matthijsy/dissect.target that referenced this pull request Mar 2, 2026
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.

Add test data related to E01

2 participants