Skip to content

Commit

Permalink
only has-nczar is needed
Browse files Browse the repository at this point in the history
  • Loading branch information
ocefpaf committed Aug 14, 2024
1 parent 32c0676 commit 42db7a3
Showing 1 changed file with 7 additions and 11 deletions.
18 changes: 7 additions & 11 deletions compliance_checker/tests/test_cli.py
Original file line number Diff line number Diff line change
Expand Up @@ -236,18 +236,9 @@ def _check_libnetcdf_version():
< 8.0
)

# TODO uncomment the third parameter once S3 support is working
@pytest.mark.skipif(
_check_libnetcdf_version(),
reason="NCZarr support was not available until netCDF version 4.8.0. Please upgrade to the latest libnetcdf version to test this functionality",
)
@pytest.mark.skipif(
subprocess.check_output(ncconfig + ["--has-nczarr"]) != b"yes\n",
reason="NCZarr is not officially supported for your OS as of when this API was written",
)
@pytest.mark.skipif(
subprocess.check_output(ncconfig + ["--has-nczarr"]) != b"yes\n",
reason="NCZarr support was not built with this netCDF version",
reason="NCZarr is not available.",
)
@pytest.mark.parametrize(
"zarr_url",
Expand All @@ -256,7 +247,12 @@ def _check_libnetcdf_version():
str(datadir / "zip.zarr"),
# "s3://hrrrzarr/sfc/20210408/20210408_10z_anl.zarr#mode=nczarr,s3"
],
ids=["local_file", "zip_file"], # ,'s3_url'
ids=[
"local_file",
"zip_file",
# TODO uncomment once S3 support is working.
# "s3_url",
],
)
def test_nczarr_pass_through(self, zarr_url):
"""
Expand Down

0 comments on commit 42db7a3

Please sign in to comment.