Page MenuHomeSoftware Heritage

tests: Ensure all empty body test cases are covered
ClosedPublic

Authored by ardumont on Sep 28 2020, 4:18 PM.

Details

Summary

As promised in D4013

Test Plan

tox

Diff Detail

Repository
rDDEP Push deposit
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 15655
Build 24103: Phabricator diff pipeline on jenkinsJenkins console · Jenkins
Build 24102: arc lint + arc unit

Event Timeline

Build is green

Patch application report for D4067 (id=14359)

Could not rebase; Attempt merge onto 87b39c10ee...

Updating 87b39c10..73120b72
Fast-forward
 swh/deposit/api/checks.py                            |  2 ++
 swh/deposit/parsers.py                               | 11 ++++++++++-
 swh/deposit/tests/api/test_deposit_atom.py           | 18 ++++++++++--------
 .../tests/api/test_deposit_private_read_metadata.py  |  8 --------
 swh/deposit/tests/api/test_deposit_update.py         | 20 +++++++++++---------
 swh/deposit/tests/api/test_parser.py                 |  4 ----
 .../data/atom/entry-data-empty-body-no-namespace.xml |  2 ++
 .../tests/data/atom/entry-data-empty-body.xml        |  2 +-
 swh/deposit/tests/data/atom/entry-data3.xml          |  2 +-
 9 files changed, 37 insertions(+), 32 deletions(-)
 create mode 100644 swh/deposit/tests/data/atom/entry-data-empty-body-no-namespace.xml
Changes applied before test
commit 73120b72171c1f8884ab5d63da8fe0a7e25ea391
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Mon Sep 28 16:18:37 2020 +0200

    tests: Ensure all empty body test cases are covered

commit d86c192234a025a027202379789e074441dadbc3
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Mon Sep 28 16:09:59 2020 +0200

    deposit.parsers: Process namespace when using xmltodict.parse
    
    This keeps the existing behavior though by mapping the expanded name to simpler
    namespaces.
    
    This also fixes some test data sample which were not properly defining their
    namespace.
    
    This will allow to deal more properly the empty body case which is incomplete
    right now.

commit cc8d12524845811345735a00e836f29dea8fac8c
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Mon Sep 28 14:15:48 2020 +0200

    deposit.api.checks: Add docstring to module

See https://jenkins.softwareheritage.org/job/DDEP/job/tests-on-diff/155/ for more details.

This revision is now accepted and ready to land.Sep 28 2020, 5:35 PM

Build is green

Patch application report for D4067 (id=14363)

Could not rebase; Attempt merge onto 87b39c10ee...

Updating 87b39c10..b15033d6
Fast-forward
 swh/deposit/parsers.py                               | 11 ++++++++++-
 swh/deposit/tests/api/test_deposit_atom.py           | 18 ++++++++++--------
 .../tests/api/test_deposit_private_read_metadata.py  |  8 --------
 swh/deposit/tests/api/test_deposit_update.py         | 20 +++++++++++---------
 swh/deposit/tests/api/test_parser.py                 |  4 ----
 .../data/atom/entry-data-empty-body-no-namespace.xml |  2 ++
 .../tests/data/atom/entry-data-empty-body.xml        |  2 +-
 swh/deposit/tests/data/atom/entry-data3.xml          |  2 +-
 8 files changed, 35 insertions(+), 32 deletions(-)
 create mode 100644 swh/deposit/tests/data/atom/entry-data-empty-body-no-namespace.xml
Changes applied before test
commit b15033d6ce7c4b60cbf0c82c299a0ed322e7e2e7
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Mon Sep 28 16:18:37 2020 +0200

    tests: Ensure all empty body test cases are covered

commit f05480888a317c5cbc181ebdaa8d98e58914cc06
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Mon Sep 28 16:09:59 2020 +0200

    deposit.parsers: Process namespace when using xmltodict.parse
    
    This keeps the existing behavior though by mapping the expanded name to simpler
    namespaces.
    
    This also fixes some test data sample which were not properly defining their
    namespace.
    
    This will allow to deal more properly the empty body case which is incomplete
    right now.

See https://jenkins.softwareheritage.org/job/DDEP/job/tests-on-diff/158/ for more details.