Page MenuHomeSoftware Heritage

tests: remove all unused conftests
ClosedPublic

Authored by tenma on Jan 26 2021, 4:43 PM.

Diff Detail

Event Timeline

Build is green

Patch application report for D4949 (id=17630)

Rebasing onto 22eeb0956e...

Current branch diff-target is up to date.
Changes applied before test
commit 8099bb56028d9131d77a6400371e7adf79683377
Author: tenma <tenma+swh@mailbox.org>
Date:   Tue Jan 26 16:37:27 2021 +0100

    tests: remove all unused conftests

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

tenma requested review of this revision.Jan 26 2021, 4:46 PM
anlambert added a subscriber: anlambert.

Looks good but the commit message could be more detailed and rephrased to something like:

tests: Remove no longer used conftest files

All the fixtures declared in them are not used anymore in the tests of the listers
ported to the new swh.lister.pattern.Lister API.
This revision now requires changes to proceed.Jan 26 2021, 4:51 PM
This revision is now accepted and ready to land.Jan 26 2021, 5:01 PM

Build is green

Patch application report for D4949 (id=17633)

Rebasing onto 22eeb0956e...

Current branch diff-target is up to date.
Changes applied before test
commit b68f8f97f630c4ba97bba1fb1e2631387ec1f9ef
Author: tenma <tenma+swh@mailbox.org>
Date:   Tue Jan 26 16:37:27 2021 +0100

    tests: Remove no longer used conftest files
    
    All the fixtures declared in them are not used anymore in the
    tests of the listers ported to the new Lister API.

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

This revision is now accepted and ready to land.Jan 26 2021, 5:27 PM

Build is green

Patch application report for D4949 (id=17642)

Rebasing onto 6cd31769c1...

First, rewinding head to replay your work on top of it...
Fast-forwarded diff-target to base-revision-188-D4949.
Changes applied before test

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