[FIX] The Guardian, Twitter, NYT: Error credentials

orangecontrib/text/widgets/owguardian.py

@@ -44,8 +44,9 @@ def __init__(self, parent):
             self.load_credentials()
 
         def load_credentials(self):
-            if self.cm_key.key:
-                self.key_edit.setText(self.cm_key.key)
+            key = self.cm_key.key
+            if key:
+                self.key_edit.setText(key)
 
         def save_credentials(self):
             self.cm_key.key = self.key_input
@@ -62,6 +63,7 @@ def accept(self, silent=False):
             if not silent: self.Error.invalid_credentials.clear()
             self.check_credentials()
             if self.api:
+
                 self.parent.update_api(self.api)
                 super().accept()
             elif not silent:

orangecontrib/text/widgets/ownyt.py

@@ -44,7 +44,9 @@ def __init__(self, parent):
             self.load_credentials()
 
         def load_credentials(self):
-            self.key_edit.setText(self.cm_key.key)
+            key = self.cm_key.key
+            if key:
+                self.key_edit.setText(key)
 
         def save_credentials(self):
             self.cm_key.key = self.key_input

orangecontrib/text/widgets/owtwitter.py

@@ -51,8 +51,12 @@ def __init__(self, parent):
             self.load_credentials()
 
         def load_credentials(self):
-            self.key_edit.setText(self.cm_key.key)
-            self.secret_edit.setText(self.cm_secret.key)
+            key = self.cm_key.key
+            secret_key = self.cm_secret.key
+            if key:
+                self.key_edit.setText(key)
+            if secret_key:
+                self.secret_edit.setText(secret_key)
 
         def save_credentials(self):
             self.cm_key.key = self.key_input

orangecontrib/text/widgets/tests/test_owguardian.py

@@ -0,0 +1,21 @@
+import unittest
+
+from Orange.widgets.tests.base import WidgetTest
+
+from orangecontrib.text.widgets.owguardian import OWGuardian
+
+
+class TestGuardian(WidgetTest):
+    def test_error_credentials(self):
+        """
+        Handling error due to password credentials.
+        GH-253
+        """
+        with unittest.mock.patch(
+            "keyring.get_password",
+            side_effect=ValueError):
+            self.create_widget(OWGuardian)
+
+
+if __name__ == "__main__":
+    unittest.main()

orangecontrib/text/widgets/tests/test_ownyt.py

@@ -0,0 +1,21 @@
+import unittest
+
+from Orange.widgets.tests.base import WidgetTest
+
+from orangecontrib.text.widgets.ownyt import OWNYT
+
+
+class TestGuardian(WidgetTest):
+    def test_error_credentials(self):
+        """
+        Handling error due to password credentials.
+        GH-253
+        """
+        with unittest.mock.patch(
+            "Orange.widgets.credentials.CredentialManager.__getattr__",
+            side_effect=Exception):
+            self.create_widget(OWNYT)
+
+
+if __name__ == "__main__":
+    unittest.main()

orangecontrib/text/widgets/tests/test_owtwitter.py

@@ -0,0 +1,21 @@
+import unittest
+
+from Orange.widgets.tests.base import WidgetTest
+
+from orangecontrib.text.widgets.owtwitter import OWTwitter
+
+
+class TestGuardian(WidgetTest):
+    def test_error_credentials(self):
+        """
+        Handling error due to password credentials.
+        GH-253
+        """
+        with unittest.mock.patch(
+            "Orange.widgets.credentials.CredentialManager.__getattr__",
+            side_effect=Exception):
+            self.create_widget(OWTwitter)
+
+
+if __name__ == "__main__":
+    unittest.main()