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
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

swh/deposit/tests/api/test_deposit_status.py
198 ↗(On Diff #1162)

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.