This is an automated email from the ASF dual-hosted git repository.

alexey pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git


The following commit(s) were added to refs/heads/master by this push:
     new 08cb9b991 Update security-itest jwt tests
08cb9b991 is described below

commit 08cb9b99116a04ae7c4fa15d05286032983cee34
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]>
---
 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");
   }
 }
 

Reply via email to