You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by da...@apache.org on 2018/04/03 23:01:07 UTC
kudu git commit: Followup to fe4d962c09012e to fix broken test
Repository: kudu
Updated Branches:
refs/heads/master 9e918bf77 -> de067f792
Followup to fe4d962c09012e to fix broken test
The newly introduced tests weren't cleaning up the test directory
because they didn't inherit from KuduTest.
Change-Id: I0e323f5c9082013510cddc6873bd81675e66d644
Reviewed-on: http://gerrit.cloudera.org:8080/9917
Reviewed-by: Adar Dembo <ad...@cloudera.com>
Tested-by: Kudu Jenkins
Project: http://git-wip-us.apache.org/repos/asf/kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/de067f79
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/de067f79
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/de067f79
Branch: refs/heads/master
Commit: de067f792afc051500a622b77d8c556511ae64f0
Parents: 9e918bf
Author: Dan Burkert <da...@apache.org>
Authored: Tue Apr 3 15:03:05 2018 -0700
Committer: Dan Burkert <da...@apache.org>
Committed: Tue Apr 3 23:00:47 2018 +0000
----------------------------------------------------------------------
src/kudu/hms/hms_catalog-test.cc | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/kudu/blob/de067f79/src/kudu/hms/hms_catalog-test.cc
----------------------------------------------------------------------
diff --git a/src/kudu/hms/hms_catalog-test.cc b/src/kudu/hms/hms_catalog-test.cc
index 1510af0..d060320 100644
--- a/src/kudu/hms/hms_catalog-test.cc
+++ b/src/kudu/hms/hms_catalog-test.cc
@@ -37,6 +37,7 @@
#include "kudu/util/net/net_util.h" // IWYU pragma: keep
#include "kudu/util/status.h"
#include "kudu/util/test_macros.h"
+#include "kudu/util/test_util.h"
DECLARE_string(hive_metastore_uris);
DECLARE_bool(hive_metastore_sasl_enabled);
@@ -113,7 +114,7 @@ TEST(HmsCatalogStaticTest, TestParseUris) {
// Base class for HmsCatalog tests. Parameterized by whether
// SASL/GSSAPI/Kerberos should be enabled.
-class HmsCatalogTest : public ::testing::Test {
+class HmsCatalogTest : public KuduTest {
public:
const char* const kMasterAddrs = "master-addrs";