This is an automated email from the ASF dual-hosted git repository. alexey pushed a commit to branch branch-1.17.x in repository https://gitbox.apache.org/repos/asf/kudu.git
commit c218ac06b6596bff0d71bfd1193a4863edb3001a Author: Zoltan Chovan <[email protected]> AuthorDate: Mon Apr 17 18:31:38 2023 +0000 Update security-itest jwt tests This change updates the 'TestJwtMiniClusterWithUntrustedCert' and 'TestJwtMiniClusterWithInvalidCert' tests so that they are not relying on third party (curl) generated error messages, but only on messages coming from Kudu. Change-Id: Ie6727ec5c4fee6ad791b9ab5446f31ee2bf97c22 Reviewed-on: http://gerrit.cloudera.org:8080/19758 Reviewed-by: Alexey Serbin <[email protected]> Tested-by: Alexey Serbin <[email protected]> (cherry picked from commit 08cb9b99116a04ae7c4fa15d05286032983cee34) Reviewed-on: http://gerrit.cloudera.org:8080/19973 Tested-by: Kudu Jenkins --- src/kudu/integration-tests/security-itest.cc | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/kudu/integration-tests/security-itest.cc b/src/kudu/integration-tests/security-itest.cc index 0b77adbb9..8987292dd 100644 --- a/src/kudu/integration-tests/security-itest.cc +++ b/src/kudu/integration-tests/security-itest.cc @@ -655,8 +655,8 @@ TEST_F(SecurityITest, TestJwtMiniClusterWithInvalidCert) { Status s = client_builder.Build(&client); ASSERT_FALSE(s.ok()); - ASSERT_STR_CONTAINS(s.ToString(), - "SSL certificate problem: unable to get local issuer certificate"); + ASSERT_TRUE(s.IsRuntimeError()); + ASSERT_STR_CONTAINS(s.ToString(), "Error initializing JWT helper: Failed to load JWKS"); } } @@ -708,7 +708,8 @@ TEST_F(SecurityITest, TestJwtMiniClusterWithUntrustedCert) { Status s = client_builder.Build(&client); ASSERT_FALSE(s.ok()); - ASSERT_STR_CONTAINS(s.ToString(), "SSL peer certificate or SSH remote key was not OK"); + ASSERT_TRUE(s.IsRuntimeError()); + ASSERT_STR_CONTAINS(s.ToString(), "Error initializing JWT helper: Failed to load JWKS"); } }
