[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Author: pre-commit-ci[bot]

pyglotaran_extras/__init__.py

@@ -73,5 +73,6 @@
 
 SCRIPT_DIR = _find_script_dir_at_import(__file__)
 """User script dir determined during import."""
+
 CONFIG = load_config(SCRIPT_DIR)
 """Global config instance."""

pyglotaran_extras/types.py

@@ -45,12 +45,12 @@ class CyclerColor(TypedDict):
 
 DatasetConvertible: TypeAlias = xr.Dataset | xr.DataArray | str | Path
 """Types of data which can be converted to a dataset."""
+
 ResultLike: TypeAlias = (
     Result | DatasetConvertible | Mapping[str, DatasetConvertible] | Sequence[DatasetConvertible]
 )
 """Result like data which can be converted to a per dataset mapping."""
 
-
 BuiltinSubPlotLabelFormatFunctionKey: TypeAlias = Literal[
     "number", "upper_case_letter", "lower_case_letter"
 ]

pyproject.toml

@@ -76,28 +76,55 @@ docs = [
 ]
 
 [tool.hatch]
+version.path = "pyglotaran_extras/__init__.py"
 build.targets.sdist.include = [
-  "/pyglotaran_extras",
   "/docs",
+  "/pyglotaran_extras",
   "/tests",
   "changelog.md",
 ]
-version.path = "pyglotaran_extras/__init__.py"
 
 [tool.docformatter]
 black = true
 wrap-summaries = 99
 wrap-descriptions = 99
 
 # For now this is not used because pydoclint does not support toml when used with flake8
-
 [tool.pydoclint]
 skip-checking-short-docstrings = false
 style = "numpy"
 exclude = "^(docs/|tests?/)"
 require-return-section-when-returning-none = false
 allow-init-docstring = true
 
+[tool.interrogate]
+fail-under = 100
+ignore-init-module = true
+exclude = [
+  "docs",
+  "tests/*",
+]
+
+[tool.mypy]
+ignore_missing_imports = true
+python_version = "3.10"
+disallow_untyped_calls = true
+disallow_untyped_defs = true
+disallow_incomplete_defs = true
+check_untyped_defs = true
+warn_unused_ignores = true
+plugins = [
+  "pydantic.mypy",
+]
+scripts_are_modules = true
+warn_unused_configs = true
+overrides = [
+  { module = "tests.*", disallow_untyped_defs = false, disallow_incomplete_defs = false },
+]
+no_implicit_optional = true
+no_implicit_reexport = true
+show_error_codes = true
+
 [tool.coverage]
 run.branch = true
 run.omit = [
@@ -123,31 +150,3 @@ report.exclude_lines = [
   "raise AssertionError",
   "raise NotImplementedError",
 ]
-
-[tool.mypy]
-plugins = [
-  "pydantic.mypy",
-]
-python_version = "3.10"
-ignore_missing_imports = true
-scripts_are_modules = true
-show_error_codes = true
-warn_unused_ignores = true
-no_implicit_optional = true
-disallow_incomplete_defs = true
-disallow_untyped_defs = true
-disallow_untyped_calls = true
-no_implicit_reexport = true
-warn_unused_configs = true
-check_untyped_defs = true
-overrides = [
-  { module = "tests.*", disallow_incomplete_defs = false, disallow_untyped_defs = false },
-]
-
-[tool.interrogate]
-exclude = [
-  "docs",
-  "tests/*",
-]
-ignore-init-module = true
-fail-under = 100

tests/config/test_config.py

@@ -76,7 +76,6 @@ def test_config_reset(tmp_path: Path, test_config_values: dict[str, Any]):
 
     If another config is passed all values (including source files) are reset.
     """
-
     test_config_path = tmp_path / f"{CONFIG_FILE_STEM}.yml"
     test_config = Config.model_validate(test_config_values)
     test_config._source_files = [test_config_path]
@@ -97,7 +96,6 @@ def test_config_reset(tmp_path: Path, test_config_values: dict[str, Any]):
 
 def test_config_reload(tmp_path: Path, test_config_values: dict[str, Any]):
     """Config values get reloaded from file."""
-
     test_config_path = tmp_path / f"{CONFIG_FILE_STEM}.yml"
     test_config = Config.model_validate(test_config_values)
     test_config._source_files = [test_config_path]
@@ -119,7 +117,6 @@ def test_config_reload(tmp_path: Path, test_config_values: dict[str, Any]):
 
 def test_config_load(tmp_path: Path, test_config_values: dict[str, Any], test_config_file: Path):
     """Loading config replaces its values and and source files."""
-
     test_config = Config()
     test_config._source_files = [test_config_file]
     test_config.reload()
@@ -358,8 +355,8 @@ def test_load_config_files(tmp_path: Path, test_config_file: Path):
 def test_merge_configs():
     """Check that the most right config overrides other values.
 
-    Since we already tested all permutations for 2 configs in ``test_config_merge``
-    this test can fucus on the case with more than 2 configs.
+    Since we already tested all permutations for 2 configs in ``test_config_merge`` this test can
+    fucus on the case with more than 2 configs.
     """
     assert merge_configs([]) == Config()
 

tests/config/test_plot_config.py

@@ -466,7 +466,6 @@ def func(
 
 def test_find_axes():
     """Get axes value from iterable of values."""
-
     data_array = xr.DataArray([[0, 1]], coords={"time": [0], "spectral": [500, 510]})
     data_set = xr.Dataset({"data": data_array})
 

tests/conftest.py

@@ -87,7 +87,6 @@ def result_sequential_spectral_decay():
 @pytest.fixture
 def mock_home(monkeypatch: pytest.MonkeyPatch, tmp_path: Path):
     """Mock ``pathlib.Path.home`` to return ``tmp_path/"home"``."""
-
     mock_home_path = tmp_path / "home"
     mock_home_path.mkdir()
 

tests/deprecation/test_deprecation_utils.py

@@ -125,7 +125,6 @@ def test_warn_deprecated():
 @pytest.mark.usefixtures("_pyglotaran_extras_1_0_0")
 def test_warn_deprecated_overdue_deprecation(monkeypatch: MonkeyPatch):
     """Current version is equal or bigger than drop_version."""
-
     with pytest.raises(OverdueDeprecationError) as excinfo:
         warn_deprecated(
             deprecated_qual_name_usage=DEPRECATION_QUAL_NAME,

tests/inspect/test_utils.py

@@ -40,12 +40,10 @@
             5,
             False,
             dedent(
-                """
-                <details >
+                """<details >
 
-                FOO
-                <br>
-                </details>"""
+                FOO <br> </details>
+                """
             ),
             id="heading no summary",
         ),
@@ -55,12 +53,10 @@
             None,
             True,
             dedent(
-                """
-                <details open>
+                """<details open>
 
-                FOO
-                <br>
-                </details>"""
+                FOO <br> </details>
+                """
             ),
             id="defaults open details",
         ),
@@ -70,15 +66,10 @@
             None,
             False,
             dedent(
-                """
-                <details >
-                <summary>
-                Bar
-                </summary>
+                """<details > <summary> Bar </summary>
 
-                FOO
-                <br>
-                </details>"""
+                FOO <br> </details>
+                """
             ),
             id="defaults with simple summary",
         ),
@@ -88,17 +79,10 @@
             5,
             False,
             dedent(
-                """
-                <details >
-                <summary>
-                <h5 style="display:inline;">
-                Bar
-                </h5>
-                </summary>
+                """<details > <summary> <h5 style="display:inline;"> Bar </h5> </summary>
 
-                FOO
-                <br>
-                </details>"""
+                FOO <br> </details>
+                """
             ),
             id="defaults with heading summary",
         ),

tests/io/test_setup_case_study.py

@@ -34,10 +34,8 @@ def test_get_script_dir_tmp_path(tmp_path: Path):
     """File in temp folder."""
     tmp_file = tmp_path / "foo.py"
     content = dedent(
-        """
-        from pyglotaran_extras.io.setup_case_study import get_script_dir
-        print(get_script_dir())
-        """
+        """From pyglotaran_extras.io.setup_case_study import get_script_dir
+        print(get_script_dir())"""
     )
     tmp_file.write_text(content)
     printed_result = subprocess.run(