Modify project:fix and project:format to include ruff

Author: ArBridgeman

exasol/toolbox/nox/_format.py

@@ -35,17 +35,26 @@ def _pyupgrade(session: Session, config: Config, files: Iterable[str]) -> None:
     )
 
 
+def _ruff(session: Session, mode: Mode, files: Iterable[str]):
+    def command(*args: str) -> Iterable[str]:
+        return args if mode == Mode.Check else list(args) + ["--fix"]
+
+    session.run(*command("ruff", "check"), *files)
+
+
 @nox.session(name="project:fix", python=False)
 def fix(session: Session) -> None:
     """Runs all automated fixes on the code base"""
     py_files = python_files(PROJECT_CONFIG.root)
     _version(session, Mode.Fix)
     _pyupgrade(session, config=PROJECT_CONFIG, files=py_files)
+    _ruff(session, mode=Mode.Fix, files=py_files)
     _code_format(session, Mode.Fix, py_files)
 
 
 @nox.session(name="project:format", python=False)
 def fmt_check(session: Session) -> None:
     """Checks the project for correct formatting"""
     py_files = python_files(PROJECT_CONFIG.root)
+    _ruff(session, mode=Mode.Check, files=py_files)
     _code_format(session=session, mode=Mode.Check, files=py_files)

test/unit/nox/_format_test.py

@@ -0,0 +1,31 @@
+import pytest
+from nox.command import CommandFailed
+from toolbox.nox._format import _ruff
+
+from exasol.toolbox.nox._shared import Mode
+
+
+@pytest.fixture
+def file_with_unneeded_import(tmp_path):
+    file_path = tmp_path / "dummy_file.py"
+    file_path.write_text("import black")
+    return file_path
+
+
+class TestRuff:
+    @staticmethod
+    def test_mode_fix(nox_session, file_with_unneeded_import):
+        _ruff(
+            session=nox_session, mode=Mode.Fix, files=[str(file_with_unneeded_import)]
+        )
+        assert file_with_unneeded_import.read_text() == ""
+
+    @staticmethod
+    def test_mode_check(nox_session, file_with_unneeded_import):
+        with pytest.raises(CommandFailed):
+            _ruff(
+                session=nox_session,
+                mode=Mode.Check,
+                files=[str(file_with_unneeded_import)],
+            )
+        assert file_with_unneeded_import.read_text() == "import black"