Changeset View
Changeset View
Standalone View
Standalone View
conftest.py
# Copyright (C) 2019-2020 The Software Heritage developers | # Copyright (C) 2019-2020 The Software Heritage developers | ||||
# See the AUTHORS file at the top-level directory of this distribution | # See the AUTHORS file at the top-level directory of this distribution | ||||
# License: GNU General Public License version 3, or any later version | # License: GNU General Public License version 3, or any later version | ||||
# See top-level LICENSE file for more information | # See top-level LICENSE file for more information | ||||
import os | import os | ||||
import pytest | import pytest | ||||
import yaml | import yaml | ||||
from typing import Any, Dict | from typing import Any, Dict | ||||
from swh.storage.tests.conftest import * # noqa | from swh.scheduler.tests.conftest import swh_app # noqa | ||||
vlorentz: you should also be able to remove this line, thanks to D3430 | |||||
Done Inline Actionsyeah right, i wanted to do that in a future commit ardumont: yeah right, i wanted to do that in a future commit | |||||
Done Inline ActionsBut yeah, i'll tag the storage v0.10.1 and add the necessary requirements on scheduler as well ;) ardumont: But yeah, i'll tag the storage v0.10.1 and add the necessary requirements on scheduler as well… | |||||
Done Inline Actionsswh_app is needed otherwise fail. # noqa i will remove ardumont: swh_app is needed otherwise fail.
`# noqa` i will remove | |||||
Done Inline Actions
no, i can't. ardumont: > # noqa i will remove
no, i can't.
Everything is fine in the end ;) | |||||
from swh.scheduler.tests.conftest import * # noqa | |||||
@pytest.fixture | @pytest.fixture | ||||
def swh_loader_config(swh_storage_postgresql) -> Dict[str, Any]: | def swh_loader_config(swh_storage_postgresql) -> Dict[str, Any]: | ||||
return { | return { | ||||
"storage": { | "storage": { | ||||
"cls": "pipeline", | "cls": "pipeline", | ||||
"steps": [ | "steps": [ | ||||
▲ Show 20 Lines • Show All 47 Lines • Show Last 20 Lines |
you should also be able to remove this line, thanks to D3430