Paths

Table of Contentst

Differential D3452

Move sharable fixtures out of conftest into a dedicated pytest plugin
ClosedPublic
Actions

Authored by ardumont on Jul 7 2020, 5:55 PM.

Details

Reviewers

anlambert

Group Reviewers

Reviewers

Maniphest Tasks

T2484: Move sharable fixtures out of conftest into a dedicated pytest plugin

Commits

rDSTOc21d0e38204a: Move sharable fixtures out of conftest into a dedicated pytest plugin

Summary

This will allow loaders to reuse those dedicated fixtures within their code
base without having to import the swh.storage.tests.conftest module.

(Note: I find the diff displayed a tad weirdly as i only move code from conftest
to a new pytest_plugin..., did not remove anything...)

Related to T2484

Test Plan

tox

And ensure client wise (locally), it's indeed reusable as in [1]:

adapting locally swh.loader.core requirement-test.txt to swh.storage[testing]
dropping spurrious swh.storage.tests.conftests.* import
tox amended with storage .tox/bin/pip install -e ../swh-storage
all tests are green

[1] See D3453

Diff Detail

Repository

rDSTO Storage manager

Branch

extract-pytest-fixture

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 13459
Build 20587: Phabricator diff pipeline on jenkins	Jenkins console · Jenkins
Build 20586: arc lint + arc unit

Event Timeline

ardumont created this revision.Jul 7 2020, 5:55 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptJul 7 2020, 5:55 PM

ardumont edited the test plan for this revision. (Show Details)Jul 7 2020, 5:59 PM

Build is green

Patch application report for D3452 (id=12227)

Could not rebase; Attempt merge onto 5ab70237d6...

Updating 5ab70237..e61997e8
Fast-forward
 setup.py                      |   8 +-
 swh/storage/pytest_plugin.py  | 182 ++++++++++++++++++++++++++++++++++++++++++
 swh/storage/tests/conftest.py | 169 ---------------------------------------
 3 files changed, 187 insertions(+), 172 deletions(-)
 create mode 100644 swh/storage/pytest_plugin.py

Changes applied before test

commit e61997e8b0d3405c6d62402247c00be218d32c06
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Tue Jul 7 11:09:25 2020 +0200

    Move sharable fixtures out of conftest into a dedicated pytest plugin
    
    This will allow loaders to reuse those dedicated fixtures within their code
    base without having to import the swh.storage.tests.conftest module.
    
    Related to T2484

commit e45ca76db409f917c42152861bbf56142d563d62
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Tue Jul 7 12:07:18 2020 +0200

    Migrate from vcversioner to setuptools-scm
    
    Related to T2105

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

Harbormaster completed remote builds in B13447: Diff 12227.Jul 7 2020, 6:03 PM

ardumont mentioned this in D3453: tests: Use dedicated storage and scheduler fixtures.Jul 7 2020, 6:05 PM

ardumont edited the test plan for this revision. (Show Details)Jul 7 2020, 6:08 PM

ardumont edited the test plan for this revision. (Show Details)Jul 8 2020, 10:14 AM

ardumont edited the summary of this revision. (Show Details)Jul 8 2020, 10:50 AM

ardumont edited the summary of this revision. (Show Details)Jul 8 2020, 11:01 AM

Looks good. Phabricator diffs are indeed surprising and misleading here, feels like a bug to me.

This revision is now accepted and ready to land.Jul 8 2020, 11:25 AM

Expose sample_data within pytest_plugin (as there is a need for it in the next loader-core tests refactoring [1])

Add retro-compatibility symbols in conftest to avoid breaking all existing modules depending on it (at least loaders, i'm gonna change this soon but still ;)

[1] T2483

Build is green

Patch application report for D3452 (id=12239)

Rebasing onto e45ca76db4...

Current branch diff-target is up to date.

Changes applied before test

commit c21d0e38204aaca3999c00f2346b0d5c69a2ddff
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Tue Jul 7 11:09:25 2020 +0200

    Move sharable fixtures out of conftest into a dedicated pytest plugin
    
    This will allow loaders to reuse those dedicated fixtures within their code
    base without having to import the swh.storage.tests.conftest module.
    
    Related to T2484

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

Harbormaster completed remote builds in B13459: Diff 12239.Jul 8 2020, 11:57 AM

Closed by commit rDSTOc21d0e38204a: Move sharable fixtures out of conftest into a dedicated pytest plugin (authored by ardumont). · Explain WhyJul 8 2020, 12:35 PM

This revision was automatically updated to reflect the committed changes.

ardumont added a commit: rDSTOc21d0e38204a: Move sharable fixtures out of conftest into a dedicated pytest plugin.

ardumont mentioned this in rDLDBASE488b515a1c87: tests: Use dedicated storage and scheduler fixtures.Jul 8 2020, 3:19 PM