MNT: skip sphinx tests per-test not per-module

tacaswell · tacaswell · commit 66b1647ac5f2 · 2022-10-20T18:55:06.000-04:00
diff --git a/lib/matplotlib/tests/test_sphinxext.py b/lib/matplotlib/tests/test_sphinxext.py
@@ -10,8 +10,10 @@
 import pytest
 
 
-pytest.importorskip('sphinx',
-                    minversion=None if sys.version_info < (3, 10) else '4.1.3')
+def sphinx_skipper(func):
+    return pytest.importorskip(
+        'sphinx',
+        minversion=None if sys.version_info < (3, 10) else '4.1.3')(func)
 
 
 def _build_sphinx_html(source_dir, doctree_dir, html_dir, extra_args=None):
@@ -31,6 +33,7 @@ def _build_sphinx_html(source_dir, doctree_dir, html_dir, extra_args=None):
     assert html_dir.is_dir()
 
 
+@sphinx_skipper
 def test_tinypages(tmpdir):
     source_dir = Path(tmpdir) / 'src'
     shutil.copytree(Path(__file__).parent / 'tinypages', source_dir)
@@ -115,6 +118,7 @@ def plot_directive_file(num):
     assert filecmp.cmp(range_6, plot_file(5))
 
 
+@sphinx_skipper
 def test_plot_html_show_source_link(tmpdir):
     source_dir = Path(tmpdir) / 'src'
     source_dir.mkdir()
@@ -135,6 +139,5 @@ def test_plot_html_show_source_link(tmpdir):
     # plot_html_show_source_link` is False
     html_dir2 = source_dir / '_build' / 'html2'
     _build_sphinx_html(source_dir, doctree_dir, html_dir2,
-                      extra_args=['-D', 'plot_html_show_source_link=0'])
+                       extra_args=['-D', 'plot_html_show_source_link=0'])
     assert "index-1.py" not in [p.name for p in html_dir2.iterdir()]
-
