Skip to content
This repository was archived by the owner on Jun 12, 2021. It is now read-only.

Commit 5ef1ce6

Browse files
committed
Fixed tests
1 parent 021156c commit 5ef1ce6

File tree

2 files changed

+6
-7
lines changed

2 files changed

+6
-7
lines changed

src/oidcendpoint/client_authn.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -142,7 +142,7 @@ def verify(self, request, **kwargs):
142142
logger.debug("authntoken: {}".format(authtoken))
143143

144144
_endpoint = kwargs.get("endpoint")
145-
if _endpoint is None:
145+
if _endpoint is None or not _endpoint:
146146
if self.endpoint_context.issuer in ca_jwt["aud"]:
147147
pass
148148
else:
@@ -205,7 +205,7 @@ def valid_client_info(cinfo):
205205

206206
def verify_client(
207207
endpoint_context, request, authorization_info=None, get_client_id_from_token=None,
208-
endpoint=""
208+
endpoint=None
209209
):
210210
"""
211211
Initiated Guessing !

tests/test_25_oidc_token_endpoint.py

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88
from oidcendpoint import JWT_BEARER
99
from oidcendpoint.client_authn import verify_client
1010
from oidcendpoint.endpoint_context import EndpointContext
11+
from oidcendpoint.exception import MultipleUsage
1112
from oidcendpoint.oidc import userinfo
1213
from oidcendpoint.oidc.authorization import Authorization
1314
from oidcendpoint.oidc.provider_config import ProviderConfiguration
@@ -239,15 +240,13 @@ def test_process_request_using_private_key_jwt(self):
239240
_assertion = _jwt.pack({"aud": [_context.endpoint["token"].full_path]})
240241
_token_request.update({"client_assertion": _assertion,
241242
"client_assertion_type": JWT_BEARER})
243+
_token_request["code"] = self.endpoint.endpoint_context.sdb[session_id]["code"]
242244

243245
_context.sdb.update(session_id, user="diana")
244246
_req = self.endpoint.parse_request(_token_request)
245247
_resp = self.endpoint.process_request(request=_req)
246248

247249
# 2nd time used
248-
_req = self.endpoint.parse_request(_token_request)
249-
_resp = self.endpoint.process_request(request=_req)
250-
251-
assert _resp
252-
assert set(_resp.keys()) == {"error", "error_description"}
250+
with pytest.raises(MultipleUsage):
251+
self.endpoint.parse_request(_token_request)
253252

0 commit comments

Comments
 (0)