remove uncovered lines

zarr-developers · Mar 10, 2021 · 3e82a79 · 3e82a79
1 parent 25412f8
commit 3e82a79
Showing 1 changed file with 45 additions and 49 deletions.
diff --git a/zarr/tests/test_storage.py b/zarr/tests/test_storage.py
@@ -352,55 +352,51 @@ def test_hierarchy(self):
 
         # test rename (optional)
         if store.is_erasable():
-            try:
-                store.rename("c/e", "c/e2")
-                assert "c/d" in store
-                assert "c/e" not in store
-                assert "c/e/f" not in store
-                assert "c/e/g" not in store
-                assert "c/e2" not in store
-                assert "c/e2/f" in store
-                assert "c/e2/g" in store
-                store.rename("c/e2", "c/e")
-                assert "c/d" in store
-                assert "c/e2" not in store
-                assert "c/e2/f" not in store
-                assert "c/e2/g" not in store
-                assert "c/e" not in store
-                assert "c/e/f" in store
-                assert "c/e/g" in store
-                store.rename("c", "c1/c2/c3")
-                assert "a" in store
-                assert "c" not in store
-                assert "c/d" not in store
-                assert "c/e" not in store
-                assert "c/e/f" not in store
-                assert "c/e/g" not in store
-                assert "c1" not in store
-                assert "c1/c2" not in store
-                assert "c1/c2/c3" not in store
-                assert "c1/c2/c3/d" in store
-                assert "c1/c2/c3/e" not in store
-                assert "c1/c2/c3/e/f" in store
-                assert "c1/c2/c3/e/g" in store
-                store.rename("c1/c2/c3", "c")
-                assert "c" not in store
-                assert "c/d" in store
-                assert "c/e" not in store
-                assert "c/e/f" in store
-                assert "c/e/g" in store
-                assert "c1" not in store
-                assert "c1/c2" not in store
-                assert "c1/c2/c3" not in store
-                assert "c1/c2/c3/d" not in store
-                assert "c1/c2/c3/e" not in store
-                assert "c1/c2/c3/e/f" not in store
-                assert "c1/c2/c3/e/g" not in store
-            except NotImplementedError:
-                pass
-
-        # test rmdir (optional)
-        if store.is_erasable():
+            store.rename("c/e", "c/e2")
+            assert "c/d" in store
+            assert "c/e" not in store
+            assert "c/e/f" not in store
+            assert "c/e/g" not in store
+            assert "c/e2" not in store
+            assert "c/e2/f" in store
+            assert "c/e2/g" in store
+            store.rename("c/e2", "c/e")
+            assert "c/d" in store
+            assert "c/e2" not in store
+            assert "c/e2/f" not in store
+            assert "c/e2/g" not in store
+            assert "c/e" not in store
+            assert "c/e/f" in store
+            assert "c/e/g" in store
+            store.rename("c", "c1/c2/c3")
+            assert "a" in store
+            assert "c" not in store
+            assert "c/d" not in store
+            assert "c/e" not in store
+            assert "c/e/f" not in store
+            assert "c/e/g" not in store
+            assert "c1" not in store
+            assert "c1/c2" not in store
+            assert "c1/c2/c3" not in store
+            assert "c1/c2/c3/d" in store
+            assert "c1/c2/c3/e" not in store
+            assert "c1/c2/c3/e/f" in store
+            assert "c1/c2/c3/e/g" in store
+            store.rename("c1/c2/c3", "c")
+            assert "c" not in store
+            assert "c/d" in store
+            assert "c/e" not in store
+            assert "c/e/f" in store
+            assert "c/e/g" in store
+            assert "c1" not in store
+            assert "c1/c2" not in store
+            assert "c1/c2/c3" not in store
+            assert "c1/c2/c3/d" not in store
+            assert "c1/c2/c3/e" not in store
+            assert "c1/c2/c3/e/f" not in store
+            assert "c1/c2/c3/e/g" not in store
+
+            # test rmdir (optional)
             store.rmdir("c/e")
             assert "c/d" in store
             assert "c/e/f" not in store