Skip to content

Commit

Permalink
disable tests
Browse files Browse the repository at this point in the history
Tests (including flake8) are importing parsl, which isn't in
base LSST env, so temporarily disabling them.
  • Loading branch information
PaulPrice committed Dec 3, 2022
1 parent c596822 commit 3cc715c
Show file tree
Hide file tree
Showing 4 changed files with 13 additions and 5 deletions.
1 change: 1 addition & 0 deletions doc/changes/DM-37215.misc.rst
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Disable tests until parsl can be installed in the LSST base env.
6 changes: 3 additions & 3 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -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"
Expand Down
7 changes: 5 additions & 2 deletions tests/test_config.py
Original file line number Diff line number Diff line change
@@ -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"
4 changes: 4 additions & 0 deletions tests/test_import.py
Original file line number Diff line number Diff line change
@@ -1,2 +1,6 @@
import pytest


@pytest.mark.skip(reason="parsl not in LSST environment")
def test_import():
import lsst.ctrl.bps.parsl # noqa

0 comments on commit 3cc715c

Please sign in to comment.