TST: move test that requires interactive backend to subprocess

tacaswell · tacaswell · commit 1326835cdb52 · 2024-07-03T15:28:38.000-04:00
diff --git a/lib/matplotlib/testing/__init__.py b/lib/matplotlib/testing/__init__.py
@@ -211,3 +211,24 @@ def ipython_in_subprocess(requested_backend_or_gui_framework, all_expected_backe
     )
 
     assert proc.stdout.strip().endswith(f"'{expected_backend}'")
+
+
+def is_ci_environment():
+    # Common CI variables
+    ci_environment_variables = [
+        'CI',        # Generic CI environment variable
+        'CONTINUOUS_INTEGRATION',  # Generic CI environment variable
+        'TRAVIS',    # Travis CI
+        'CIRCLECI',  # CircleCI
+        'JENKINS',   # Jenkins
+        'GITLAB_CI',  # GitLab CI
+        'GITHUB_ACTIONS',  # GitHub Actions
+        'TEAMCITY_VERSION'  # TeamCity
+        # Add other CI environment variables as needed
+    ]
+
+    for env_var in ci_environment_variables:
+        if os.getenv(env_var):
+            return True
+
+    return False
diff --git a/lib/matplotlib/tests/test_backends_interactive.py b/lib/matplotlib/tests/test_backends_interactive.py
@@ -19,7 +19,7 @@
 import matplotlib as mpl
 from matplotlib import _c_internal_utils
 from matplotlib.backend_tools import ToolToggleBase
-from matplotlib.testing import subprocess_run_helper as _run_helper
+from matplotlib.testing import subprocess_run_helper as _run_helper, is_ci_environment
 
 
 class _WaitForStringPopen(subprocess.Popen):
@@ -110,27 +110,6 @@ def _get_testable_interactive_backends():
             for env, marks in _get_available_interactive_backends()]
 
 
-def is_ci_environment():
-    # Common CI variables
-    ci_environment_variables = [
-        'CI',        # Generic CI environment variable
-        'CONTINUOUS_INTEGRATION',  # Generic CI environment variable
-        'TRAVIS',    # Travis CI
-        'CIRCLECI',  # CircleCI
-        'JENKINS',   # Jenkins
-        'GITLAB_CI',  # GitLab CI
-        'GITHUB_ACTIONS',  # GitHub Actions
-        'TEAMCITY_VERSION'  # TeamCity
-        # Add other CI environment variables as needed
-    ]
-
-    for env_var in ci_environment_variables:
-        if os.getenv(env_var):
-            return True
-
-    return False
-
-
 # Reasonable safe values for slower CI/Remote and local architectures.
 _test_timeout = 120 if is_ci_environment() else 20
 
diff --git a/lib/matplotlib/tests/test_pickle.py b/lib/matplotlib/tests/test_pickle.py
@@ -8,7 +8,7 @@
 
 import matplotlib as mpl
 from matplotlib import cm
-from matplotlib.testing import subprocess_run_helper
+from matplotlib.testing import subprocess_run_helper, is_ci_environment
 from matplotlib.testing.decorators import check_figures_equal
 from matplotlib.dates import rrulewrapper
 from matplotlib.lines import VertexSelector
@@ -311,7 +311,14 @@ def test_cycler():
 
 # Run under an interactive backend to test that we don't try to pickle the
 # (interactive and non-picklable) canvas.
-@pytest.mark.backend('tkagg')
-def test_axeswidget_interactive():
+def _test_axeswidget_interactive():
     ax = plt.figure().add_subplot()
     pickle.dumps(mpl.widgets.Button(ax, "button"))
+
+
+def test_axeswidget_interactive():
+    subprocess_run_helper(
+        _test_axeswidget_interactive,
+        timeout=120 if is_ci_environment() else 20,
+        extra_env={'MPLBACKEND': 'tkagg'}
+    )
