pytest_plugin: Avoid fixture client to declare optional dependency
ClosedPublic
Actions

Authored by ardumont on Jul 16 2020, 9:36 AM.

Details

Reviewers

anlambert
douardda

Group Reviewers

Reviewers

Maniphest Tasks

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

Commits

rDSTO8bc79443eda7: pytest_plugin: Avoid fixture client to declare optional dependency

Summary

Before this commit, this would make swh_storage_backend_config fixture clients
require to declare an optional dependency on swh.journal. Otherwise, it would not
build [1].

This commit fixes it by dropping this configuration in the main pytest plugin.
It keeps the storage tests testing with that journal_writer collaborator though (as initially)
by declaring an override which still provides it.

This fixes the debian build [1]

[1] https://jenkins.softwareheritage.org/view/Debian%20packages/job/debian/job/packages/job/DLDBASE/job/gbp-buildpackage/152/console

This replaces D3514 by fixing the problem at the source. D3514 took the path to
change the configuration within the clients. This is not sustainable as this
would require other fixture clients to do the same.

Test Plan

tox

Diff Detail

Repository

rDSTO Storage manager

Branch

master

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 13646
Build 20904: Phabricator diff pipeline on jenkins	Jenkins console · Jenkins
Build 20903: arc lint + arc unit

Event Timeline

ardumont created this revision.Jul 16 2020, 9:36 AM

Herald added a reviewer: Reviewers. · View Herald TranscriptJul 16 2020, 9:36 AM

ardumont edited the summary of this revision. (Show Details)Jul 16 2020, 9:36 AM

ardumont mentioned this in D3514: tests: Override storage config fixture to not specify journal writer.

ardumont mentioned this in D3501: Use bytes as snapshot id and get rid of dict manipulation in check_snapshot().Jul 16 2020, 9:39 AM

ardumont edited the summary of this revision. (Show Details)

Build is green

Patch application report for D3515 (id=12425)

Rebasing onto f5811da184...

Current branch diff-target is up to date.

Changes applied before test

commit 8bc79443eda76975df65fa302184ce1a1fc62b11
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Thu Jul 16 09:33:00 2020 +0200

    pytest_plugin: Avoid fixture client to declare optional dependency
    
    Prior to this commit, this would make swh_storage_backend_config fixture clients
    need to declare an optional dependency on swh.journal. Otherwise, it would not
    work [1].
    
    This commit fixes it by dropping this configuration in the main pytest plugin.
    It keeps the storage tests testing with that journal_writer collaborator though
    by declaring an override which still provides it.
    
    This fixes the debian build [1]
    
    [1] https://jenkins.softwareheritage.org/view/Debian%20packages/job/debian/job/packages/job/DLDBASE/job/gbp-buildpackage/152/console

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

Harbormaster completed remote builds in B13646: Diff 12425.Jul 16 2020, 9:41 AM

ardumont added a task: T2494: tests: Use data model objects within tests (drop dicts).Jul 16 2020, 10:30 AM

ardumont added a task: T2484: Move sharable fixtures out of conftest into a dedicated pytest plugin.