chore: pre-commit autoupdate (#373)

* chore: pre-commit autoupdate

updates:
- [github.com/rbubley/mirrors-prettier: v3.4.2 → v3.5.3](rbubley/mirrors-prettier@v3.4.2...v3.5.3)
- [github.com/astral-sh/ruff-pre-commit: v0.9.4 → v0.9.9](astral-sh/ruff-pre-commit@v0.9.4...v0.9.9)
- [github.com/streetsidesoftware/cspell-cli: v8.17.2 → v8.17.3](streetsidesoftware/cspell-cli@v8.17.2...v8.17.3)
- [github.com/jsh9/pydoclint: 0.6.0 → 0.6.2](jsh9/pydoclint@0.6.0...0.6.2)
- [github.com/pre-commit/mirrors-mypy.git: v1.14.1 → v1.15.0](https://github.com/pre-commit/mirrors-mypy.git/compare/v1.14.1...v1.15.0)

* Fix pydoclint failures

* Fix pydoclint failures: some more

* Fix fixtures

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: shatakshiiii <[email protected]>

Author: pre-commit-ci[bot]

.pre-commit-config.yaml

@@ -32,7 +32,7 @@ repos:
           )$
 
   - repo: https://github.com/rbubley/mirrors-prettier
-    rev: v3.4.2
+    rev: v3.5.3
     hooks:
       - id: prettier
         always_run: true
@@ -52,7 +52,7 @@ repos:
       - id: tox-ini-fmt
 
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.9.4
+    rev: v0.9.9
     hooks:
       - id: ruff
         args:
@@ -63,13 +63,13 @@ repos:
         types_or: [python, pyi]
 
   - repo: https://github.com/streetsidesoftware/cspell-cli
-    rev: v8.17.2
+    rev: v8.17.3
     hooks:
       - id: cspell
         name: Spell check with cspell
 
   - repo: https://github.com/jsh9/pydoclint
-    rev: "0.6.0"
+    rev: "0.6.2"
     hooks:
       - id: pydoclint
         # This allows automatic reduction of the baseline file when needed.
@@ -88,7 +88,7 @@ repos:
           - pyyaml
 
   - repo: https://github.com/pre-commit/mirrors-mypy.git
-    rev: v1.14.1
+    rev: v1.15.0
     hooks:
       - id: mypy
         additional_dependencies:

src/ansible_creator/resources/collection_project/tests/integration/test_integration.py.j2

@@ -10,6 +10,8 @@ def test_integration(molecule_scenario: MoleculeScenario) -> None:
 
     Args:
         molecule_scenario: The molecule scenario object
+    Raises:
+        AssertionError: If the molecule scenario test does not return a zero exit code.
     """
     proc = molecule_scenario.test()
     assert proc.returncode == 0

src/ansible_creator/resources/collection_project/tests/unit/test_basic.py.j2

@@ -2,5 +2,9 @@
 
 
 def test_basic() -> None:
-    """Dummy unit test that always passes."""
+    """Dummy unit test that always passes.
+
+    Raises:
+        AssertionError: If the assertion fails.
+    """
     assert bool(1) is True

tests/fixtures/collection/testorg/testcol/tests/integration/test_integration.py

@@ -10,6 +10,8 @@ def test_integration(molecule_scenario: MoleculeScenario) -> None:
 
     Args:
         molecule_scenario: The molecule scenario object
+    Raises:
+        AssertionError: If the molecule scenario test does not return a zero exit code.
     """
     proc = molecule_scenario.test()
     assert proc.returncode == 0

tests/fixtures/collection/testorg/testcol/tests/unit/test_basic.py

@@ -2,5 +2,9 @@
 
 
 def test_basic() -> None:
-    """Dummy unit test that always passes."""
+    """Dummy unit test that always passes.
+
+    Raises:
+        AssertionError: If the assertion fails.
+    """
     assert bool(1) is True

tests/integration/test_init.py

@@ -23,6 +23,9 @@ def test_run_help(cli: CliRunCallable) -> None:
 
     Args:
         cli: cli_run function.
+
+    Raises:
+        AssertionError: If the assertion fails.
     """
     # Get the path to the current python interpreter
     result = cli(f"{CREATOR_BIN} --help")
@@ -41,6 +44,9 @@ def test_run_no_subcommand(cli: CliRunCallable) -> None:
 
     Args:
         cli: cli_run function.
+
+    Raises:
+        AssertionError: If the assertion fails.
     """
     result = cli(str(CREATOR_BIN))
     assert result.returncode != 0
@@ -52,6 +58,9 @@ def test_run_init_no_input(cli: CliRunCallable) -> None:
 
     Args:
         cli: cli_run function.
+
+    Raises:
+        AssertionError: If the assertion fails.
     """
     result = cli(f"{CREATOR_BIN} init")
     assert result.returncode != 0
@@ -70,6 +79,9 @@ def test_run_deprecated_failure(command: str, cli: CliRunCallable) -> None:
     Args:
         command: Command to run.
         cli: cli_run function.
+
+    Raises:
+        AssertionError: If the assertion fails.
     """
     result = cli(f"{CREATOR_BIN} {command}")
     assert result.returncode != 0
@@ -95,6 +107,9 @@ def test_run_init_invalid_name(command: str, args: str, expected: str, cli: CliR
         args: Arguments to pass to the CLI.
         expected: Expected error message.
         cli: cli_run function.
+
+    Raises:
+        AssertionError: If the assertion fails.
     """
     result = cli(f"{CREATOR_BIN} init {command} {args}")
     assert result.returncode != 0
@@ -108,6 +123,9 @@ def test_run_init_basic(cli: CliRunCallable, tmp_path: Path) -> None:
     Args:
         cli: cli_run function.
         tmp_path: Temporary path.
+
+    Raises:
+        AssertionError: If the assertion fails.
     """
     final_dest = f"{tmp_path}/collections/ansible_collections"
     cli(f"mkdir -p {final_dest}")
@@ -149,6 +167,9 @@ def test_run_init_ee(cli: CliRunCallable, tmp_path: Path) -> None:
     Args:
         cli: cli_run function.
         tmp_path: Temporary path.
+
+    Raises:
+        AssertionError: If the assertion fails.
     """
     final_dest = f"{tmp_path}/ee_project"
     cli(f"mkdir -p {final_dest}")

tests/integration/test_lint.py

@@ -42,6 +42,9 @@ def test_lint_collection(
     Args:
         cli: CLI callable.
         monkeypatch: Monkeypatch fixture.
+
+    Raises:
+        AssertionError: If the assertion fails.
     """
     project_path = FIXTURES_DIR / "collection"
     monkeypatch.chdir(project_path)
@@ -77,6 +80,9 @@ def test_lint_playbook_project(
         tmp_path: Temporary path.
         cli: CLI callable.
         monkeypatch: Monkeypatch fixture.
+
+    Raises:
+        AssertionError: If the assertion fails.
     """
     req_path = str(
         FIXTURES_DIR / "project" / "playbook_project" / "collections" / "requirements.yml",

tests/units/test_add.py

@@ -1,4 +1,5 @@
 # cspell: ignore dcmp, subdcmp
+# pylint: disable=C0302
 """Unit tests for ansible-creator add."""
 
 from __future__ import annotations
@@ -122,6 +123,9 @@ def test_run_success_add_devfile(
         tmp_path: Temporary directory path.
         cli_args: Dictionary, partial Add class object.
         monkeypatch: Pytest monkeypatch fixture.
+
+    Raises:
+        AssertionError: If the assertion fails.
     """
     cli_args["resource_type"] = "devfile"
     add = Add(
@@ -196,6 +200,9 @@ def test_run_error_no_overwrite(
         capsys: Pytest fixture to capture stdout and stderr.
         tmp_path: Temporary directory path.
         cli_args: Dictionary, partial Add class object.
+
+    Raises:
+        AssertionError: If the assertion fails.
     """
     cli_args["resource_type"] = "devfile"
     add = Add(
@@ -248,6 +255,9 @@ def test_error_invalid_path(
 
     Args:
         cli_args: Dictionary, partial Add class object.
+
+    Raises:
+        AssertionError: If the assertion fails.
     """
     cli_args["resource_type"] = "devfile"
     cli_args["path"] = "/invalid"
@@ -269,6 +279,9 @@ def test_error_invalid_collection_path(
 
     Args:
         cli_args: Dictionary, partial Add class object.
+
+    Raises:
+        AssertionError: If the assertion fails.
     """
     cli_args["plugin_type"] = "lookup"
     add = Add(
@@ -297,6 +310,9 @@ def test_run_error_unsupported_resource_type(
     Args:
         cli_args: Dictionary, partial Add class object.
         monkeypatch: Pytest monkeypatch fixture.
+
+    Raises:
+        AssertionError: If the assertion fails.
     """
     cli_args["resource_type"] = "devfile"
     add = Add(
@@ -326,6 +342,9 @@ def test_run_success_add_devcontainer(
         tmp_path: Temporary directory path.
         cli_args: Dictionary, partial Add class object.
         monkeypatch: Pytest monkeypatch fixture.
+
+    Raises:
+        AssertionError: If the assertion fails.
     """
     # Set the resource_type to devcontainer
     cli_args["resource_type"] = "devcontainer"
@@ -393,6 +412,7 @@ def test_devcontainer_usability(
         cli_args: Dictionary, partial Add class object.
 
     Raises:
+        AssertionError: If the assertion fails.
         FileNotFoundError: If the 'npm' or 'docker' executable is not found in the PATH.
     """
     # Set the resource_type to devcontainer
@@ -467,6 +487,9 @@ def test_run_success_add_plugin_filter(
         tmp_path: Temporary directory path.
         cli_args: Dictionary, partial Add class object.
         monkeypatch: Pytest monkeypatch fixture.
+
+    Raises:
+        AssertionError: If the assertion fails.
     """
     cli_args["plugin_type"] = "filter"
     add = Add(
@@ -544,6 +567,9 @@ def test_run_success_add_plugin_lookup(
         tmp_path: Temporary directory path.
         cli_args: Dictionary, partial Add class object.
         monkeypatch: Pytest monkeypatch fixture.
+
+    Raises:
+        AssertionError: If the assertion fails.
     """
     cli_args["plugin_type"] = "lookup"
     add = Add(
@@ -620,6 +646,9 @@ def test_run_success_add_plugin_action(
         tmp_path: Temporary directory path.
         cli_args: Dictionary, partial Add class object.
         monkeypatch: Pytest monkeypatch fixture.
+
+    Raises:
+        AssertionError: If the assertion fails.
     """
     cli_args["plugin_type"] = "action"
     add = Add(
@@ -690,6 +719,9 @@ def test_run_success_add_plugin_module(
         tmp_path: Temporary directory path.
         cli_args: Dictionary, partial Add class object.
         monkeypatch: Pytest monkeypatch fixture.
+
+    Raises:
+        AssertionError: If the assertion fails.
     """
     cli_args["plugin_type"] = "module"
     add = Add(
@@ -767,6 +799,9 @@ def test_run_error_plugin_no_overwrite(
         tmp_path: Temporary directory path.
         cli_args: Dictionary, partial Add class object.
         monkeypatch: Pytest monkeypatch fixture.
+
+    Raises:
+        AssertionError: If the assertion fails.
     """
     cli_args["plugin_type"] = "lookup"
     add = Add(
@@ -823,6 +858,9 @@ def test_run_error_unsupported_plugin_type(
     Args:
         cli_args: Dictionary, partial Add class object.
         monkeypatch: Pytest monkeypatch fixture.
+
+    Raises:
+        AssertionError: If the assertion fails.
     """
     add = Add(
         Config(**cli_args),
@@ -860,6 +898,9 @@ def test_run_success_add_execution_env(
         tmp_path: Temporary directory path.
         cli_args: Dictionary, partial Add class object.
         monkeypatch: Pytest monkeypatch fixture.
+
+    Raises:
+        AssertionError: If the assertion fails.
     """
     # Set the resource_type to execution-environment
     cli_args["resource_type"] = "execution-environment"
@@ -916,6 +957,9 @@ def test_update_galaxy_dependency(tmp_path: Path, cli_args: ConfigDict) -> None:
     Args:
         tmp_path: Temporary directory path.
         cli_args: Dictionary, partial Add class object.
+
+    Raises:
+        AssertionError: If the assertion fails.
     """
     galaxy_file = tmp_path / "galaxy.yml"
     initial_data: dict[str, Any]

tests/units/test_argparse_help.py

@@ -10,7 +10,11 @@
 
 
 def test_custom_help_single() -> None:
-    """Test the custom help formatter with single."""
+    """Test the custom help formatter with single.
+
+    Raises:
+        AssertionError: If the assertion fails.
+    """
     parser = argparse.ArgumentParser(formatter_class=CustomHelpFormatter)
     parser.add_argument("--foo", help="foo help")
     help_text = parser.format_help()
@@ -19,7 +23,11 @@ def test_custom_help_single() -> None:
 
 
 def test_custom_help_double() -> None:
-    """Test the custom help formatter with double."""
+    """Test the custom help formatter with double.
+
+    Raises:
+        AssertionError: If the assertion fails.
+    """
     parser = argparse.ArgumentParser(formatter_class=CustomHelpFormatter)
     parser.add_argument("-f", "--foo", help="foo help")
     help_text = parser.format_help()