Add tests for toolmanager.

anntzer · anntzer · commit 229dbfc23c46 · 2020-03-29T20:32:49.000+02:00
... and fix Help tool for tk (it needs a gif icon, generated (like the
other gifs) with `convert -antialias -geometry 24x24 help.svg help.gif`).

... and also edit the matplotlibrc template to mention its availability.
diff --git a/lib/matplotlib/backend_tools.py b/lib/matplotlib/backend_tools.py
@@ -1030,7 +1030,7 @@ def _mouse_move(self, event):
 class ToolHelpBase(ToolBase):
     description = 'Print tool list, shortcuts and description'
     default_keymap = mpl.rcParams['keymap.help']
-    image = 'help.png'
+    image = 'help'
 
     @staticmethod
     def format_shortcut(key_sequence):
diff --git a/lib/matplotlib/mpl-data/images/help.gif b/lib/matplotlib/mpl-data/images/help.gif
diff --git a/lib/matplotlib/tests/test_backends_interactive.py b/lib/matplotlib/tests/test_backends_interactive.py
@@ -1,5 +1,6 @@
 import importlib
 import importlib.util
+import json
 import os
 import signal
 import subprocess
@@ -57,6 +58,7 @@ def _get_testable_interactive_backends():
 _test_script = """\
 import importlib
 import importlib.util
+import json
 import sys
 from unittest import TestCase
 
@@ -67,6 +69,8 @@ def _get_testable_interactive_backends():
     "webagg.open_in_browser": False,
     "webagg.port_retries": 1,
 })
+if len(sys.argv) >= 2:  # Second argument is json-encoded rcParams.
+    rcParams.update(json.loads(sys.argv[1]))
 backend = plt.rcParams["backend"].lower()
 assert_equal = TestCase().assertEqual
 assert_raises = TestCase().assertRaises
@@ -119,10 +123,12 @@ def check_alt_backend(alt_backend):
 
 
 @pytest.mark.parametrize("backend", _get_testable_interactive_backends())
+@pytest.mark.parametrize("toolbar", ["toolbar2", "toolmanager"])
 @pytest.mark.flaky(reruns=3)
-def test_interactive_backend(backend):
+def test_interactive_backend(backend, toolbar):
     proc = subprocess.run(
-        [sys.executable, "-c", _test_script],
+        [sys.executable, "-c", _test_script,
+         json.dumps({"toolbar": toolbar})],
         env={**os.environ, "MPLBACKEND": backend}, timeout=_test_timeout,
         stdout=subprocess.PIPE, universal_newlines=True)
     if proc.returncode:
diff --git a/matplotlibrc.template b/matplotlibrc.template
@@ -92,7 +92,7 @@
 #backend_fallback: True
 
 #interactive: False
-#toolbar:     toolbar2  # {None, toolbar2}
+#toolbar:     toolbar2  # {None, toolbar2, toolmanager}
 #timezone:    UTC       # a pytz timezone string, e.g., US/Central or Europe/Paris
 
 
