Page MenuHomeSoftware Heritage

test_retry: Drop validate proxy when we can
ClosedPublic

Authored by ardumont on Jul 15 2020, 7:08 PM.

Details

Summary

When we use the sample_data_model (almost all object types except the metadata
ones), we can use a storage with no validate proxy.

Depends on D3510

Related to T2494

Test Plan

tox

Diff Detail

Repository
rDSTO Storage manager
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 13641
Build 20894: Phabricator diff pipeline on jenkinsJenkins console · Jenkins
Build 20893: arc lint + arc unit

Event Timeline

Build is green

Patch application report for D3511 (id=12420)

Could not rebase; Attempt merge onto f5811da184...

Updating f5811da1..11d3e162
Fast-forward
 swh/storage/pytest_plugin.py            |  43 +++-
 swh/storage/tests/conftest.py           |  23 --
 swh/storage/tests/test_pytest_plugin.py |  60 ++++++
 swh/storage/tests/test_retry.py         | 368 +++++++++++++++-----------------
 4 files changed, 275 insertions(+), 219 deletions(-)
 create mode 100644 swh/storage/tests/test_pytest_plugin.py
Changes applied before test
commit 11d3e162318873a76f9956c5795f1eb0e03ccd7c
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Wed Jul 15 19:07:32 2020 +0200

    test_retry: Drop validate proxy when we can
    
    When we use the sample_data_model (almost all object types except the metadata
    ones), we can use a storage with no validate proxy.
    
    Depends on D3510

commit 2e1ca5a2132545ab793f0a2e3cc74afd0afb7223
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Wed Jul 15 17:40:04 2020 +0200

    test_retry: Use sample_data_model fixture to manipulate model objects

commit 60a2dae6b461245cd6532e234cf651b247264b3c
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Wed Jul 15 17:03:33 2020 +0200

    pytest-plugin: Expose a sample_data_model fixture
    
    This is almost the same fixture as sample_data except:
    - it's BaseModel object instance within
    - not complete as we cannot convert yet the metadata objects (there is a diff
      pending which will allow it but right now we cannot).
    
    The next commits will use this fixture to allow the switch from dict to model
    objects.

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

This revision is now accepted and ready to land.Jul 16 2020, 10:57 AM

Build is green

Patch application report for D3511 (id=12445)

Rebasing onto 14b1648889...

Current branch diff-target is up to date.
Changes applied before test
commit df45641008cd4a74db1d40f307a70013bed6f7b8
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Wed Jul 15 19:07:32 2020 +0200

    test_retry: Drop validate proxy when we can
    
    When we use the sample_data_model (almost all object types except the metadata
    ones), we can use a storage with no validate proxy.
    
    Depends on D3510

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