Merge pull request #3359 from ales-erjavec/fixes/credentials-test

[FIX] Use minimal keyring implementation for tests

Author: lanzagar

Orange/widgets/tests/test_credentials.py

@@ -1,13 +1,42 @@
 import unittest
 from unittest.mock import patch
 
+import keyring
+import keyring.errors
+import keyring.backend
+
 from Orange.widgets.credentials import CredentialManager
 
 
+# minimal in-memory keyring implementation so the test is not dependent on
+# the system config/services.
+class Keyring(keyring.backend.KeyringBackend):
+    priority = 0
+
+    def __init__(self):
+        self.__store = {}
+
+    def set_password(self, service, username, password=None):
+        self.__store[(service, username)] = password
+
+    def get_password(self, service, username):
+        return self.__store.get((service, username), None)
+
+    def delete_password(self, service, username):
+        try:
+            del self.__store[service, username]
+        except KeyError:
+            raise keyring.errors.PasswordDeleteError()
+
+
 class TestCredentialManager(unittest.TestCase):
     def setUp(self):
+        self._ring = keyring.get_keyring()
+        keyring.set_keyring(Keyring())
         self.cm = CredentialManager('Orange')
-        self.cm.key = "Foo"
+
+    def tearDown(self):
+        keyring.set_keyring(self._ring)
 
     def test_credential_manager(self):
         cm = CredentialManager('Orange')

requirements-core.txt

@@ -1,5 +1,5 @@
 pip>=9.0
-numpy>=1.9.0
+numpy>=1.13.0
 scipy>=0.16.1
 scikit-learn>=0.18.1
 bottleneck>=1.0.0