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

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

Author: pre-commit-ci[bot]

tests/unit/command_runner/test_command_runner.py

@@ -7,8 +7,6 @@
 from unittest.mock import MagicMock
 from unittest.mock import patch
 
-import pytest
-
 from ansible_navigator.command_runner.command_runner import Command
 from ansible_navigator.command_runner.command_runner import CommandRunner
 from ansible_navigator.command_runner.command_runner import run_command
@@ -84,7 +82,10 @@ def test_success(self, mock_run: MagicMock) -> None:
     def test_failure(self, mock_run: MagicMock) -> None:
         """Test run_command with CalledProcessError."""
         mock_run.side_effect = subprocess.CalledProcessError(
-            returncode=1, cmd="false", output="out", stderr="err",
+            returncode=1,
+            cmd="false",
+            output="out",
+            stderr="err",
         )
 
         cmd = Command(identity="t", command="false", post_process=_noop)

tests/unit/test_content_defs.py

@@ -4,7 +4,6 @@
 
 from dataclasses import dataclass
 
-from ansible_navigator.content_defs import CFormat
 from ansible_navigator.content_defs import ContentBase
 from ansible_navigator.content_defs import ContentFormat
 from ansible_navigator.content_defs import ContentView

tests/unit/test_image_introspection_parsers.py

@@ -57,7 +57,7 @@ def test_leading_space(self) -> None:
 
     def test_regex_separator(self) -> None:
         """Test with regex separator."""
-        key, delim, content = CmdParser.re_partition("key: value", r":\s+")
+        key, _delim, content = CmdParser.re_partition("key: value", r":\s+")
         assert key == "key"
         assert content == "value"
 

tests/unit/test_steps.py

@@ -5,8 +5,8 @@
 import pytest
 
 from ansible_navigator.steps import Step
-from ansible_navigator.steps import StepType
 from ansible_navigator.steps import Steps
+from ansible_navigator.steps import StepType
 from ansible_navigator.steps import TypedStep
 
 

tests/unit/ui_framework/test_validators.py

@@ -3,7 +3,7 @@
 from __future__ import annotations
 
 from dataclasses import dataclass
-from pathlib import Path
+from typing import TYPE_CHECKING
 
 import pytest
 
@@ -12,6 +12,10 @@
 from ansible_navigator.ui_framework.validators import Validation
 
 
+if TYPE_CHECKING:
+    from pathlib import Path
+
+
 class TestFieldValidatorsHttp:
     """Tests for the http validator."""
 
@@ -362,7 +366,9 @@ def test_valid_selection(self) -> None:
         """Test valid number of selections."""
         choices = [MockCheckbox(checked=True), MockCheckbox(checked=False)]
         result = FieldValidators.some_of_or_none(
-            choices=choices, min_selected=1, max_selected=2,
+            choices=choices,
+            min_selected=1,
+            max_selected=2,
         )
         assert isinstance(result, Validation)
         assert result.error_msg == ""
@@ -371,7 +377,9 @@ def test_too_few_selected(self) -> None:
         """Test too few selections."""
         choices = [MockCheckbox(checked=False), MockCheckbox(checked=False)]
         result = FieldValidators.some_of_or_none(
-            choices=choices, min_selected=1, max_selected=2,
+            choices=choices,
+            min_selected=1,
+            max_selected=2,
         )
         assert isinstance(result, Validation)
         assert result.error_msg != ""
@@ -382,7 +390,9 @@ def test_type_error(self) -> None:
 
         with pytest.raises(TypeError):
             FieldValidators.some_of_or_none(
-                choices=unknown, min_selected=unknown, max_selected=unknown,
+                choices=unknown,
+                min_selected=unknown,
+                max_selected=unknown,
             )
 
 

tests/unit/utils/test_ansi.py

@@ -2,10 +2,6 @@
 
 from __future__ import annotations
 
-import sys
-
-from unittest.mock import patch
-
 import pytest
 
 from ansible_navigator.utils.ansi import blank_line
@@ -136,6 +132,7 @@ def test_prompt_enter(monkeypatch: pytest.MonkeyPatch) -> None:
 
 def test_prompt_enter_keyboard_interrupt(monkeypatch: pytest.MonkeyPatch) -> None:
     """Test prompt_enter exits on KeyboardInterrupt."""
+
     def raise_keyboard_interrupt(_: str) -> None:
         raise KeyboardInterrupt
 
@@ -165,6 +162,7 @@ def test_prompt_yn_no(monkeypatch: pytest.MonkeyPatch) -> None:
 
 def test_prompt_yn_keyboard_interrupt(monkeypatch: pytest.MonkeyPatch) -> None:
     """Test prompt_yn exits on KeyboardInterrupt."""
+
     def raise_keyboard_interrupt(_: str) -> None:
         raise KeyboardInterrupt
 

tests/unit/utils/test_definitions.py

@@ -126,28 +126,34 @@ def test_single_message(self) -> None:
 
     def test_multiple_same_prefix(self) -> None:
         """Test with multiple messages of same prefix."""
-        msgs = ExitMessages(messages=[
-            ExitMessage(message="err1"),
-            ExitMessage(message="err2"),
-        ])
+        msgs = ExitMessages(
+            messages=[
+                ExitMessage(message="err1"),
+                ExitMessage(message="err2"),
+            ]
+        )
         result = msgs.to_strings(color=False, width=80)
         assert len(result) >= 2
 
     def test_different_prefixes_with_separator(self) -> None:
         """Test messages with different prefixes get separator."""
