Page MenuHomeSoftware Heritage

Add a 'reset' endpoint to the in-mem storage.
ClosedPublic

Authored by vlorentz on Jun 28 2019, 8:34 PM.

Details

Summary

It will be used for API client/server property tests,
because they need to remotely reset the state and cannot
create a new storage.

Diff Detail

Repository
rDSTO Storage manager
Branch
inmem-reset
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 6606
Build 9220: tox-on-jenkinsJenkins
Build 9219: arc lint + arc unit

Event Timeline

I may have missed it, but where is this specific endpoint used? I cannot find any usage in following diffs... Why is this "cannot create a new storage" statement true?

I may have missed it, but where is this specific endpoint used? I cannot find any usage in following diffs...

D1664, ctrl-f .reset()

Why is this "cannot create a new storage" statement true?

Because the storage does not run in the same process as the tests.

I may have missed it, but where is this specific endpoint used? I cannot find any usage in following diffs...

D1664, ctrl-f .reset()

I saw this but wasn't sure it was the same exposed reset() function.

Why is this "cannot create a new storage" statement true?

Because the storage does not run in the same process as the tests.

humm ok then.

This revision is now accepted and ready to land.Jul 1 2019, 2:12 PM
This revision was landed with ongoing or failed builds.Jul 1 2019, 3:50 PM
This revision was automatically updated to reflect the committed changes.