diff --git a/autohooks/plugins/ruff/ruff.py b/autohooks/plugins/ruff/ruff.py index e03bddf..12a537f 100644 --- a/autohooks/plugins/ruff/ruff.py +++ b/autohooks/plugins/ruff/ruff.py @@ -37,9 +37,8 @@ def get_ruff_arguments(config: Optional[Config]) -> Iterable[str]: if not config: return DEFAULT_ARGUMENTS - ruff_config = get_ruff_config(config) arguments = ensure_iterable( - ruff_config.get_value("arguments", DEFAULT_ARGUMENTS) + config.get_value("arguments", DEFAULT_ARGUMENTS) ) return arguments diff --git a/tests/test_ruff.py b/tests/test_ruff.py index a48fb42..83f54ed 100644 --- a/tests/test_ruff.py +++ b/tests/test_ruff.py @@ -34,7 +34,9 @@ def test_ruff_installed(self): ), patch("importlib.util.find_spec", return_value=None): check_ruff_installed() - def test_get_ruff_config(self): + def test_get_ruff_config( + self, + ): config_path = get_test_config_path("pyproject.test.toml") self.assertTrue(config_path.is_file()) @@ -50,12 +52,19 @@ def test_get_default_ruff_arguments(self): args = get_ruff_arguments(None) self.assertEqual(args, DEFAULT_ARGUMENTS) - def test_get_ruff_arguments(self): + @patch("autohooks.plugins.ruff.ruff.get_ruff_config") + def test_get_ruff_arguments( + self, + _get_ruff_config: MagicMock, + ): config_path = get_test_config_path("pyproject.test.toml") args = get_ruff_arguments( - load_config_from_pyproject_toml(config_path).get_config() + load_config_from_pyproject_toml(config_path) + .get_config() + .get("tool", "autohooks", "plugins", "ruff") ) self.assertEqual(args, ["--test", "foo,bar", "--foo", "bar"]) + _get_ruff_config.assert_not_called() @patch("autohooks.plugins.ruff.ruff.get_staged_status") def test_precommit_no_files( @@ -79,7 +88,7 @@ def test_precommit_errors( error_mock: MagicMock, out_mock: MagicMock, ok_mock: MagicMock, - _get_ruff_config, + _get_ruff_config: MagicMock, get_ruff_arguments_mock: MagicMock, ): code = """import subprocess