reformatting, fixes after linting

Author: forman

examples/zappend-demo.ipynb

@@ -18,12 +18,10 @@
    "outputs": [],
    "source": [
     "import glob\n",
-    "import math\n",
     "import os\n",
     "import os.path\n",
     "import shutil\n",
     "\n",
-    "import numpy as np\n",
     "import xarray as xr"
    ]
   },
@@ -2951,10 +2949,7 @@
    "id": "e3b5f156-88c7-4ce1-908e-88ddc7b7e18c",
    "metadata": {},
    "outputs": [],
-   "source": [
-    "from IPython.display import Markdown\n",
-    "from zappend.config import get_config_schema"
-   ]
+   "source": []
   },
   {
    "cell_type": "code",
@@ -3023,7 +3018,7 @@
    "metadata": {},
    "outputs": [],
    "source": [
-    "config_path = f\"zappend-config.yaml\""
+    "config_path = \"zappend-config.yaml\""
    ]
   },
   {

tests/config/test_attrs.py

@@ -7,14 +7,17 @@
 import unittest
 
 import numpy as np
-import xarray as xr
 import pytest
+import xarray as xr
+
+from zappend.config.attrs import (
+    ConfigAttrsUserFunctions,
+    eval_dyn_config_attrs,
+    eval_expr,
+    get_dyn_config_attrs_env,
+    has_dyn_config_attrs,
+)
 
-from zappend.config.attrs import ConfigAttrsUserFunctions
-from zappend.config.attrs import eval_dyn_config_attrs
-from zappend.config.attrs import eval_expr
-from zappend.config.attrs import get_dyn_config_attrs_env
-from zappend.config.attrs import has_dyn_config_attrs
 from ..helpers import make_test_dataset
 
 

tests/config/test_config.py

@@ -11,8 +11,8 @@
 from zappend.context import Context
 from zappend.fsutil.fileobj import FileObj
 from zappend.slice import SliceSource
-from ..helpers import clear_memory_fs
-from ..helpers import make_test_dataset
+
+from ..helpers import clear_memory_fs, make_test_dataset
 
 
 class ConfigTest(unittest.TestCase):
@@ -151,8 +151,7 @@ def test_slice_source_as_type(self):
         with pytest.raises(
             TypeError,
             match=(
-                "slice_source must a callable"
-                " or the fully qualified name of a callable"
+                "slice_source must a callable or the fully qualified name of a callable"
             ),
         ):
             Config(

tests/config/test_normalize.py

@@ -10,10 +10,9 @@
 import pytest
 import yaml
 
-from zappend.config import exclude_from_config
-from zappend.config import merge_configs
-from zappend.config import normalize_config
+from zappend.config import exclude_from_config, merge_configs, normalize_config
 from zappend.fsutil import FileObj
+
 from ..helpers import clear_memory_fs
 
 
@@ -88,7 +87,7 @@ def test_it_raises_if_config_is_not_object(self):
         file_obj.write("what?")
         with pytest.raises(
             TypeError,
-            match="Invalid configuration:" " memory://config.yaml: object expected",
+            match="Invalid configuration: memory://config.yaml: object expected",
         ):
             normalize_config(file_obj)
 

tests/config/test_validate.py

@@ -23,7 +23,7 @@ def test_validate_versions_fail(self):
         config = {"zarr_version": 1}
         with pytest.raises(
             ValueError,
-            match="Invalid configuration:" " 2 was expected for zarr_version",
+            match="Invalid configuration: 2 was expected for zarr_version",
         ):
             validate_config(config)
 

tests/contrib/test_levels.py

@@ -8,11 +8,10 @@
 import pytest
 import xarray as xr
 
-from zappend.fsutil import FileObj
+from tests.helpers import clear_memory_fs, make_test_dataset
 from zappend.contrib import write_levels
 from zappend.contrib.levels import get_variables_config
-from tests.helpers import clear_memory_fs
-from tests.helpers import make_test_dataset
+from zappend.fsutil import FileObj
 
 try:
     # noinspection PyUnresolvedReferences

tests/fsutil/test_fileobj.py

@@ -2,8 +2,8 @@
 # Permissions are hereby granted under the terms of the MIT License:
 # https://opensource.org/licenses/MIT.
 
-import unittest
 import os.path
+import unittest
 import uuid
 
 import fsspec
@@ -27,7 +27,7 @@ def test_repr(self):
             "FileObj('memory://test.zarr')", repr(FileObj("memory://test.zarr"))
         )
         self.assertEqual(
-            "FileObj('memory://test.zarr'," " storage_options={'asynchronous': False})",
+            "FileObj('memory://test.zarr', storage_options={'asynchronous': False})",
             repr(
                 FileObj("memory://test.zarr", storage_options=dict(asynchronous=False))
             ),

tests/fsutil/test_path.py

@@ -6,8 +6,7 @@
 
 import pytest
 
-from zappend.fsutil.path import split_components
-from zappend.fsutil.path import split_parent
+from zappend.fsutil.path import split_components, split_parent
 
 
 class SplitFilenameTest(unittest.TestCase):

tests/fsutil/test_transaction.py

@@ -8,8 +8,8 @@
 import pytest
 
 from zappend.fsutil.fileobj import FileObj
-from zappend.fsutil.transaction import Transaction
-from zappend.fsutil.transaction import ROLLBACK_FILE
+from zappend.fsutil.transaction import ROLLBACK_FILE, Transaction
+
 from ..helpers import clear_memory_fs
 
 
@@ -175,7 +175,7 @@ def test_it_raises_if_not_used_with_with(self):
         transaction = Transaction(test_root, rollback_dir)
         with pytest.raises(
             ValueError,
-            match="Transaction instance must be" " used with the 'with' statement",
+            match="Transaction instance must be used with the 'with' statement",
         ):
             transaction._add_rollback_action("delete_file", "path", None)
 

tests/slice/test_callable.py

@@ -7,8 +7,7 @@
 import pytest
 
 from zappend.context import Context
-from zappend.slice.callable import import_attribute
-from zappend.slice.callable import to_slice_args
+from zappend.slice.callable import import_attribute, to_slice_args
 
 
 class ToSliceArgsTest(unittest.TestCase):