Handle ruff config error (#25)

* Fail when ruff config is broken

* Add a test with broken ruff config

pytest_ruff/__init__.py

@@ -95,10 +95,14 @@ def check_file(path):
         "--force-exclude",
     ]
     child = Popen(command, stdout=PIPE, stderr=PIPE)
-    stdout, _ = child.communicate()
-    if stdout:
+    stdout, stderr = child.communicate()
+
+    if child.returncode == 1:
         raise RuffError(stdout.decode())
 
+    if child.returncode == 2:
+        raise RuffError(stderr.decode())
+
 
 def format_file(path):
     ruff = find_ruff_bin()
@@ -109,6 +113,9 @@ def format_file(path):
     if child.returncode == 1:
         raise RuffError("File would be reformatted")
 
+    if child.returncode == 2:
+        raise RuffError("Ruff terminated abnormally")
+
 
 def pytest_exception_interact(node, call, report):
     if isinstance(call.excinfo.value, RuffError):

tests/assets/broken_config/empty.py

@@ -0,0 +1 @@
+# Empty file to try to run ruff, but config in ruff.toml is broken.

tests/assets/broken_config/ruff.toml

@@ -0,0 +1 @@
+broken = true

tests/test_plugin.py

@@ -79,3 +79,20 @@ def test_pytest_ruff_noformat():
     ).communicate()
     assert err.decode() == ""
     assert "File would be reformatted" not in out.decode("utf-8")
+
+
+def test_broken_ruff_config():
+    process = subprocess.Popen(
+        [
+            sys.executable,
+            "-m",
+            "pytest",
+            "--ruff",
+            "tests/assets/broken_config/empty.py",
+        ],
+        stdout=subprocess.PIPE,
+        stderr=subprocess.PIPE,
+    )
+    out, err = process.communicate()
+    assert err.decode() == ""
+    assert "unknown field `broken`" in out.decode()