Fix bytes vs str for sessionID

Signed-off-by: Justin Cinkelj <justin.cinkelj@xlab.si>

Author: justinc1

plugins/module_utils/client.py

@@ -84,7 +84,7 @@ def __init__(
         self.timeout = timeout
         self.auth_method = auth_method
 
-        self._auth_header: Optional[dict[str, bytes]] = None
+        self._auth_header: Optional[dict[str, str]] = None
         self._client = Request()
 
     @classmethod
@@ -98,15 +98,15 @@ def get_client(cls, cluster_instance: TypedClusterInstance) -> Client:
         )
 
     @property
-    def auth_header(self) -> dict[str, bytes]:
+    def auth_header(self) -> dict[str, str]:
         if not self._auth_header:
             self._auth_header = self._login()
         return self._auth_header
 
-    def _login(self) -> dict[str, bytes]:
+    def _login(self) -> dict[str, str]:
         return self._login_username_password()
 
-    def _login_username_password(self) -> dict[str, bytes]:
+    def _login_username_password(self) -> dict[str, str]:
         headers = {
             "Accept": "application/json",
             "Content-type": "application/json",

plugins/module_utils/typed_classes.py

@@ -21,6 +21,7 @@ class TypedClusterInstance(TypedDict):
     username: str
     password: str
     timeout: float
+    auth_method: str
 
 
 # Registration to ansible return dict.

tests/unit/plugins/module_utils/test_client.py

@@ -95,7 +95,7 @@ def test_basic_auth(self, mocker):
         resp_mock = mocker.MagicMock()
         resp_mock.status = 200  # Used when testing on Python 3
         resp_mock.read.return_value = (
-            '{"sessionID":"7e3a2a70-7130-41c4-9402-fc0953cc1d7b"}'
+            '{"sessionID":"7e3a2a70-7130-41c4-9402-fc0953cc1d7b"}'.encode("utf-8")
         )
 
         request_mock = mocker.patch.object(client, "Request").return_value