Page MenuHomeSoftware Heritage

tests: Use dedicated storage and scheduler fixtures
ClosedPublic

Authored by ardumont on Jul 7 2020, 6:05 PM.

Diff Detail

Unit TestsFailed

TimeTest
1 msJenkins > .tox.py3.lib.python3.7.site-packages.swh.loader.package.archive.tests.test_archive::test_2_visits_with_new_artifact
file /var/lib/jenkins/workspace/DLDBASE/tests-on-diff/.tox/py3/lib/python3.7/site-packages/swh/loader/package/archive/tests/test_archive.py, line 230 def test_2_visits_with_new_artifact(swh_config, requests_mock_datadir): file /var/lib/jenkins/workspace/DLDBASE/tests-on-diff/conftest.py, line 38
1 msJenkins > .tox.py3.lib.python3.7.site-packages.swh.loader.package.archive.tests.test_archive::test_2_visits_without_change
file /var/lib/jenkins/workspace/DLDBASE/tests-on-diff/.tox/py3/lib/python3.7/site-packages/swh/loader/package/archive/tests/test_archive.py, line 202 def test_2_visits_without_change(swh_config, requests_mock_datadir): file /var/lib/jenkins/workspace/DLDBASE/tests-on-diff/conftest.py, line 38
1 msJenkins > .tox.py3.lib.python3.7.site-packages.swh.loader.package.archive.tests.test_archive::test_2_visits_without_change_not_gnu
file /var/lib/jenkins/workspace/DLDBASE/tests-on-diff/.tox/py3/lib/python3.7/site-packages/swh/loader/package/archive/tests/test_archive.py, line 306 def test_2_visits_without_change_not_gnu(swh_config, requests_mock_datadir): file /var/lib/jenkins/workspace/DLDBASE/tests-on-diff/conftest.py, line 38
2 msJenkins > .tox.py3.lib.python3.7.site-packages.swh.loader.package.archive.tests.test_archive::test_check_revision_metadata_structure
file /var/lib/jenkins/workspace/DLDBASE/tests-on-diff/.tox/py3/lib/python3.7/site-packages/swh/loader/package/archive/tests/test_archive.py, line 126 def test_check_revision_metadata_structure(swh_config, requests_mock_datadir): file /var/lib/jenkins/workspace/DLDBASE/tests-on-diff/conftest.py, line 38
1 msJenkins > .tox.py3.lib.python3.7.site-packages.swh.loader.package.archive.tests.test_archive::test_visit_with_release_artifact_no_prior_visit
file /var/lib/jenkins/workspace/DLDBASE/tests-on-diff/.tox/py3/lib/python3.7/site-packages/swh/loader/package/archive/tests/test_archive.py, line 158 def test_visit_with_release_artifact_no_prior_visit(swh_config, requests_mock_datadir): file /var/lib/jenkins/workspace/DLDBASE/tests-on-diff/conftest.py, line 38
View Full Test Results (56 Failed · 86 Passed)

Event Timeline

Build has FAILED

Patch application report for D3453 (id=12228)

Rebasing onto 3e48d63e88...

Current branch diff-target is up to date.
Changes applied before test
commit 225cd77d80e281fd9dbeef7c0976e99803a28f3d
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Tue Jul 7 18:03:11 2020 +0200

    tests: Use dedicated storage fixture
    
    Related to D3452
    Related to T2484

Link to build: https://jenkins.softwareheritage.org/job/DLDBASE/job/tests-on-diff/113/
See console output for more information: https://jenkins.softwareheritage.org/job/DLDBASE/job/tests-on-diff/113/console

vlorentz added inline comments.
conftest.py
12

you should also be able to remove this line, thanks to D3430

conftest.py
12

yeah right, i wanted to do that in a future commit

conftest.py
12

But yeah, i'll tag the storage v0.10.1 and add the necessary requirements on scheduler as well ;)

ardumont retitled this revision from tests: Use dedicated storage fixture to tests: Use dedicated storage and scheduler fixtures.Jul 8 2020, 2:34 PM
ardumont edited the test plan for this revision. (Show Details)

Build is green

Patch application report for D3453 (id=12251)

Rebasing onto 3e48d63e88...

Current branch diff-target is up to date.
Changes applied before test
commit 488b515a1c876a4b56fc7dadcc2ef4c8f3854c43
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Tue Jul 7 18:03:11 2020 +0200

    tests: Use dedicated storage and scheduler fixtures
    
    Related to D3452
    Related to D3430
    Related to T2484

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

conftest.py
12

swh_app is needed otherwise fail.

# noqa i will remove

conftest.py
12
  1. noqa i will remove

no, i can't.
Everything is fine in the end ;)

This revision is now accepted and ready to land.Jul 8 2020, 3:15 PM