Differential D5098

Add basic tests for the provenanceDB and archive abstraction classes
ClosedPublic
Actions

Authored by douardda on Feb 17 2021, 5:36 PM.

Details

Reviewers

aeviso

Commits

rDPROV7b12ed0b3726: Add basic tests for the provenanceDB and archive abstraction classes

Summary

Depends on D5083.

Diff Detail

Repository

rDPROV Provenance database

Branch

refactorings

Lint

Lint Skipped

Unit

Unit Tests Skipped

Build Status

Buildable 19358
Build 30016: arc lint + arc unit

Event Timeline

douardda requested review of this revision.Feb 17 2021, 5:36 PM

douardda created this revision.

Harbormaster completed remote builds in B19304: Diff 18205.Feb 17 2021, 5:36 PM

douardda added a child revision: D5099: Make ArchiveStorage() accept a StorageInterface object as argument.Feb 17 2021, 5:37 PM

I don't really understand what's going on here, so there is no much I can review.

The swh_storage_with_objects fixtures returns a (postgresal-based) storage with some objects in it (only a few of each type). The test that given storage do indeed have at least a content, a directory and a revision. I'll improve the commit message a bit.

Rebase + better commit message + add a docstring

Harbormaster completed remote builds in B19358: Diff 18264.Feb 18 2021, 5:57 PM

aeviso accepted this revision.Feb 18 2021, 7:30 PM

This revision is now accepted and ready to land.Feb 18 2021, 7:30 PM

Closed by commit rDPROV7b12ed0b3726: Add basic tests for the provenanceDB and archive abstraction classes (authored by douardda). · Explain WhyFeb 19 2021, 10:01 AM

This revision was automatically updated to reflect the committed changes.

douardda added a commit: rDPROV7b12ed0b3726: Add basic tests for the provenanceDB and archive abstraction classes.