Page MenuHomeSoftware Heritage

Re-raise StorageArgumentException through API calls.
ClosedPublic

Authored by vlorentz on Feb 6 2020, 2:22 PM.

Details

Summary

So clients will get a nice exception looking like the
original one, instead of generic RemoteApiError.

Depends on D2628, D2627

Test Plan

No new test needed, because the current tests are broken by D2627, and this diff unbreaks them.

Diff Detail

Repository
rDSTO Storage manager
Branch
reraise-storageargumentexception
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 10542
Build 15727: tox-on-jenkinsJenkins
Build 15726: arc lint + arc unit

Event Timeline

vlorentz edited the test plan for this revision. (Show Details)
ardumont added a subscriber: ardumont.
ardumont added inline comments.
swh/storage/api/client.py
1

copyright update missing.

49

please, remove the print statement

This revision is now accepted and ready to land.Feb 6 2020, 2:31 PM

use last update in D2627 to simplify code