-        msgs = ExitMessages(messages=[
-            ExitMessage(message="error", prefix=ExitPrefix.ERROR),
-            ExitMessage(message="note", prefix=ExitPrefix.NOTE),
-        ])
+        msgs = ExitMessages(
+            messages=[
+                ExitMessage(message="error", prefix=ExitPrefix.ERROR),
+                ExitMessage(message="note", prefix=ExitPrefix.NOTE),
+            ]
+        )
         result = msgs.to_strings(color=False, width=80)
         assert "" in result
 
     def test_hint_follows_without_break(self) -> None:
         """Test hint after error doesn't get blank line separator."""
-        msgs = ExitMessages(messages=[
-            ExitMessage(message="error", prefix=ExitPrefix.ERROR),
-            ExitMessage(message="try this", prefix=ExitPrefix.HINT),
-        ])
+        msgs = ExitMessages(
+            messages=[
+                ExitMessage(message="error", prefix=ExitPrefix.ERROR),
+                ExitMessage(message="try this", prefix=ExitPrefix.HINT),
+            ]
+        )
         result = msgs.to_strings(color=False, width=80)
         assert "" not in result
 

tests/unit/utils/test_functions_extended.py

@@ -4,8 +4,7 @@
 
 import shutil
 
-from pathlib import Path
-from typing import Any
+from typing import TYPE_CHECKING
 from unittest.mock import patch
 
 import pytest
@@ -29,6 +28,10 @@
 from ansible_navigator.utils.functions import to_list
 
 
+if TYPE_CHECKING:
+    from pathlib import Path
+
+
 def test_check_for_ansible_found(monkeypatch: pytest.MonkeyPatch) -> None:
     """Test check_for_ansible when ansible-playbook is found."""
     monkeypatch.setattr(shutil, "which", lambda _: "/usr/bin/ansible-playbook")
@@ -71,7 +74,9 @@ def test_clear_screen_vscode(
     """Test clear_screen prints blank lines in vscode terminal."""
     monkeypatch.setenv("TERM_PROGRAM", "vscode")
     fake_size = type("TermSize", (), {"lines": 5, "columns": 80})()
-    with patch("ansible_navigator.utils.functions.shutil.get_terminal_size", return_value=fake_size):
+    with patch(
+        "ansible_navigator.utils.functions.shutil.get_terminal_size", return_value=fake_size
+    ):
         clear_screen()
     captured = capsys.readouterr()
     assert captured.out.count("\n") == 5
@@ -91,22 +96,28 @@ def test_clear_screen_normal(
 def test_console_width_small() -> None:
     """Test console_width with small terminal."""
     fake_size = type("TermSize", (), {"columns": 60})()
-    with patch("ansible_navigator.utils.functions.shutil.get_terminal_size", return_value=fake_size):
+    with patch(
+        "ansible_navigator.utils.functions.shutil.get_terminal_size", return_value=fake_size
+    ):
         assert console_width() == 60
 
 
 def test_console_width_medium() -> None:
     """Test console_width with medium terminal."""
     fake_size = type("TermSize", (), {"columns": 120})()
-    with patch("ansible_navigator.utils.functions.shutil.get_terminal_size", return_value=fake_size):
+    with patch(
+        "ansible_navigator.utils.functions.shutil.get_terminal_size", return_value=fake_size
+    ):
         result = console_width()
     assert 80 <= result <= 120
 
 
 def test_console_width_large() -> None:
     """Test console_width with large terminal."""
     fake_size = type("TermSize", (), {"columns": 200})()
-    with patch("ansible_navigator.utils.functions.shutil.get_terminal_size", return_value=fake_size):
+    with patch(
+        "ansible_navigator.utils.functions.shutil.get_terminal_size", return_value=fake_size
+    ):
         assert console_width() == 132
 
 
@@ -261,7 +272,7 @@ def test_templar_simple() -> None:
 
 def test_templar_error() -> None:
     """Test templar with undefined variable."""
-    errors, result = templar("{{ undefined_var }}", {})
+    errors, _result = templar("{{ undefined_var }}", {})
     assert len(errors) > 0
 
 

tests/unit/utils/test_json_schema.py

@@ -117,10 +117,12 @@ def test_invalid_data(self) -> None:
 
     def test_schema_as_json_string(self) -> None:
         """Test validate with schema as JSON string."""
-        schema_str = json.dumps({
-            "type": "object",
-            "properties": {"age": {"type": "integer"}},
-        })
+        schema_str = json.dumps(
+            {
+                "type": "object",
+                "properties": {"age": {"type": "integer"}},
+            }
+        )
         errors = validate(schema_str, {"age": 25})
         assert errors == []
 

tests/unit/utils/test_key_value_store_extended.py

@@ -2,13 +2,17 @@
 
 from __future__ import annotations
 
-from pathlib import Path
+from typing import TYPE_CHECKING
 
 import pytest
 
 from ansible_navigator.utils.key_value_store import KeyValueStore
 
 
+if TYPE_CHECKING:
+    from pathlib import Path
+
+
 @pytest.fixture
 def kvs(tmp_path: Path) -> KeyValueStore:
     """Provide a fresh KeyValueStore instance.