Merge pull request #25721 from QuLogic/skip-black-test

jklymak · web-flow · commit 59373183f9f6 · 2023-04-18T18:27:34.000-07:00
diff --git a/lib/matplotlib/tests/test_pyplot.py b/lib/matplotlib/tests/test_pyplot.py
@@ -12,6 +12,8 @@
 
 
 def test_pyplot_up_to_date(tmpdir):
+    pytest.importorskip("black")
+
     gen_script = Path(mpl.__file__).parents[2] / "tools/boilerplate.py"
     if not gen_script.exists():
         pytest.skip("boilerplate.py not found")
diff --git a/tools/boilerplate.py b/tools/boilerplate.py
@@ -386,7 +386,8 @@ def build_pyplot(pyplot_path):
 
     # Run black to autoformat pyplot
     subprocess.run(
-        [sys.executable, "-m", "black", "--line-length=88", pyplot_path]
+        [sys.executable, "-m", "black", "--line-length=88", pyplot_path],
+        check=True
     )
 
 

Original file line number	Diff line number	Diff line change
`@@ -386,7 +386,8 @@ def build_pyplot(pyplot_path):`
`386`	`386`
`387`	`387`	`# Run black to autoformat pyplot`
`388`	`388`	`subprocess.run(`
`389`		`- [sys.executable, "-m", "black", "--line-length=88", pyplot_path]`
	`389`	`+ [sys.executable, "-m", "black", "--line-length=88", pyplot_path],`
	`390`	`+ check=True`
`390`	`391`	`)`
`391`	`392`
`392`	`393`