matplotlib
diff --git a/‎lib/matplotlib/tests/test_sphinxext.py
Lines changed: 24 additions & 24 deletions b/‎lib/matplotlib/tests/test_sphinxext.py
Lines changed: 24 additions & 24 deletions
@@ -14,6 +14,23 @@
                     minversion=None if sys.version_info < (3, 10) else '4.1.3')
 
 
+def _build_sphinx_html(source_dir, doctree_dir, html_dir, extra_args=None):
+    # Build the pages with warnings turned into errors
+    extra_args = [] if extra_args is None else extra_args
+    cmd = [sys.executable, '-msphinx', '-W', '-b', 'html',
+           '-d', str(doctree_dir), str(source_dir), str(html_dir), *extra_args]
+    proc = Popen(cmd, stdout=PIPE, stderr=PIPE, universal_newlines=True,
+                 env={**os.environ, "MPLBACKEND": ""})
+    out, err = proc.communicate()
+
+    assert proc.returncode == 0, \
+        f"sphinx build failed with stdout:\n{out}\nstderr:\n{err}\n"
+    if err:
+        pytest.fail(f"sphinx build emitted the following warnings:\n{err}")
+
+    assert html_dir.is_dir()
+
+
 def test_tinypages(tmp_path):
     shutil.copytree(Path(__file__).parent / 'tinypages', tmp_path,
                     dirs_exist_ok=True)
@@ -33,7 +50,7 @@ def test_tinypages(tmp_path):
     out, err = proc.communicate()
 
     # Build the pages with warnings turned into errors
-    build_sphinx_html(tmp_path, doctree_dir, html_dir)
+    _build_sphinx_html(tmp_path, doctree_dir, html_dir)
 
     def plot_file(num):
         return img_dir / f'some_plots-{num}.png'
@@ -82,7 +99,7 @@ def plot_directive_file(num):
     # Build the pages again and check that the modified file was updated
     modification_times = [plot_directive_file(i).stat().st_mtime
                           for i in (1, 2, 3, 5)]
-    build_sphinx_html(tmp_path, doctree_dir, html_dir)
+    _build_sphinx_html(tmp_path, doctree_dir, html_dir)
     assert filecmp.cmp(range_4, plot_file(17))
     # Check that the plots in the plot_directive folder weren't changed.
     # (plot_directive_file(1) won't be modified, but it will be copied to html/
@@ -111,13 +128,13 @@ def test_plot_html_show_source_link(tmp_path):
 """)
     # Make sure source scripts are created by default
     html_dir1 = tmp_path / '_build' / 'html1'
-    build_sphinx_html(tmp_path, doctree_dir, html_dir1)
+    _build_sphinx_html(tmp_path, doctree_dir, html_dir1)
     assert len(list(html_dir1.glob("**/index-1.py"))) == 1
     # Make sure source scripts are NOT created when
     # plot_html_show_source_link` is False
     html_dir2 = tmp_path / '_build' / 'html2'
-    build_sphinx_html(tmp_path, doctree_dir, html_dir2,
-                      extra_args=['-D', 'plot_html_show_source_link=0'])
+    _build_sphinx_html(tmp_path, doctree_dir, html_dir2,
+                       extra_args=['-D', 'plot_html_show_source_link=0'])
     assert len(list(html_dir2.glob("**/index-1.py"))) == 0
 
 
@@ -136,7 +153,7 @@ def test_show_source_link_true(tmp_path, plot_html_show_source_link):
     plt.plot(range(2))
 """)
     html_dir = tmp_path / '_build' / 'html'
-    build_sphinx_html(tmp_path, doctree_dir, html_dir, extra_args=[
+    _build_sphinx_html(tmp_path, doctree_dir, html_dir, extra_args=[
         '-D', f'plot_html_show_source_link={plot_html_show_source_link}'])
     assert len(list(html_dir.glob("**/index-1.py"))) == 1
 
@@ -156,23 +173,6 @@ def test_show_source_link_false(tmp_path, plot_html_show_source_link):
     plt.plot(range(2))
 """)
     html_dir = tmp_path / '_build' / 'html'
-    build_sphinx_html(tmp_path, doctree_dir, html_dir, extra_args=[
+    _build_sphinx_html(tmp_path, doctree_dir, html_dir, extra_args=[
         '-D', f'plot_html_show_source_link={plot_html_show_source_link}'])
     assert len(list(html_dir.glob("**/index-1.py"))) == 0
-
-
-def build_sphinx_html(tmp_path, doctree_dir, html_dir, extra_args=None):
-    # Build the pages with warnings turned into errors
-    extra_args = [] if extra_args is None else extra_args
-    cmd = [sys.executable, '-msphinx', '-W', '-b', 'html',
-           '-d', str(doctree_dir), str(tmp_path), str(html_dir), *extra_args]
-    proc = Popen(cmd, stdout=PIPE, stderr=PIPE, universal_newlines=True,
-                 env={**os.environ, "MPLBACKEND": ""})
-    out, err = proc.communicate()
-
-    assert proc.returncode == 0, \
-        f"sphinx build failed with stdout:\n{out}\nstderr:\n{err}\n"
-    if err:
-        pytest.fail(f"sphinx build emitted the following warnings:\n{err}")
-
-    assert html_dir.is_dir()