test,refactor: misc fixes for linters

pyproject.toml

@@ -51,6 +51,7 @@ ignore = [
  "ANN201", # Missing return type annotation for public function
  "ANN204", # Missing return type annotation for special method `__init__`
  "C408", # Unnecessary `dict` call (rewrite as a literal)
+ "PLR0913", # Too many arguments in function definition (n > 5)
  "S101", # Use of `assert` detected
 ]
 

src/darker/tests/test_main_isort.py

@@ -15,9 +15,11 @@
 pytestmark = pytest.mark.usefixtures("find_project_root_cache_clear")
 
 
-def test_isort_option_without_isort(git_repo, caplog):
+def test_isort_option_without_isort(git_repo): # noqa: ARG001
  """Without isort, provide isort install instructions and error."""
- with isort_present(False), patch.object(
+ # The `git_repo` fixture ensures test is not run in the Darker repository clone in
+ # CI builds. It helps avoid a NixOS test issue.
+ with isort_present(present=False), patch.object(
  darker.__main__, "isort", None,
  ), pytest.raises(MissingPackageError) as exc_info:
 
@@ -29,7 +31,7 @@ def test_isort_option_without_isort(git_repo, caplog):
  )
 
 
-@pytest.fixture
+@pytest.fixture()
 def run_isort(git_repo, monkeypatch, caplog, request):
  """Fixture for running Darker with requested arguments and a patched `isort`.