Skip to content

Commit 24db6d6

Browse files
authored
test: skip native DSS integration on macOS (#252)
## Summary - skip the real native DSS round-trip integration test on macOS - document that the bundled `hecdss` native library is not available in Mach-O format ## Why The cross-platform matrix and DSS integration tests were merged independently. Their combination revealed that `hecdss` installs a Linux `libhecdss.so` on macOS, which fails to load because it is not a Mach-O library. Unit and mocked DSS tests remain enabled on macOS; only the native file round trip is skipped. ## Validation - `poetry run pytest -q` (271 passed on Windows, including all three native DSS round trips) - targeted pre-commit checks passed - `poetry run sphinx-build -W -b html docs .tmp_docs_html` - `git diff --check`
1 parent 93f459f commit 24db6d6

2 files changed

Lines changed: 9 additions & 0 deletions

File tree

docs/cli/dss.rst

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,9 @@ HEC-DSS support is optional:
1010
1111
pip install "cwms-cli[dss]"
1212
13+
The native ``hecdss`` library is not currently available in macOS Mach-O
14+
format. DSS file operations are therefore not supported on macOS.
15+
1316
.. _dss-commands:
1417

1518
Commands

tests/dss/test_dss_integration.py

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,15 @@
1+
import sys
2+
13
import pytest
24

35
from cwmscli.dss.naming import ExportRule, default_pathname
46
from cwmscli.dss.transfer import DssSink, DssSource, transform_export
57

68

9+
@pytest.mark.skipif(
10+
sys.platform == "darwin",
11+
reason="hecdss does not currently provide a macOS native library",
12+
)
713
@pytest.mark.integration
814
@pytest.mark.parametrize(
915
("tsid", "times", "dss_time_zone"),

0 commit comments

Comments
 (0)