Merge pull request #92 from ArcanaFramework/touch-ups

Minor touch  ups

Author: tclose

.pre-commit-config.yaml

@@ -23,13 +23,19 @@ repos:
         args:
           - --ignore-words=.codespell-ignorewords
   - repo: https://github.com/PyCQA/flake8
-    rev: 4.0.1
+    rev: 7.0.0
     hooks:
       - id: flake8
   - repo: https://github.com/pre-commit/mirrors-mypy
     rev: v1.11.2
     hooks:
       - id: mypy
-        args: [--strict, --install-types, --non-interactive, --no-warn-unused-ignores]
+        args:
+          [
+            --strict,
+            --install-types,
+            --non-interactive,
+            --no-warn-unused-ignores,
+          ]
         exclude: tests
         additional_dependencies: [pytest, attrs, imageio, pydicom]

fileformats/core/collection.py

@@ -44,7 +44,8 @@ def _validate_required_content_types(self) -> None:
                         return
         assert not_found
         raise FormatMismatchError(
-            f"Did not find the required content types, {not_found}, in {self}"
+            f"Did not find the required content types, {not_found}, in {self}, paths: "
+            f"{[str(p) for p in self.content_fspaths]}"
         )
 
     @classproperty

fileformats/core/fileset.py

@@ -101,6 +101,8 @@ def __init__(
         metadata: ty.Optional[ty.Dict[str, ty.Any]] = None,
         **load_kwargs: ty.Any,
     ):
+        if not fspaths:
+            raise ValueError("No file-system paths provided to FileSet")
         self._explicit_metadata = metadata
         self._load_kwargs = load_kwargs
         self._validate_class()
@@ -800,7 +802,7 @@ def byte_chunks(
         rel_path: str
             relative path to either 'relative_to' arg or common base path for each file
             in the file set
-        byte_iter : Generator[bytes]
+        byte_iter : Generator[bytes, None, None]
             an iterator over the bytes contents of the file, chunked into 'chunk_len'
             chunks
         """

fileformats/core/utils.py

@@ -182,6 +182,12 @@ def __init__(self, imported: bool, pkg: ty.Optional[str], pypi: ty.Optional[str]
         self.pkg = pkg
         self.pypi = pypi
 
+    def __repr__(self) -> str:
+        return (
+            f"{type(self).__name__}(pkg={self.pkg}, pypi={self.pypi}, "
+            f"imported={self.imported})"
+        )
+
 
 def import_extras_module(klass: ty.Type["fileformats.core.DataType"]) -> ExtrasModule:
     """Attempt to load extras module corresponding to the provided class's module