Skip to content

Commit

Permalink
only has-nczar is needed, users can figure the platform and version
Browse files Browse the repository at this point in the history
  • Loading branch information
ocefpaf committed Aug 14, 2024
1 parent 32c0676 commit f6533c5
Showing 1 changed file with 1 addition and 9 deletions.
10 changes: 1 addition & 9 deletions compliance_checker/tests/test_cli.py
Original file line number Diff line number Diff line change
Expand Up @@ -237,17 +237,9 @@ def _check_libnetcdf_version():
)

# 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 Down

0 comments on commit f6533c5

Please sign in to comment.