Page MenuHomeSoftware Heritage

Add test for DirectoryBuilder on missing directories
AcceptedPublic

Authored by vlorentz on Thu, Jun 23, 10:19 AM.

Details

Reviewers
ardumont
Group Reviewers
Reviewers
Summary

Just to make sure they don't crash like contents currently do (T4345)

Diff Detail

Event Timeline

Build is green

Patch application report for D8031 (id=28919)

Could not rebase; Attempt merge onto 6da99426e2...

Updating 6da9942..c07d934
Fast-forward
 swh/vault/tests/test_to_disk.py | 60 +++++++++++++++++++++++++++++++++++++----
 swh/vault/to_disk.py            | 17 ++++++++----
 2 files changed, 67 insertions(+), 10 deletions(-)
Changes applied before test
commit c07d934aeff664c2e4b74818133344576d99b23f
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Jun 23 10:18:59 2022 +0200

    Add test for DirectoryBuilder on missing directories

commit 141b4531b1a6924806787252db15172422553c04
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Jun 23 10:04:16 2022 +0200

    Fix crash on directories pointing to missing contents
    
    This can happen for various reasons, such as running the vault on
    a mirror that is partially replayed

See https://jenkins.softwareheritage.org/job/DVAU/job/tests-on-diff/241/ for more details.

This revision is now accepted and ready to land.Fri, Jun 24, 11:24 AM