Test timers and (a bit) key_press_event for interactive backends.

anntzer · anntzer · commit 29268813798c · 2018-03-17T22:12:13.000-07:00
diff --git a/.travis.yml b/.travis.yml
@@ -46,6 +46,7 @@ env:
     - secure: RgJI7BBL8aX5FTOQe7xiXqWHMxWokd6GNUWp1NUV2mRLXPb9dI0RXqZt3UJwKTAzf1z/OtlHDmEkBoTVK81E9iUxK5npwyyjhJ8yTJmwfQtQF2n51Q1Ww9p+XSLORrOzZc7kAo6Kw6FIXN1pfctgYq2bQkrwJPRx/oPR8f6hcbY=
     - secure: E7OCdqhZ+PlwJcn+Hd6ns9TDJgEUXiUNEI0wu7xjxB2vBRRIKtZMbuaZjd+iKDqCKuVOJKu0ClBUYxmgmpLicTwi34CfTUYt6D4uhrU+8hBBOn1iiK51cl/aBvlUUrqaRLVhukNEBGZcyqAjXSA/Qsnp2iELEmAfOUa92ZYo1sk=
     - secure: "dfjNqGKzQG5bu3FnDNwLG8H/C4QoieFo4PfFmZPdM2RY7WIzukwKFNT6kiDfOrpwt+2bR7FhzjOGlDECGtlGOtYPN8XuXGjhcP4a4IfakdbDfF+D3NPIpf5VlE6776k0VpvcZBTMYJKNFIMc7QPkOwjvNJ2aXyfe3hBuGlKJzQU="
+    # Variables controlling Python dependencies.
     - CYCLER=cycler
     - DATEUTIL=python-dateutil
     - NOSE=
@@ -56,12 +57,15 @@ env:
     - PYTEST_COV=pytest-cov
     - PYTEST_PEP8=
     - SPHINX=sphinx
-    - OPENBLAS_NUM_THREADS=1
+    # Variables controlling the test run.
+    - DELETE_FONT_CACHE=
+    - NO_AT_BRIDGE=1  # Necessary for GTK3 interactive test.
     - NPROC=2
-    - RUN_PEP8=
+    - OPENBLAS_NUM_THREADS=1
+    - PYTHONFAULTHANDLER=1
     - PYTEST_ARGS="-rawR --maxfail=50 --timeout=300 --durations=25 --cov-report= --cov=lib -n $NPROC"
     - PYTHON_ARGS=
-    - DELETE_FONT_CACHE=
+    - RUN_PEP8=
 
 matrix:
   include:
diff --git a/lib/matplotlib/backends/backend_wx.py b/lib/matplotlib/backends/backend_wx.py
@@ -663,9 +663,6 @@ def do_nothing(*args, **kwargs):
 
         self.macros = {}  # dict from wx id to seq of macros
 
-    def Destroy(self, *args, **kwargs):
-        wx.Panel.Destroy(self, *args, **kwargs)
-
     def Copy_to_Clipboard(self, event=None):
         "copy bitmap of canvas to system clipboard"
         bmp_obj = wx.BitmapDataObject()
diff --git a/lib/matplotlib/tests/test_backends_interactive.py b/lib/matplotlib/tests/test_backends_interactive.py
@@ -30,14 +30,24 @@ def _get_testable_interactive_backends():
     return backends
 
 
+# 1. Going through a timer rather than directly calling key_press_event("q") in
+#    the draw_event handler is necessary on wx (where the widget is otherwise
+#    deleted too early.
+# 2. Calling key_press_event via the base class rather than as
+#    fig.canvas.key_press_event is necessary on gtk3 (which redefines
+#    key_press_event with a different signature).
 _test_script = """\
 import sys
 from matplotlib import pyplot as plt
+from matplotlib.backend_bases import FigureCanvasBase
 
 fig = plt.figure()
 ax = fig.add_subplot(111)
-ax.plot([1,2,3], [1,3,1])
-fig.canvas.mpl_connect("draw_event", lambda event: sys.exit())
+ax.plot([0, 1])
+timer = fig.canvas.new_timer(1)
+timer.add_callback(FigureCanvasBase.key_press_event, fig.canvas, "q")
+# Trigger quitting upon draw.
+fig.canvas.mpl_connect("draw_event", lambda event: timer.start())
 plt.show()
 """
 
