Fix deposit atom test to actually check the response message
45c7c35e471c
Actions

Description

Fix deposit atom test to actually check the response message

Prior to this commit, the actual test was wrongfully ok because the slug was
missing. This resulted in a 400 response which passed because we missed
checking the actual response content. This now fixes it.

Details

Provenance

ardumont	Authored on Sep 26 2020, 6:37 PM
ardumont	Pushed on Sep 28 2020, 1:07 PM

Differential Revision

D4057: Fix deposit atom test to actually check the response message

Parents

rDDEPb04c457d6568: Fix transition deposit to deposited state with empty post on SE IRI

Branches

Unknown

Tags

Unknown

Build Status

Buildable 15626
Build 24062: test-and-build	Jenkins console · Jenkins

Event Timeline

ardumont added an edge: D4057: Fix deposit atom test to actually check the response message.Sep 28 2020, 1:07 PM

ardumont committed rDDEP45c7c35e471c: Fix deposit atom test to actually check the response message (authored by ardumont).Sep 26 2020, 7:09 PM

Harbormaster completed building B15626: rDDEP45c7c35e471c: Fix deposit atom test to actually check the response message.Sep 28 2020, 1:09 PM

swh-public-ci mentioned this in D4013: Allow deposit metadata update on deposit already complete.Sep 28 2020, 1:10 PM

swh-public-ci mentioned this in D4047: Add functional metadata checks prior to actually update metadata.Sep 28 2020, 1:13 PM

swh-public-ci mentioned this in D4058: Check updated metadata are correctly stored in the metadata storage.Sep 28 2020, 1:15 PM

Changes (2)

Path

Size

swh/

deposit/

tests/

api/

test_deposit_atom.py

data/

atom/

entry-data-empty-body.xml

rDDEP45c7c35e471c

View Options

swh/deposit/tests/api/test_deposit_atom.py