chore: add test for credentials parsing

Author: kangasta

requirements-dev.in

@@ -1,5 +1,6 @@
+keyring
 mock
 pip-tools
 pytest
 pytest-cov
-responses
+responses

requirements-dev.txt

@@ -18,8 +18,20 @@ idna==3.10
     # via requests
 iniconfig==2.1.0
     # via pytest
+jaraco-classes==3.4.0
+    # via keyring
+jaraco-context==6.0.1
+    # via keyring
+jaraco-functools==4.2.1
+    # via keyring
+keyring==25.6.0
+    # via -r requirements-dev.in
 mock==5.2.0
     # via -r requirements-dev.in
+more-itertools==10.7.0
+    # via
+    #   jaraco-classes
+    #   jaraco-functools
 packaging==25.0
     # via
     #   build

test/test_cloud_manager.py

@@ -1,10 +1,18 @@
 import json
 
+import pytest
 import responses
+
 from conftest import Mock
+from upcloud_api import CloudManager
+from upcloud_api.errors import UpCloudClientError
 
 
 class TestCloudManagerBasic:
+    def test_no_credentials(self):
+        with pytest.raises(UpCloudClientError):
+            CloudManager()
+
     @responses.activate
     def test_get_account(self, manager):
         data = Mock.mock_get("account")

test/test_credentials.py

@@ -0,0 +1,64 @@
+import keyring
+
+from upcloud_api import Credentials
+
+class DictBackend(keyring.backend.KeyringBackend):
+    priority = 1
+    def __init__(self, secrets=None):
+        super().__init__()
+        self._secrets = secrets or {}
+
+    def set_password(self, servicename, username, password):
+        pass
+
+    def get_password(self, servicename, username):
+        return self._secrets.get(servicename, {}).get(username)
+
+    def delete_password(self, servicename, username):
+        pass
+
+
+class TestCredentials:
+    def test_precedence(self, monkeypatch):
+        param_basic = 'Basic cGFyYW1fdXNlcjpwYXJhbV9wYXNz'
+        param_bearer = 'Bearer param_token'
+        env_basic = 'Basic ZW52X3VzZXI6ZW52X3Bhc3M='
+        env_bearer = 'Bearer env_token'
+        keyring_basic = 'Basic ZW52X3VzZXI6a2V5cmluZ19wYXNz'
+        keyring_bearer = 'Bearer keyring_token'
+
+        backend = DictBackend({
+            "UpCloud": {
+                "env_user": "keyring_pass",
+                "": "keyring_token",
+            }
+        })
+        keyring.set_keyring(backend)
+
+        credentials = Credentials.parse()
+        assert credentials.authorization == keyring_bearer
+
+
+        monkeypatch.setenv("UPCLOUD_USERNAME", 'env_user')
+
+        credentials = Credentials.parse()
+        assert credentials.authorization == keyring_basic
+
+
+        monkeypatch.setenv("UPCLOUD_PASSWORD", 'env_pass')
+
+        credentials = Credentials.parse(username='param_user', password='param_pass')
+        assert credentials.authorization == param_basic
+
+        credentials = Credentials.parse()
+        assert credentials.authorization == env_basic
+
+        monkeypatch.setenv("UPCLOUD_TOKEN", 'env_token')
+        credentials = Credentials.parse(username='param_user', password='param_pass')
+        assert credentials.authorization == param_basic
+
+        credentials = Credentials.parse()
+        assert credentials.authorization == env_bearer
+
+        credentials = Credentials.parse(token='param_token')
+        assert credentials.authorization == param_bearer