PerfectDomaincontextHandler: Fix clone_context

As PerfectDomainContextHandler uses different *args from its BaseClass,
it needs to override the clone_context method.

Author: astaric

Orange/widgets/data/tests/test_owcolor.py

@@ -0,0 +1,19 @@
+from Orange.data import Table
+from Orange.widgets.data.owcolor import OWColor
+from Orange.widgets.tests.base import WidgetTest
+
+
+class TestOWColor(WidgetTest):
+    def setUp(self):
+        self.widget = self.create_widget(OWColor)
+
+        self.iris = Table("iris")
+
+    def test_reuse_old_settings(self):
+        self.send_signal("Data", self.iris)
+
+        assert isinstance(self.widget, OWColor)
+        self.widget.saveSettings()
+
+        w = self.create_widget(OWColor, reset_default_settings=False)
+        self.send_signal("Data", self.iris, widget=w)

Orange/widgets/settings.py

@@ -1083,6 +1083,13 @@ def new_context(self, domain, attributes, class_vars, metas):
         context.class_vars = class_vars
         return context
 
+    def clone_context(self, old_context, *args):
+        """Copy of context is always valid, since widgets are using
+        the same domain."""
+        context = self.new_context(*args)
+        context.values = copy.deepcopy(old_context.values)
+        return context
+
     def encode_domain(self, domain):
         """Encode domain into tuples (name, type)
         A tuple is returned for each of attributes, class_vars and metas.

Orange/widgets/tests/base.py

@@ -89,7 +89,7 @@ def tearDownClass(cls):
             w.onDeleteWidget()
             sip.delete(w)
 
-    def create_widget(self, cls, stored_settings=None):
+    def create_widget(self, cls, stored_settings=None, reset_default_settings=True):
         """Create a widget instance.
 
         Parameters
@@ -98,12 +98,16 @@ def create_widget(self, cls, stored_settings=None):
             Widget class to instantiate
         stored_settings : dict
             Default values for settings
+        reset_default_settings : bool
+            If set, widget will start with default values for settings,
+            if not, values accumulated through the session will be used
 
         Returns
         -------
         Widget instance : cls
         """
-        self.reset_default_settings(cls)
+        if reset_default_settings:
+            self.reset_default_settings(cls)
         widget = cls.__new__(cls, signal_manager=self.signal_manager,
                              stored_settings=stored_settings)
         widget.__init__()
@@ -138,6 +142,15 @@ def process_events():
         """
         app.processEvents()
 
+    def show(self, widget=None):
+        """Show widget in interactive mode.
+
+        Useful for debugging tests, as widget can be inspected manually.
+        """
+        widget = widget or self.widget
+        widget.show()
+        app.exec()
+
     def send_signal(self, input_name, value, id=None, widget=None):
         """ Send signal to widget by calling appropriate triggers.