From d74befd9cda447a2e05245f2aa97ad157529f46a Mon Sep 17 00:00:00 2001 From: Daniela Plascencia Date: Wed, 31 Jul 2024 12:31:45 +0200 Subject: [PATCH] tests: remove public-url config and add dex-oidc-config relation (#489) The public-url configuration option has been replaced by a relation between dex-auth and oidc-gatekeeper. Because of that, configuring this option is not required anymore, and in fact if kept, can cause errors as it won't be recognised by Juju anymore. This commit removes the extra configuration from the integration tests. Part of #487 --- tests/test_bundle.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/tests/test_bundle.py b/tests/test_bundle.py index c6bf05bb..b5a013c9 100644 --- a/tests/test_bundle.py +++ b/tests/test_bundle.py @@ -255,7 +255,6 @@ async def test_enable_ingress_auth(ops_test: OpsTest): config={ "static-username": USERNAME, "static-password": PASSWORD, - "public-url": regular_ingress_gateway_ip, }, ) @@ -263,12 +262,12 @@ async def test_enable_ingress_auth(ops_test: OpsTest): OIDC_GATEKEEPER, channel=OIDC_GATEKEEPER_CHANNEL, trust=OIDC_GATEKEEPER_TRUST, - config={"public-url": regular_ingress_gateway_ip}, ) await ops_test.model.add_relation(f"{ISTIO_PILOT}:ingress", f"{DEX_AUTH}:ingress") await ops_test.model.add_relation(f"{ISTIO_PILOT}:ingress", f"{OIDC_GATEKEEPER}:ingress") await ops_test.model.add_relation(f"{OIDC_GATEKEEPER}:oidc-client", f"{DEX_AUTH}:oidc-client") + await ops_test.model.add_relation(f"{OIDC_GATEKEEPER}:dex-oidc-config", f"{DEX_AUTH}:dex-oidc-config") await ops_test.model.add_relation( f"{ISTIO_PILOT}:ingress-auth", f"{OIDC_GATEKEEPER}:ingress-auth" )