specify reason for skipping

bioimage-io · Oct 16, 2023 · 3dc6c4e · 3dc6c4e
1 parent e9f312e
commit 3dc6c4e
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 9 deletions.
diff --git a/tests/conftest.py b/tests/conftest.py
@@ -2,8 +2,6 @@
 
 import pytest
 
-SKIP_ZENODO = True  # skip tests relying on zenodo
-
 
 @pytest.fixture
 def unet2d_nuclei_broad_base_path():

diff --git a/tests/test_cli.py b/tests/test_cli.py
@@ -13,6 +13,7 @@
 from bioimageio.spec.shared import yaml
 
 SKIP_ZENODO = True
+SKIP_ZENODO_REASON = "zenodo api changes"
 
 
 def run_subprocess(commands: Sequence[str], **kwargs) -> subprocess.CompletedProcess:
@@ -49,7 +50,7 @@ def test_cli_validate_model_url_wo_cache():
     assert ret.returncode == 0
 
 
-@pytest.mark.skipif(SKIP_ZENODO)
+@pytest.mark.skipif(SKIP_ZENODO, reason=SKIP_ZENODO_REASON)
 def test_cli_validate_model_doi():
     ret = run_subprocess(["bioimageio", "validate", "10.5281/zenodo.5744489"])
     assert ret.returncode == 0

diff --git a/tests/test_commands.py b/tests/test_commands.py
@@ -12,6 +12,7 @@
 from bioimageio.spec.shared import yaml
 
 SKIP_ZENODO = True
+SKIP_ZENODO_REASON = "zenodo api changes"
 
 
 def test_validate_dataset(dataset_rdf):
@@ -45,15 +46,15 @@ def test_validate_model_as_url():
     )["error"]
 
 
-@pytest.mark.skipif(SKIP_ZENODO)
+@pytest.mark.skipif(SKIP_ZENODO, reason=SKIP_ZENODO_REASON)
 def test_validate_model_as_zenodo_sandbox_doi():
     from bioimageio.spec.commands import validate
 
     doi = "10.5281/zenodo.5744489"
     assert not validate(doi, update_format=False, update_format_inner=False)["error"]
 
 
-@pytest.mark.skipif(SKIP_ZENODO)
+@pytest.mark.skipif(SKIP_ZENODO, reason=SKIP_ZENODO_REASON)
 def test_validate_model_as_zenodo_doi():
     from bioimageio.spec.commands import validate
 
@@ -72,7 +73,7 @@ def test_validate_model_as_bioimageio_full_version_id_partner():
     assert summary["status"] == "passed", summary["error"]
 
 
-@pytest.mark.skipif(SKIP_ZENODO)
+@pytest.mark.skipif(SKIP_ZENODO, reason=SKIP_ZENODO_REASON)
 def test_validate_model_as_bioimageio_full_version_id_zenodo():
     from bioimageio.spec.commands import validate
 
@@ -89,7 +90,7 @@ def test_validate_model_as_bioimageio_resource_id_partner():
     assert summary["status"] == "passed", summary["error"]
 
 
-@pytest.mark.skipif(SKIP_ZENODO)
+@pytest.mark.skipif(SKIP_ZENODO, reason=SKIP_ZENODO_REASON)
 def test_validate_model_as_bioimageio_resource_id_zenodo():
     from bioimageio.spec.commands import validate
 

diff --git a/tests/test_schema_model.py b/tests/test_schema_model.py
@@ -7,6 +7,7 @@
 from bioimageio.spec.shared import yaml
 
 SKIP_ZENODO = True
+SKIP_ZENODO_REASON = "zenodo api changes"
 
 
 def test_model_rdf_is_valid_general_rdf(unet2d_nuclei_broad_latest):
@@ -218,7 +219,7 @@ def test_output_ref_shape_too_small(model_dict):
     }
 
 
-@pytest.mark.skipif(SKIP_ZENODO)
+@pytest.mark.skipif(SKIP_ZENODO, reason=SKIP_ZENODO_REASON)
 def test_model_has_parent_with_uri(model_dict):
     from bioimageio.spec.model.schema import Model
 
@@ -228,7 +229,7 @@ def test_model_has_parent_with_uri(model_dict):
     assert isinstance(valid_data, raw_nodes_m04.Model)
 
 
-@pytest.mark.skipif(SKIP_ZENODO)
+@pytest.mark.skipif(SKIP_ZENODO, reason=SKIP_ZENODO_REASON)
 def test_model_has_parent_with_id(model_dict):
     from bioimageio.spec.model.schema import Model