From 5527566a3614189d0a7e9e23279cf28a62719e22 Mon Sep 17 00:00:00 2001 From: Matt Moyer Date: Fri, 9 Jul 2021 16:26:05 -0500 Subject: [PATCH] Fix TestCLILoginOIDC when running directly against Okta. Our actual CLI code behaved correctly, but this test made some invalid assumptions about the "upstream" IDP we're testing. It assumed that the upstream didn't support `response_mode=form_post`, but Okta does. This means that when we end up on the localhost callback page, there are no URL query parameters. Adjusting this regex makes the test pass as expected. Signed-off-by: Matt Moyer --- test/integration/cli_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/integration/cli_test.go b/test/integration/cli_test.go index 2e69bc32..ea710e92 100644 --- a/test/integration/cli_test.go +++ b/test/integration/cli_test.go @@ -355,7 +355,7 @@ func runPinnipedLoginOIDC( // Expect to be redirected to the localhost callback. t.Logf("waiting for redirect to callback") - callbackURLPattern := regexp.MustCompile(`\A` + regexp.QuoteMeta(env.CLIUpstreamOIDC.CallbackURL) + `\?.+\z`) + callbackURLPattern := regexp.MustCompile(`\A` + regexp.QuoteMeta(env.CLIUpstreamOIDC.CallbackURL) + `(\?.+)?\z`) browsertest.WaitForURL(t, page, callbackURLPattern) // Wait for the "pre" element that gets rendered for a `text/plain` page, and