You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by ap...@apache.org on 2018/03/14 22:06:19 UTC

[4/4] hbase git commit: HBASE-18864 (addendum) Fixed unit test failure TestFromClientSide#testIllegalTableDescriptor

HBASE-18864 (addendum) Fixed unit test failure TestFromClientSide#testIllegalTableDescriptor

Signed-off-by: Andrew Purtell <ap...@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/3d8f8aea
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/3d8f8aea
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/3d8f8aea

Branch: refs/heads/branch-1.2
Commit: 3d8f8aea7068b35ea59fa8975db3d3d77e7a106f
Parents: e4c0c97
Author: Sakthi <ja...@cloudera.com>
Authored: Wed Mar 14 11:43:01 2018 -0700
Committer: Andrew Purtell <ap...@apache.org>
Committed: Wed Mar 14 13:57:28 2018 -0700

----------------------------------------------------------------------
 .../apache/hadoop/hbase/client/TestFromClientSide.java    | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/3d8f8aea/hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestFromClientSide.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestFromClientSide.java b/hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestFromClientSide.java
index 67f6287..22ad5b2 100644
--- a/hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestFromClientSide.java
+++ b/hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestFromClientSide.java
@@ -5462,9 +5462,13 @@ public class TestFromClientSide {
     checkTableIsIllegal(htd);
     hcd.setMinVersions(3);
 
-    hcd.setScope(-1);
-    checkTableIsIllegal(htd);
-    hcd.setScope(0);
+    try {
+      hcd.setScope(-1);
+      fail("Illegal value for setScope did not throw");
+    } catch (IllegalArgumentException e) {
+      // expected
+      hcd.setScope(0);
+    }
     checkTableIsLegal(htd);
 
     try {