chore(linters): Apply ruff autofixes

Author: MaxymVlasov

src/pre_commit_terraform/__main__.py

@@ -1,6 +1,7 @@
 """A runpy-style CLI entry-point module."""
 
-from sys import argv, exit as exit_with_return_code
+from sys import argv
+from sys import exit as exit_with_return_code
 
 from ._cli import invoke_cli_app
 

src/pre_commit_terraform/_cli.py

@@ -23,7 +23,7 @@ def invoke_cli_app(cli_args: list[str]) -> ReturnCodeType:
     parsed_cli_args = root_cli_parser.parse_args(cli_args)
     invoke_cli_app = cast_to(
         # FIXME: attempt typing per https://stackoverflow.com/a/75666611/595220
-        CLIAppEntryPointCallableType,
+        'CLIAppEntryPointCallableType',
         parsed_cli_args.invoke_cli_app,
     )
 

src/pre_commit_terraform/_cli_parsing.py

@@ -22,7 +22,9 @@ def attach_subcommand_parsers_to(root_cli_parser: ArgumentParser, /) -> None:
         required=True,
     )
     for subcommand_module in SUBCOMMAND_MODULES:
-        subcommand_parser = subcommand_parsers.add_parser(subcommand_module.CLI_SUBCOMMAND_NAME)
+        subcommand_parser = subcommand_parsers.add_parser(
+            subcommand_module.CLI_SUBCOMMAND_NAME,
+        )
         subcommand_parser.set_defaults(
             invoke_cli_app=subcommand_module.invoke_cli_app,
         )

src/pre_commit_terraform/_errors.py

@@ -6,8 +6,8 @@ class PreCommitTerraformBaseError(Exception):
 
 
 class PreCommitTerraformRuntimeError(
-        PreCommitTerraformBaseError,
-        RuntimeError,
+    PreCommitTerraformBaseError,
+    RuntimeError,
 ):
     """An exception representing a runtime error condition."""
 

src/pre_commit_terraform/_types.py

@@ -18,7 +18,8 @@ class CLISubcommandModuleProtocol(Protocol):
     """This constant contains a CLI."""
 
     def populate_argument_parser(
-            self, subcommand_parser: ArgumentParser,
+        self,
+        subcommand_parser: ArgumentParser,
     ) -> None:
         """Run a module hook for populating the subcommand parser."""
 

src/pre_commit_terraform/terraform_docs_replace.py

@@ -18,14 +18,20 @@ def populate_argument_parser(subcommand_parser: ArgumentParser) -> None:
         'replace the entire README.md file each time.'
     )
     subcommand_parser.add_argument(
-        '--dest', dest='dest', default='README.md',
+        '--dest',
+        dest='dest',
+        default='README.md',
     )
     subcommand_parser.add_argument(
-        '--sort-inputs-by-required', dest='sort', action='store_true',
+        '--sort-inputs-by-required',
+        dest='sort',
+        action='store_true',
         help='[deprecated] use --sort-by-required instead',
     )
     subcommand_parser.add_argument(
-        '--sort-by-required', dest='sort', action='store_true',
+        '--sort-by-required',
+        dest='sort',
+        action='store_true',
     )
     subcommand_parser.add_argument(
         '--with-aggregate-type-defaults',
@@ -50,9 +56,10 @@ def invoke_cli_app(parsed_cli_args: Namespace) -> ReturnCodeType:
     )
 
     dirs: list[str] = []
-    for filename in cast_to(list[str], parsed_cli_args.filenames):
-        if (os.path.realpath(filename) not in dirs and
-                (filename.endswith(".tf") or filename.endswith(".tfvars"))):
+    for filename in cast_to('list[str]', parsed_cli_args.filenames):
+        if os.path.realpath(filename) not in dirs and (
+            filename.endswith('.tf') or filename.endswith('.tfvars')
+        ):
             dirs.append(os.path.dirname(filename))
 
     retval = ReturnCode.OK
@@ -61,16 +68,15 @@ def invoke_cli_app(parsed_cli_args: Namespace) -> ReturnCodeType:
         try:
             procArgs = []
             procArgs.append('terraform-docs')
-            if cast_to(bool, parsed_cli_args.sort):
+            if cast_to('bool', parsed_cli_args.sort):
                 procArgs.append('--sort-by-required')
             procArgs.append('md')
-            procArgs.append("./{dir}".format(dir=dir))
+            procArgs.append(f'./{dir}')
             procArgs.append('>')
             procArgs.append(
-                './{dir}/{dest}'.
-                format(dir=dir, dest=cast_to(bool, parsed_cli_args.dest)),
+                f'./{dir}/{cast_to("bool", parsed_cli_args.dest)}',
             )
-            subprocess.check_call(" ".join(procArgs), shell=True)
+            subprocess.check_call(' '.join(procArgs), shell=True)
         except subprocess.CalledProcessError as e:
             print(e)
             retval = ReturnCode.ERROR

tests/pytest/_cli_test.py

@@ -1,13 +1,13 @@
 """Tests for the high-level CLI entry point."""
 
 from argparse import ArgumentParser, Namespace
-import pytest
 
