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");
}
}