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

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

commit 69cbf8526b4ef5f5b19e7a19f1993a7a9bcd536a
Author: Alexey Serbin <[email protected]>
AuthorDate: Wed Sep 25 17:32:15 2019 -0700

    [mini-cluster] check for failures in TestKerberosReacquire
    
    Wrapped SmokeTestKerberizedCluster() into NO_FATALS()
    in ExternalMiniClusterTest.TestKerberosReacquire scenario.
    
    Change-Id: I2790bd461cea5caf4ba03b3487600f92362ddc89
    Reviewed-on: http://gerrit.cloudera.org:8080/14305
    Tested-by: Alexey Serbin <[email protected]>
    Reviewed-by: Adar Dembo <[email protected]>
---
 src/kudu/mini-cluster/external_mini_cluster-test.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/kudu/mini-cluster/external_mini_cluster-test.cc 
b/src/kudu/mini-cluster/external_mini_cluster-test.cc
index 08891f3..8e60bfa 100644
--- a/src/kudu/mini-cluster/external_mini_cluster-test.cc
+++ b/src/kudu/mini-cluster/external_mini_cluster-test.cc
@@ -111,7 +111,7 @@ void SmokeTestKerberizedCluster(ExternalMiniClusterOptions 
opts) {
 }
 
 TEST_F(ExternalMiniClusterTest, TestKerberosReacquire) {
-  if (!AllowSlowTests()) return;
+  SKIP_IF_SLOW_NOT_ALLOWED();
 
   ExternalMiniClusterOptions opts;
   opts.enable_kerberos = true;
@@ -120,7 +120,7 @@ TEST_F(ExternalMiniClusterTest, TestKerberosReacquire) {
   opts.mini_kdc_options.ticket_lifetime = "15s";
   opts.num_tablet_servers = 1;
 
-  SmokeTestKerberizedCluster(std::move(opts));
+  NO_FATALS(SmokeTestKerberizedCluster(std::move(opts)));
 }
 
 TEST_P(ExternalMiniClusterTest, TestBasicOperation) {

Reply via email to