Allow to configure isort through pyproject.toml

CHANGES.rst

@@ -5,6 +5,7 @@
 - Feature: Add support for ``-l``/``--line-length`` and ``-S``/``--skip-string-normalization``
 - Feature: ``--diff`` outputs a diff for each file on standard output
 - Feature: Require ``isort`` >= 5.0.1 and be compatible with it.
+- Feature: Allow to configure ``isort`` through pyproject.toml
 
 
 0.2.0 / 2020-03-11

pyproject.toml

@@ -11,3 +11,4 @@ include_trailing_comma = true
 force_grid_wrap = 0
 use_parentheses = true
 line_length = 88
+known_third_party = ["pytest"]

src/darker/__main__.py

@@ -61,8 +61,10 @@ def format_edited_parts(
 
  # 1. run isort
  if isort:
+ config = black_args.get("config")
+ line_length = black_args.get("line_length")
  edited_srcs = {
- src: apply_isort(edited_content)
+ src: apply_isort(edited_content, src, config, line_length)
  for src, edited_content in worktree_srcs.items()
  }
  else:

src/darker/import_sorting.py

@@ -1,4 +1,15 @@
 import logging
+import sys
+from pathlib import Path
+from typing import Optional
+
+from black import find_project_root
+
+if sys.version_info >= (3, 8):
+ from typing import TypedDict
+else:
+ from typing_extensions import TypedDict
+
 
 try:
  import isort
@@ -8,19 +19,30 @@
 logger = logging.getLogger(__name__)
 
 
-def apply_isort(content: str) -> str:
- isort_config_kwargs = dict(
- multi_line_output=3,
- include_trailing_comma=True,
- force_grid_wrap=0,
- use_parentheses=True,
- line_length=88,
- quiet=True,
- )
+class IsortArgs(TypedDict, total=False):
+ line_length: int
+ settings_file: str
+ settings_path: str
+
+
+def apply_isort(
+ content: str,
+ src: Optional[Path] = None,
+ config: Optional[str] = None,
+ line_length: Optional[int] = None,
+) -> str:
+ isort_args = IsortArgs()
+ if src and not config:
+ isort_args["settings_path"] = str(find_project_root((str(src),)))
+ if config:
+ isort_args["settings_file"] = config
+ if line_length:
+ isort_args["line_length"] = line_length
+
  logger.debug(
  "isort.code(code=..., {})".format(
- ", ".join(f"{k}={v!r}" for k, v in isort_config_kwargs.items())
+ ", ".join(f"{k}={v!r}" for k, v in isort_args.items())
  )
  )
- result: str = isort.code(code=content, **isort_config_kwargs)
+ result: str = isort.code(code=content, **isort_args)
  return result

src/darker/tests/test_import_sorting.py

@@ -1,3 +1,8 @@
+from pathlib import Path
+from textwrap import dedent
+
+import pytest
+
 from darker.import_sorting import apply_isort
 
 ORIGINAL_SOURCE = "import sys\nimport os\n"
@@ -8,3 +13,28 @@ def test_apply_isort():
  result = apply_isort(ORIGINAL_SOURCE)
 
  assert result == ISORTED_SOURCE
+
+
+@pytest.mark.parametrize("settings_file", [None, "pyproject.toml"])
+@pytest.mark.parametrize("line_length", [20, 60])
+def test_isort_config(monkeypatch, tmpdir, line_length, settings_file):
+ from black import find_project_root
+
+ find_project_root.cache_clear()
+ monkeypatch.chdir(tmpdir)
+ (tmpdir / 'pyproject.toml').write(
+ dedent(
+ f"""\
+ [tool.isort]
+ line_length = {line_length}
+ """
+ )
+ )
+
+ content = "from module import ab, cd, ef, gh, ij, kl, mn, op, qr, st, uv, wx, yz"
+ src = Path(tmpdir / "test1.py") if not settings_file else None
+ config = str(tmpdir / settings_file) if settings_file else None
+
+ actual = apply_isort(content, src, config)
+ expected = apply_isort(content, line_length=line_length)
+ assert actual == expected

src/darker/tests/test_main.py

@@ -22,14 +22,19 @@ def test_isort_option_without_isort(tmpdir, without_isort, caplog):
 
 
 @pytest.fixture
-def run_isort(git_repo, monkeypatch, caplog):
+def run_isort(git_repo, monkeypatch, caplog, request):
+ from black import find_project_root
+
+ find_project_root.cache_clear()
+
  monkeypatch.chdir(git_repo.root)
  paths = git_repo.add({'test1.py': 'original'}, commit='Initial commit')
  paths['test1.py'].write('changed')
+ args = getattr(request, "param", ())
  with patch.multiple(
  darker.__main__, run_black=Mock(return_value=[]), verify_ast_unchanged=Mock(),
  ), patch("darker.import_sorting.isort.code"):
- darker.__main__.main(["--isort", "./test1.py"])
+ darker.__main__.main(["--isort", "./test1.py", *args])
  return SimpleNamespace(
  isort_code=darker.import_sorting.isort.code, caplog=caplog
  )
@@ -39,15 +44,14 @@ def test_isort_option_with_isort(run_isort):
  assert "Please run" not in run_isort.caplog.text
 
 
-def test_isort_option_with_isort_calls_sortimports(run_isort):
+@pytest.mark.parametrize(
+ "run_isort,isort_args",
+ [((), {}), (("--line-length", "120"), {"line_length": 120})],
+ indirect=["run_isort"],
+)
+def test_isort_option_with_isort_calls_sortimports(tmpdir, run_isort, isort_args):
  run_isort.isort_code.assert_called_once_with(
- code="changed",
- force_grid_wrap=0,
- include_trailing_comma=True,
- line_length=88,
- multi_line_output=3,
- use_parentheses=True,
- quiet=True,
+ code="changed", settings_path=str(tmpdir), **isort_args
  )