Merge pull request #4583 from ales-erjavec/tests-cleanup

Tests cleanup

Author: janezd

Orange/widgets/data/tests/test_owoutliers.py

@@ -38,10 +38,15 @@ def test_no_data(self):
 
 class TestOWOutliers(WidgetTest):
     def setUp(self):
+        super().setUp()
         self.widget = self.create_widget(OWOutliers)
         self.iris = Table("iris")
         self.heart_disease = Table("heart_disease")
 
+    def tearDown(self):
+        self.widget.onDeleteWidget()
+        super().tearDown()
+
     def test_outputs(self):
         """Check widget's data and the output with data on the input"""
         self.send_signal(self.widget.Inputs.data, self.iris)

Orange/widgets/evaluate/owtestlearners.py

@@ -1089,6 +1089,7 @@ def cancel(self):
 
     def onDeleteWidget(self):
         self.cancel()
+        self.__executor.shutdown(True)
         super().onDeleteWidget()
 
 

Orange/widgets/unsupervised/tests/test_owtsne.py

@@ -55,7 +55,7 @@ def tearDown(self):
         # Some tests may not wait for the widget to finish, and the patched
         # methods might be unpatched before the widget finishes, resulting in
         # a very confusing crash.
-        self.widget.cancel()
+        self.widget.onDeleteWidget()
         try:
             self.restore_mocked_functions()
         # If `restore_mocked_functions` was called in the test itself, stopping
@@ -150,6 +150,7 @@ def test_multiscale_changed_updates_ui(self):
         self.send_signal(w.Inputs.data, self.data, widget=w)
         self.assertTrue(w.controls.multiscale.isChecked())
         self.assertFalse(w.perplexity_spin.isEnabled())
+        w.onDeleteWidget()
 
     def test_normalize_data(self):
         # Normalization should be checked by default