From 3cc715ce43acc5eda6b0115a74d7a4e2c917774a Mon Sep 17 00:00:00 2001 From: Paul Price Date: Sat, 3 Dec 2022 09:51:20 -0500 Subject: [PATCH] disable tests Tests (including flake8) are importing parsl, which isn't in base LSST env, so temporarily disabling them. --- doc/changes/DM-37215.misc.rst | 1 + pyproject.toml | 6 +++--- tests/test_config.py | 7 +++++-- tests/test_import.py | 4 ++++ 4 files changed, 13 insertions(+), 5 deletions(-) create mode 100644 doc/changes/DM-37215.misc.rst diff --git a/doc/changes/DM-37215.misc.rst b/doc/changes/DM-37215.misc.rst new file mode 100644 index 0000000..8002814 --- /dev/null +++ b/doc/changes/DM-37215.misc.rst @@ -0,0 +1 @@ +Disable tests until parsl can be installed in the LSST base env. diff --git a/pyproject.toml b/pyproject.toml index b273261..9e43232 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -99,9 +99,9 @@ line_length = 110 [tool.lsst_versions] write_to = "python/lsst/ctrl/bps/parsl/version.py" -[tool.pytest.ini_options] -addopts = "--flake8" -flake8-ignore = ["W503", "E203", "N802", "N803", "N806", "N812", "N815", "N816"] +#[tool.pytest.ini_options] +#addopts = "--flake8" +#flake8-ignore = ["W503", "E203", "N802", "N803", "N806", "N812", "N815", "N816"] [tool.pydocstyle] convention = "numpy" diff --git a/tests/test_config.py b/tests/test_config.py index e77d20e..76507fc 100644 --- a/tests/test_config.py +++ b/tests/test_config.py @@ -1,12 +1,15 @@ -from lsst.ctrl.bps import BpsConfig -from lsst.ctrl.bps.parsl.configuration import get_bps_config_value +import pytest from lsst.daf.butler import Config +@pytest.mark.skip(reason="parsl not in LSST environment") def test_config(): """Super-basic test of configuration reading This is intended as a test of testing more than anything else. """ + from lsst.ctrl.bps import BpsConfig + from lsst.ctrl.bps.parsl.configuration import get_bps_config_value + config = BpsConfig(Config.fromString("foo: bar")) # BpsConfig doesn't work directly with fromString assert get_bps_config_value(config, "foo", str) == "bar" diff --git a/tests/test_import.py b/tests/test_import.py index 6859807..8360b5b 100644 --- a/tests/test_import.py +++ b/tests/test_import.py @@ -1,2 +1,6 @@ +import pytest + + +@pytest.mark.skip(reason="parsl not in LSST environment") def test_import(): import lsst.ctrl.bps.parsl # noqa