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

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

Author: pre-commit-ci[bot]

cmake_pc_hooks/_utils.py

@@ -88,8 +88,7 @@ def parse_args(self, args):
             '--read-json-db',
             action='store_true',
             help=(
-                'Run hooks on files found in compile_commands.json '
-                '(if found and in addition to files specified on CLI)'
+                'Run hooks on files found in compile_commands.json (if found and in addition to files specified on CLI)'
             ),
         )
 

tests/python/_argparse_test.py

@@ -35,7 +35,7 @@ def _add_simple_args(parser):
 # ------------------------------------------------------------------------------
 
 
-@pytest.fixture()
+@pytest.fixture
 def simple_toml_content():
     return dedent(
         """

tests/python/_cmake_test.py

@@ -238,8 +238,8 @@ def test_configure_cmake(mocker, tmp_path, clean_build, returncode, no_cmake_con
     mocker.patch(filelock_module_name, FileLock)
     InterProcessReaderWriterLock = mocker.MagicMock(fasteners.InterProcessReaderWriterLock)  # noqa: N806
     mocker.patch(interprocess_rw_lock_module_name, InterProcessReaderWriterLock)
-    _configure = mocker.Mock(return_value=returncode)
-    mocker.patch(internal_cmake_configure_name, _configure)
+    configure = mocker.Mock(return_value=returncode)
+    mocker.patch(internal_cmake_configure_name, configure)
     parse_log = mocker.patch('cmake_pc_hooks._cmake.CMakeCommand._parse_cmake_trace_log')
 
     # ----------------------------------
@@ -262,12 +262,12 @@ def test_configure_cmake(mocker, tmp_path, clean_build, returncode, no_cmake_con
     if no_cmake_configure:
         FileLock.assert_not_called()
         InterProcessReaderWriterLock.assert_not_called()
-        _configure.assert_not_called()
+        configure.assert_not_called()
         return
 
     FileLock.assert_called_once_with(build_dir / '_cmake_configure_try_lock')
     InterProcessReaderWriterLock.assert_called_once_with(build_dir / '_cmake_configure_lock')
-    _configure.assert_called_once_with(
+    configure.assert_called_once_with(
         lock_files=(InterProcessReaderWriterLock.call_args[0][0], FileLock.call_args[0][0]), clean_build=clean_build
     )
 
@@ -297,8 +297,8 @@ def timeout(blocking):  # noqa: ARG001
     interprocess_lock = mocker.MagicMock()
     InterProcessReaderWriterLock = mocker.MagicMock(return_value=interprocess_lock)  # noqa: N806
     mocker.patch(interprocess_rw_lock_module_name, InterProcessReaderWriterLock)
-    _configure = mocker.Mock(return_value=0)
-    mocker.patch(internal_cmake_configure_name, _configure)
+    configure = mocker.Mock(return_value=0)
+    mocker.patch(internal_cmake_configure_name, configure)
 
     # ----------------------------------
 
@@ -317,7 +317,7 @@ def timeout(blocking):  # noqa: ARG001
     FileLock.assert_called_once()
     InterProcessReaderWriterLock.assert_called_once()
     interprocess_lock.read_lock.assert_called_once()
-    _configure.assert_not_called()
+    configure.assert_not_called()
 
 
 def test_configure_invalid(mocker):
@@ -327,8 +327,8 @@ def test_configure_invalid(mocker):
     mocker.patch(filelock_module_name, FileLock)
     InterProcessReaderWriterLock = mocker.MagicMock(fasteners.InterProcessReaderWriterLock)  # noqa: N806
     mocker.patch(interprocess_rw_lock_module_name, InterProcessReaderWriterLock)
-    _configure = mocker.Mock(return_value=1)
-    mocker.patch(internal_cmake_configure_name, _configure)
+    configure = mocker.Mock(return_value=1)
+    mocker.patch(internal_cmake_configure_name, configure)
 
     # ----------------------------------
 

tests/python/conftest.py

@@ -8,7 +8,7 @@
 # ==============================================================================
 
 
-@pytest.fixture()
+@pytest.fixture
 def compile_commands(tmp_path):
     path = tmp_path / 'build' / 'compile_commands.json'