diff --git a/conftest.py b/conftest.py new file mode 100644 --- /dev/null +++ b/conftest.py @@ -0,0 +1,15 @@ +# Copyright (C) 2020 The Software Heritage developers +# See the AUTHORS file at the top-level directory of this distribution +# License: GNU General Public License version 3, or any later version +# See top-level LICENSE file for more information + +import pytest + +pytest_plugins = ["swh.scheduler.pytest_plugin", "swh.storage.pytest_plugin"] + + +@pytest.fixture(scope="session") +def swh_scheduler_celery_includes(swh_scheduler_celery_includes): + return swh_scheduler_celery_includes + [ + "swh.loader.svn.tasks", + ] diff --git a/pytest.ini b/pytest.ini --- a/pytest.ini +++ b/pytest.ini @@ -1,4 +1,7 @@ [pytest] +# Drop this when these fixtures aren't imported automatically +addopts = -p no:pytest_swh_scheduler -p no:pytest_swh_storage + norecursedirs = docs markers = fs: execute tests that write to the filesystem diff --git a/swh/loader/svn/tests/conftest.py b/swh/loader/svn/tests/conftest.py --- a/swh/loader/svn/tests/conftest.py +++ b/swh/loader/svn/tests/conftest.py @@ -9,9 +9,6 @@ from typing import Any, Dict -from swh.scheduler.tests.conftest import swh_app # noqa -from swh.storage.tests.conftest import * # noqa - @pytest.fixture def swh_loader_config(swh_storage_backend_config) -> Dict[str, Any]: @@ -50,10 +47,3 @@ f.write(yaml.dump(swh_loader_config)) monkeypatch.setenv("SWH_CONFIG_FILENAME", conffile) return conffile - - -@pytest.fixture(scope="session") -def celery_includes(): - return [ - "swh.loader.svn.tasks", - ] diff --git a/swh/loader/svn/tests/test_task.py b/swh/loader/svn/tests/test_task.py --- a/swh/loader/svn/tests/test_task.py +++ b/swh/loader/svn/tests/test_task.py @@ -4,11 +4,13 @@ # See top-level LICENSE file for more information -def test_svn_loader(mocker, swh_app, celery_session_worker, swh_config): +def test_svn_loader( + mocker, swh_scheduler_celery_app, swh_scheduler_celery_worker, swh_config +): mock_loader = mocker.patch("swh.loader.svn.loader.SvnLoader.load") mock_loader.return_value = {"status": "eventful"} - res = swh_app.send_task( + res = swh_scheduler_celery_app.send_task( "swh.loader.svn.tasks.LoadSvnRepository", kwargs=dict(url="some-technical-url", origin_url="origin-url"), ) @@ -19,11 +21,13 @@ assert res.result == {"status": "eventful"} -def test_svn_loader_from_dump(mocker, swh_app, celery_session_worker, swh_config): +def test_svn_loader_from_dump( + mocker, swh_scheduler_celery_app, swh_scheduler_celery_worker, swh_config +): mock_loader = mocker.patch("swh.loader.svn.loader.SvnLoaderFromDumpArchive.load") mock_loader.return_value = {"status": "eventful"} - res = swh_app.send_task( + res = swh_scheduler_celery_app.send_task( "swh.loader.svn.tasks.MountAndLoadSvnRepository", kwargs=dict(url="some-url", archive_path="some-path"), ) @@ -35,12 +39,12 @@ def test_svn_loader_from_remote_dump( - mocker, swh_app, celery_session_worker, swh_config + mocker, swh_scheduler_celery_app, swh_scheduler_celery_worker, swh_config ): mock_loader = mocker.patch("swh.loader.svn.loader.SvnLoaderFromRemoteDump.load") mock_loader.return_value = {"status": "eventful"} - res = swh_app.send_task( + res = swh_scheduler_celery_app.send_task( "swh.loader.svn.tasks.DumpMountAndLoadSvnRepository", kwargs=dict(url="some-remote-dump-url", origin_url="origin-url"), ) diff --git a/tox.ini b/tox.ini --- a/tox.ini +++ b/tox.ini @@ -6,6 +6,7 @@ testing deps = pytest-cov + swh.scheduler[testing] >= 0.5.0 dev: pdbpp commands = pytest --cov={envsitepackagesdir}/swh/loader/svn \