Ensure observers in GlobalFigureManager are destroyed

mantidproject · Aug 14, 2023 · 4d64687 · 4d64687
1 parent e8369a1
commit 4d64687
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/src/mslice/plotting/globalfiguremanager.py b/src/mslice/plotting/globalfiguremanager.py
@@ -186,6 +186,7 @@ def destroy_all(cls):
         }
         cls._activeQue = []
         cls._figures.clear()
+        cls.observers = []
 
     @classmethod
     def has_fignum(cls, num):

diff --git a/tests/app_test.py b/tests/app_test.py
@@ -3,6 +3,7 @@
 import unittest
 from mock import patch
 from mslice.app.mainwindow import MainWindow
+from mslice.plotting.globalfiguremanager import GlobalFigureManager
 
 qapp = None
 
@@ -15,6 +16,8 @@ def setUp(self):
             qapp = QtWidgets.QApplication([' '])
 
     def tearDown(self):
+        GlobalFigureManager.destroy_all()
+
         # Required to sendPostedEvents twice to ensure the MainWindow is deleted
         qapp.sendPostedEvents()
         qapp.sendPostedEvents()
@@ -28,4 +31,3 @@ def test_mainwindow(self):
             window = MainWindow()
             window.setAttribute(Qt.WA_DeleteOnClose, True)
             QTimer.singleShot(0, window.close)
-