Merge pull request #667 from nkaretnikov/skip-conda-docker-test

Disable code that depends on conda-docker

conda-store-server/conda_store_server/action/generate_conda_docker.py

@@ -1,5 +1,4 @@
 import pathlib
-import sys
 
 from conda_store_server import action
 
@@ -13,10 +12,10 @@ def action_generate_conda_docker(
     output_image_name: str,
     output_image_tag: str,
 ):
-    if sys.platform != "linux":
-        raise RuntimeError(
-            "Generating Docker images is currently only supported on Linux"
-        )
+    raise RuntimeError(
+        "Generating Docker images is currently not supported, see "
+        "https://github.com/conda-incubator/conda-store/issues/666"
+    )
 
     # Import is inside the function because conda_docker is only available on
     # Linux

conda-store-server/conda_store_server/build.py

@@ -313,6 +313,14 @@ def build_conda_pack(db: Session, conda_store, build: orm.Build):
 
 
 def build_conda_docker(db: Session, conda_store, build: orm.Build):
+    import warnings
+
+    warnings.warn(
+        "Generating Docker images is currently not supported, see "
+        "https://github.com/conda-incubator/conda-store/issues/666"
+    )
+    return
+
     conda_prefix = build.build_path(conda_store)
     settings = conda_store.get_settings(
         db=db,

conda-store-server/tests/test_actions.py

@@ -128,7 +128,9 @@ def test_generate_conda_pack(tmp_path, conda_prefix):
     assert output_filename.exists()
 
 
-@pytest.mark.skipif(sys.platform != "linux", reason="conda-docker only works on linux")
+@pytest.mark.xfail(reason=(
+    "Generating Docker images is currently not supported, see "
+    "https://github.com/conda-incubator/conda-store/issues/666"))
 def test_generate_conda_docker(conda_store, conda_prefix):
     action.action_generate_conda_docker(
         conda_prefix=conda_prefix,