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

tacaswell · tacaswell · commit e097a7eae647 · 2022-11-23T17:10:00.000-05:00
diff --git a/lib/matplotlib/tests/test_sphinxext.py b/lib/matplotlib/tests/test_sphinxext.py
@@ -10,8 +10,11 @@
 import pytest
 
 
-pytest.importorskip('sphinx',
-                    minversion=None if sys.version_info < (3, 10) else '4.1.3')
+def sphinx_skipper(func):
+    pytest.importorskip(
+        'sphinx',
+        minversion=None if sys.version_info < (3, 10) else '4.1.3')
+    return func
 
 
 def _build_sphinx_html(source_dir, doctree_dir, html_dir, extra_args=None):
@@ -31,6 +34,7 @@ def _build_sphinx_html(source_dir, doctree_dir, html_dir, extra_args=None):
     assert html_dir.is_dir()
 
 
+@sphinx_skipper
 def test_tinypages(tmp_path):
     shutil.copytree(Path(__file__).parent / 'tinypages', tmp_path,
                     dirs_exist_ok=True)
@@ -116,6 +120,7 @@ def plot_directive_file(num):
     assert filecmp.cmp(range_6, plot_file(5))
 
 
+@sphinx_skipper
 def test_plot_html_show_source_link(tmp_path):
     parent = Path(__file__).parent
     shutil.copyfile(parent / 'tinypages/conf.py', tmp_path / 'conf.py')
@@ -138,6 +143,7 @@ def test_plot_html_show_source_link(tmp_path):
     assert len(list(html_dir2.glob("**/index-1.py"))) == 0
 
 
+@sphinx_skipper
 @pytest.mark.parametrize('plot_html_show_source_link', [0, 1])
 def test_show_source_link_true(tmp_path, plot_html_show_source_link):
     # Test that a source link is generated if :show-source-link: is true,
@@ -158,6 +164,7 @@ def test_show_source_link_true(tmp_path, plot_html_show_source_link):
     assert len(list(html_dir.glob("**/index-1.py"))) == 1
 
 
+@sphinx_skipper
 @pytest.mark.parametrize('plot_html_show_source_link', [0, 1])
 def test_show_source_link_false(tmp_path, plot_html_show_source_link):
     # Test that a source link is NOT generated if :show-source-link: is false,
