Skip to content
Snippets Groups Projects

Expose deposit configuration only within the deposit loader tests

Test Plan

tox


Migrated from D4111 (view on Phabricator)

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Build is green

    Patch application report for D4111 (id=14484)

    Could not rebase; Attempt merge onto 63e9d3ca...

    Updating 63e9d3c..d5f9a45
    Fast-forward
     swh/loader/package/deposit/loader.py               |  60 ++++++---
     swh/loader/package/deposit/tests/conftest.py       |  23 ++++
     .../hello_2.10.json                                |  48 ++++---
     .../hello_2.11.json                                |  52 ++++----
     swh/loader/package/deposit/tests/test_deposit.py   | 145 +++++++++++++--------
     swh/loader/pytest_plugin.py                        |   4 -
     6 files changed, 201 insertions(+), 131 deletions(-)
     create mode 100644 swh/loader/package/deposit/tests/conftest.py
    Changes applied before test
    commit d5f9a4574d4c6947868e0ced2ff4cbb72a1baf2e
    Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
    Date:   Thu Oct 1 13:51:42 2020 +0200
    
        Expose deposit configuration only within the deposit tests
    
    commit 777ea4468ad3c4e8981151b29f8b70ff7124c4ef
    Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
    Date:   Wed Sep 30 17:54:49 2020 +0200
    
        deposit: Adapt loader to send extrinsic raw metadata to the metadata storage
        
        This keeps the metadata written in the revision in the same format as before
        though (json dict).
        
        Related to swh/devel/swh-deposit#2649

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

  • Merge request was accepted

  • Antoine Lambert approved this merge request

    approved this merge request

  • Merge request was merged

Please register or sign in to reply
Loading