+import pytest
 from pre_commit_terraform import _cli_parsing as _cli_parsing_mod
 from pre_commit_terraform._cli import invoke_cli_app
 from pre_commit_terraform._errors import (
-    PreCommitTerraformExit,
     PreCommitTerraformBaseError,
+    PreCommitTerraformExit,
     PreCommitTerraformRuntimeError,
 )
 from pre_commit_terraform._structs import ReturnCode
@@ -42,17 +42,19 @@
     ),
 )
 def test_known_interrupts(
-        capsys: pytest.CaptureFixture[str],
-        expected_stderr: str,
-        monkeypatch: pytest.MonkeyPatch,
-        raised_error: BaseException,
+    capsys: pytest.CaptureFixture[str],
+    expected_stderr: str,
+    monkeypatch: pytest.MonkeyPatch,
+    raised_error: BaseException,
 ) -> None:
     """Check that known interrupts are turned into return code 1."""
+
     class CustomCmdStub:
         CLI_SUBCOMMAND_NAME = 'sentinel'
 
         def populate_argument_parser(
-                self, subcommand_parser: ArgumentParser,
+            self,
+            subcommand_parser: ArgumentParser,
         ) -> None:
             return None
 
@@ -65,22 +67,24 @@ def invoke_cli_app(self, parsed_cli_args: Namespace) -> ReturnCodeType:
         [CustomCmdStub()],
     )
 
-    assert ReturnCode.ERROR == invoke_cli_app(['sentinel'])
+    assert invoke_cli_app(['sentinel']) == ReturnCode.ERROR
 
     captured_outputs = capsys.readouterr()
     assert captured_outputs.err == f'{expected_stderr !s}\n'
 
 
 def test_app_exit(
-        capsys: pytest.CaptureFixture[str],
-        monkeypatch: pytest.MonkeyPatch,
+    capsys: pytest.CaptureFixture[str],
+    monkeypatch: pytest.MonkeyPatch,
 ) -> None:
     """Check that an exit exception is re-raised."""
+
     class CustomCmdStub:
         CLI_SUBCOMMAND_NAME = 'sentinel'
 
         def populate_argument_parser(
-                self, subcommand_parser: ArgumentParser,
+            self,
+            subcommand_parser: ArgumentParser,
         ) -> None:
             return None
 

tests/pytest/terraform_docs_replace_test.py

@@ -3,13 +3,15 @@
 from argparse import ArgumentParser, Namespace
 from subprocess import CalledProcessError
 
-import pytest
 import pytest_mock
 
+import pytest
 from pre_commit_terraform._structs import ReturnCode
 from pre_commit_terraform.terraform_docs_replace import (
     invoke_cli_app,
     populate_argument_parser,
+)
+from pre_commit_terraform.terraform_docs_replace import (
     subprocess as replace_docs_subprocess_mod,
 )
 
@@ -24,8 +26,7 @@ def test_arg_parser_populated() -> None:
 def test_check_is_deprecated() -> None:
     """Verify that `replace-docs` shows a deprecation warning."""
     deprecation_msg_regex = (
-        r'^`terraform_docs_replace` hook is DEPRECATED\.'
-        'For migration.*$'
+        r'^`terraform_docs_replace` hook is DEPRECATED\.' 'For migration.*$'
     )
     with pytest.warns(UserWarning, match=deprecation_msg_regex):
         # not `pytest.deprecated_call()` due to this being a user warning
@@ -70,10 +71,10 @@ def test_check_is_deprecated() -> None:
     'pre_commit_terraform.terraform_docs_replace',
 )
 def test_control_flow_positive(
-        expected_cmds: list[str],
-        mocker: pytest_mock.MockerFixture,
-        monkeypatch: pytest.MonkeyPatch,
-        parsed_cli_args: Namespace,
+    expected_cmds: list[str],
+    mocker: pytest_mock.MockerFixture,
+    monkeypatch: pytest.MonkeyPatch,
+    parsed_cli_args: Namespace,
 ) -> None:
     """Check that the subcommand's happy path works."""
     check_call_mock = mocker.Mock()
@@ -83,10 +84,10 @@ def test_control_flow_positive(
         check_call_mock,
     )
 
-    assert ReturnCode.OK == invoke_cli_app(parsed_cli_args)
+    assert invoke_cli_app(parsed_cli_args) == ReturnCode.OK
 
     executed_commands = [
-        cmd for ((cmd, ), _shell) in check_call_mock.call_args_list
+        cmd for ((cmd,), _shell) in check_call_mock.call_args_list
     ]
 
     assert len(expected_cmds) == check_call_mock.call_count
@@ -98,8 +99,8 @@ def test_control_flow_positive(
     'pre_commit_terraform.terraform_docs_replace',
 )
 def test_control_flow_negative(
-        mocker: pytest_mock.MockerFixture,
-        monkeypatch: pytest.MonkeyPatch,
+    mocker: pytest_mock.MockerFixture,
+    monkeypatch: pytest.MonkeyPatch,
 ) -> None:
     """Check that the subcommand's error processing works."""
     parsed_cli_args = Namespace(
@@ -118,6 +119,6 @@ def test_control_flow_negative(
         check_call_mock,
     )
 
-    assert ReturnCode.ERROR == invoke_cli_app(parsed_cli_args)
+    assert invoke_cli_app(parsed_cli_args) == ReturnCode.ERROR
 
     check_call_mock.assert_called_once_with(expected_cmd, shell=True)