Page MenuHomeSoftware Heritage

api.status: Open detail status on rejected deposit for client
ClosedPublic

Authored by ardumont on Jul 10 2018, 3:59 PM.

Details

Summary
  • deposit_status:

So far, the status detail was generic. It still is except for the rejected deposit.
We now provide the detailed explanation of the failed checks in the status endpoint.

  • deposit_check: Fix url_fields beeing too wildly populated

We were adding all tested fields tested while we wanted only to keep
the ones matching 'url'.

Test Plan

Tests ok

Diff Detail

Repository
rDDEP Push deposit
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 1304
Build 1648: arc lint + arc unit

Event Timeline

swh/deposit/tests/api/test_deposit_status.py
198

Naturally, i forgot this one...

Add missing test on deposit status endpoint

  • api.status: Open detail status on rejected deposit for client
  • deposit_check: Fix url_fields beeing too wildly populated
This revision was not accepted when it landed; it landed in state Needs Review.Jul 11 2018, 11:46 AM
This revision was automatically updated to reflect the committed changes.