From 49a0fc9607116e7144f843f044668ad731839e93 Mon Sep 17 00:00:00 2001 From: Jinna Kiisuo Date: Tue, 10 Sep 2024 00:01:35 +0300 Subject: [PATCH] Fix test logic for correct assert and tangential miss from existing test --- tests/read_only | 3 +++ tests/read_only.lock | 0 tests/test_06_oidc.py | 7 +++++-- 3 files changed, 8 insertions(+), 2 deletions(-) create mode 100644 tests/read_only create mode 100644 tests/read_only.lock diff --git a/tests/read_only b/tests/read_only new file mode 100644 index 00000000..4cb29ea3 --- /dev/null +++ b/tests/read_only @@ -0,0 +1,3 @@ +one +two +three diff --git a/tests/read_only.lock b/tests/read_only.lock new file mode 100644 index 00000000..e69de29b diff --git a/tests/test_06_oidc.py b/tests/test_06_oidc.py index 3b589e4e..b6747af3 100644 --- a/tests/test_06_oidc.py +++ b/tests/test_06_oidc.py @@ -470,6 +470,7 @@ def test_example_response(self): [ "issuer", "authorization_endpoint", + "token_endpoint", "jwks_uri", "response_types_supported", "subject_types_supported", @@ -480,6 +481,7 @@ def test_required_parameters(self, required_param): provider_config = { "issuer": "https://server.example.com", "authorization_endpoint": "https://server.example.com/connect/authorize", + "token_endpoint": "https://server.example.com/connect/token", "jwks_uri": "https://server.example.com/jwks.json", "response_types_supported": ["code", "code id_token", "id_token", "token id_token"], "subject_types_supported": ["public", "pairwise"], @@ -520,19 +522,20 @@ def test_required_parameters_without_rs256(self): provider_config = { "issuer": "https://server.example.com", "authorization_endpoint": "https://server.example.com/connect/authorize", + "token_endpoint": "https://server.example.com/connect/token", "jwks_uri": "https://server.example.com/jwks.json", "response_types_supported": ["code", "code id_token", "id_token", "token id_token"], "subject_types_supported": ["public", "pairwise"], "id_token_signing_alg_values_supported": ["none", "ES256", "HS256"], } - with pytest.raises(MissingRequiredAttribute): - ProviderConfigurationResponse(**provider_config).verify() + assert ProviderConfigurationResponse(**provider_config).verify() def test_required_parameters_only_none_signing_alg(self): provider_config = { "issuer": "https://server.example.com", "authorization_endpoint": "https://server.example.com/connect/authorize", + "token_endpoint": "https://server.example.com/connect/token", "jwks_uri": "https://server.example.com/jwks.json", "response_types_supported": ["code", "code id_token", "id_token", "token id_token"], "subject_types_supported": ["public", "pairwise"],