Skip to content

Commit c06902e

Browse files
committed
Fix test logic for correct assert and tangential miss from existing test
1 parent 01c22cf commit c06902e

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

tests/test_06_oidc.py

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -470,6 +470,7 @@ def test_example_response(self):
470470
[
471471
"issuer",
472472
"authorization_endpoint",
473+
"token_endpoint",
473474
"jwks_uri",
474475
"response_types_supported",
475476
"subject_types_supported",
@@ -480,6 +481,7 @@ def test_required_parameters(self, required_param):
480481
provider_config = {
481482
"issuer": "https://server.example.com",
482483
"authorization_endpoint": "https://server.example.com/connect/authorize",
484+
"token_endpoint": "https://server.example.com/connect/token",
483485
"jwks_uri": "https://server.example.com/jwks.json",
484486
"response_types_supported": ["code", "code id_token", "id_token", "token id_token"],
485487
"subject_types_supported": ["public", "pairwise"],
@@ -520,19 +522,20 @@ def test_required_parameters_without_rs256(self):
520522
provider_config = {
521523
"issuer": "https://server.example.com",
522524
"authorization_endpoint": "https://server.example.com/connect/authorize",
525+
"token_endpoint": "https://server.example.com/connect/token",
523526
"jwks_uri": "https://server.example.com/jwks.json",
524527
"response_types_supported": ["code", "code id_token", "id_token", "token id_token"],
525528
"subject_types_supported": ["public", "pairwise"],
526529
"id_token_signing_alg_values_supported": ["none", "ES256", "HS256"],
527530
}
528531

529-
with pytest.raises(MissingRequiredAttribute):
530-
ProviderConfigurationResponse(**provider_config).verify()
532+
assert ProviderConfigurationResponse(**provider_config).verify()
531533

532534
def test_required_parameters_only_none_signing_alg(self):
533535
provider_config = {
534536
"issuer": "https://server.example.com",
535537
"authorization_endpoint": "https://server.example.com/connect/authorize",
538+
"token_endpoint": "https://server.example.com/connect/token",
536539
"jwks_uri": "https://server.example.com/jwks.json",
537540
"response_types_supported": ["code", "code id_token", "id_token", "token id_token"],
538541
"subject_types_supported": ["public", "pairwise"],

0 commit comments

Comments
 (0)