Fix: Guest Issuer tests

- Check for required environment variables.
- Don't call decode on an str instance.

Author: cmlccie

tests/api/test_guest_issuer.py

@@ -34,6 +34,16 @@
     "WEBEX_TEAMS_GUEST_ISSUER_SECRET"
 )
 
+if not WEBEX_TEAMS_GUEST_ISSUER_ID:
+    raise ValueError(
+        "The WEBEX_TEAMS_GUEST_ISSUER_ID environment variable must be set."
+    )
+
+if not WEBEX_TEAMS_GUEST_ISSUER_SECRET:
+    raise ValueError(
+        "The WEBEX_TEAMS_GUEST_ISSUER_SECRET environment variable must be set."
+    )
+
 
 # Helper Functions
 
@@ -56,4 +66,5 @@ def test_get_guest_issuer_token(api):
         exp=str(int(time.time()) + TOKEN_EXPIRATION_SECONDS),
         secret=WEBEX_TEAMS_GUEST_ISSUER_SECRET,
     )
+
     assert is_valid_guest_issuer_token(guest_issuer_token)

webexteamssdk/api/guest_issuer.py

@@ -115,7 +115,7 @@ def create(self, sub, name, iss, exp, secret):
         key = base64.b64decode(secret)
         jwt_token = jwt.encode(payload, key, algorithm="HS256")
 
-        headers = {"Authorization": "Bearer " + jwt_token.decode("utf-8")}
+        headers = {"Authorization": "Bearer " + jwt_token}
 
         json_data = self._session.post(
             API_ENDPOINT + "/" + "login", headers=headers