Merge pull request #260 from dandi/fixes

Fix failing tests

CHANGELOG.md

@@ -15,6 +15,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 - Moved code for navigating Dandi Archive into new `dandiarchive` submodule
 - YAML output now sorts keys
 - `dandiset.yaml` is no longer uploaded to the archive
+- Restrict h5py dependency to pre-v3.0
 
 ## [0.6.4] - 2020-09-04
 

dandi/tests/test_download.py

@@ -72,7 +72,10 @@ def sleep(duration):
 @pytest.mark.parametrize(
     "url",
     [  # Should go through API
-        "https://dandiarchive.org/dandiset/000027/0.200721.2222",
+        pytest.param(
+            "https://dandiarchive.org/dandiset/000027/0.200721.2222",
+            marks=pytest.mark.xfail(reason="publish.dandiarchive.org is gone"),
+        ),
         # Drafts do not go through API ATM, but that should not be visible to user
         "https://dandiarchive.org/dandiset/000027/draft",
     ],
@@ -107,7 +110,10 @@ def test_download_000027(url, tmpdir):
 @pytest.mark.parametrize(
     "url",
     [  # Should go through API
-        "https://dandiarchive.org/dandiset/000027/0.200721.2222",
+        pytest.param(
+            "https://dandiarchive.org/dandiset/000027/0.200721.2222",
+            marks=pytest.mark.xfail(reason="publish.dandiarchive.org is gone"),
+        ),
         # Drafts do not go through API ATM, but that should not be visible to user
         "https://dandiarchive.org/dandiset/000027/draft",
     ],
@@ -123,7 +129,10 @@ def test_download_000027_metadata_only(url, tmpdir):
 @pytest.mark.parametrize(
     "url",
     [  # Should go through API
-        "https://dandiarchive.org/dandiset/000027/0.200721.2222",
+        pytest.param(
+            "https://dandiarchive.org/dandiset/000027/0.200721.2222",
+            marks=pytest.mark.xfail(reason="publish.dandiarchive.org is gone"),
+        ),
         # Drafts do not go through API ATM, but that should not be visible to user
         "https://dandiarchive.org/dandiset/000027/draft",
     ],
@@ -148,7 +157,16 @@ def raise_assertion_error(*args, **kwargs):
 
 
 @pytest.mark.parametrize("resizer", [lambda sz: 0, lambda sz: sz // 2, lambda sz: sz])
-@pytest.mark.parametrize("version", ["0.200721.2222", "draft"])
+@pytest.mark.parametrize(
+    "version",
+    [
+        pytest.param(
+            "0.200721.2222",
+            marks=pytest.mark.xfail(reason="publish.dandiarchive.org is gone"),
+        ),
+        "draft",
+    ],
+)
 def test_download_000027_resume(tmp_path, resizer, version):
     from ..support.digests import Digester
 

setup.cfg

@@ -41,6 +41,7 @@ install_requires =
     email-validator
     etelemetry >= 0.2.0
     fasteners
+    h5py ~= 2.9
     joblib
     pydantic
     pyout != 0.6.0