Skip to content

Commit

Permalink
could it be the extra /?
Browse files Browse the repository at this point in the history
  • Loading branch information
ocefpaf committed Aug 15, 2024
1 parent 1d28299 commit 7d43120
Showing 1 changed file with 6 additions and 5 deletions.
11 changes: 6 additions & 5 deletions compliance_checker/tests/test_cli.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,9 @@

from .conftest import datadir, static_files

if platform.system() == "Windows":
on_windows = platform.system() == "Windows"

if on_windows:
ncconfig = ["sh", f"{os.environ['CONDA_PREFIX']}\\Library\\bin\\nc-config"]
else:
ncconfig = ["nc-config"]
Expand Down Expand Up @@ -236,10 +238,6 @@ def _check_libnetcdf_version():
< 8.0
)

@pytest.mark.skipif(
platform.system() == "Windows",
reason="NCZarr triggers a segfault on Windows.",
)
@pytest.mark.skipif(
subprocess.check_output(ncconfig + ["--has-nczarr"]) != b"yes\n",
reason="NCZarr is not available.",
Expand Down Expand Up @@ -267,6 +265,9 @@ def test_nczarr_pass_through(self, zarr_url):
"check_filename", # .zarr cannot pass a test that requires it to be named .nc
"check_coordinate_variables_strict_monotonicity", # FIXME: I believe there is a real problem with the original test data!
]
if on_windows:
zarr_url = zarr_url.replace("///", "//")

return_value, errors = ComplianceChecker.run_checker(
skip_checks=skip_checks,
ds_loc=zarr_url,
Expand Down

0 comments on commit 7d43120

Please sign in to comment.