Make azure-storage-blob optional for testing (#420)

* Make azure-storage-blob optional for testing

* Update 2.3.0 release title

* Note azure-storage-blob is optional for testing

* Skip covering the azure-storage-blob missing case

docs/release.rst

@@ -1,10 +1,22 @@
 Release notes
 =============
 
+.. _release_2.3.1:
+
+2.3.1
+-----
+
+Bug fixes
+~~~~~~~~~
+
+* Makes ``azure-storage-blob`` optional for testing.
+  By :user:`John Kirkham <jakirkham>`; :issue:`419`, :issue:`420`
+
+
 .. _release_2.3.0:
 
-2.3.0 (Work in Progress)
-------------------------
+2.3.0
+-----
 
 Enhancements
 ~~~~~~~~~~~~

zarr/tests/test_core.py

@@ -13,7 +13,11 @@
 import numpy as np
 from numpy.testing import assert_array_equal, assert_array_almost_equal
 import pytest
-from azure.storage.blob import BlockBlobService
+
+try:
+    import azure.storage.blob as asb
+except ImportError:  # pragma: no cover
+    asb = None
 
 
 from zarr.storage import (DirectoryStore, init_array, init_group, NestedDirectoryStore,
@@ -1409,11 +1413,13 @@ def test_nbytes_stored(self):
         assert expect_nbytes_stored == z.nbytes_stored
 
 
+@pytest.mark.skipif(asb is None,
+                    reason="azure-blob-storage could not be imported")
 class TestArrayWithABSStore(TestArray):
 
     @staticmethod
     def absstore():
-        blob_client = BlockBlobService(is_emulated=True)
+        blob_client = asb.BlockBlobService(is_emulated=True)
         blob_client.delete_container('test')
         blob_client.create_container('test')
         store = ABSStore(container='test', prefix='zarrtesting/', account_name='foo',

zarr/tests/test_hierarchy.py

@@ -13,7 +13,11 @@
 import numpy as np
 from numpy.testing import assert_array_equal
 import pytest
-from azure.storage.blob import BlockBlobService
+
+try:
+    import azure.storage.blob as asb
+except ImportError:  # pragma: no cover
+    asb = None
 
 
 from zarr.storage import (DictStore, DirectoryStore, ZipStore, init_group, init_array,
@@ -865,11 +869,13 @@ def create_store():
         return store, None
 
 
+@pytest.mark.skipif(asb is None,
+                    reason="azure-blob-storage could not be imported")
 class TestGroupWithABSStore(TestGroup):
 
     @staticmethod
     def create_store():
-        blob_client = BlockBlobService(is_emulated=True)
+        blob_client = asb.BlockBlobService(is_emulated=True)
         blob_client.delete_container('test')
         blob_client.create_container('test')
         store = ABSStore(container='test', prefix='zarrtesting/', account_name='foo',

zarr/tests/test_storage.py

@@ -15,7 +15,11 @@
 import numpy as np
 from numpy.testing import assert_array_equal, assert_array_almost_equal
 import pytest
-from azure.storage.blob import BlockBlobService
+
+try:
+    import azure.storage.blob as asb
+except ImportError:  # pragma: no cover
+    asb = None
 
 
 from zarr.storage import (init_array, array_meta_key, attrs_key, DictStore,
@@ -1514,10 +1518,12 @@ def test_format_compatibility():
                 assert compressor.get_config() == z.compressor.get_config()
 
 
+@pytest.mark.skipif(asb is None,
+                    reason="azure-blob-storage could not be imported")
 class TestABSStore(StoreTests, unittest.TestCase):
 
     def create_store(self):
-        blob_client = BlockBlobService(is_emulated=True)
+        blob_client = asb.BlockBlobService(is_emulated=True)
         blob_client.delete_container('test')
         blob_client.create_container('test')
         store = ABSStore(container='test', prefix='zarrtesting/', account_name='foo',