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 2017/08/03 01:12:13 UTC

hbase git commit: HBASE-18102 (addendum fixing shell tests) - Purge close_region command that allows by-pass of Master

Repository: hbase
Updated Branches:
  refs/heads/master 71151eb0e -> 504a1f14e


HBASE-18102 (addendum fixing shell tests) - Purge close_region command that allows by-pass of Master

Change-Id: I753f20d50954969ee473f6a016d5c5f9ff4fd631


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

Branch: refs/heads/master
Commit: 504a1f14e39255c4bf398875d6d96578792547d2
Parents: 71151eb
Author: Apekshit Sharma <ap...@apache.org>
Authored: Wed Aug 2 18:05:27 2017 -0700
Committer: Apekshit Sharma <ap...@apache.org>
Committed: Wed Aug 2 18:11:48 2017 -0700

----------------------------------------------------------------------
 hbase-shell/src/test/ruby/hbase/admin_test.rb | 15 +++++++++------
 1 file changed, 9 insertions(+), 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/504a1f14/hbase-shell/src/test/ruby/hbase/admin_test.rb
----------------------------------------------------------------------
diff --git a/hbase-shell/src/test/ruby/hbase/admin_test.rb b/hbase-shell/src/test/ruby/hbase/admin_test.rb
index 6a75886..5184996 100644
--- a/hbase-shell/src/test/ruby/hbase/admin_test.rb
+++ b/hbase-shell/src/test/ruby/hbase/admin_test.rb
@@ -177,7 +177,7 @@ module Hbase
         command(:create, @create_test_name)
       end
     end
-    
+
     define_test "create should fail without columns when called with options" do
       drop_test_table(@create_test_name)
       assert_raise(ArgumentError) do
@@ -241,7 +241,7 @@ module Hbase
       assert_equal(['a:', 'b:'], table(@create_test_name).get_all_columns.sort)
       assert_match(/987654321/, admin.describe(@create_test_name))
     end
-    
+
     define_test "create should work with SPLITALGO" do
       drop_test_table(@create_test_name)
       command(:create, @create_test_name, 'a', 'b',
@@ -334,12 +334,15 @@ module Hbase
       shutdown
     end
 
-    define_test "unassign should allow encoded & non-encoded region names" do
+    define_test "unassign should allow encoded region names" do
       region = command(:locate_region, @test_name, '')
       regionName = region.getRegionInfo().getRegionNameAsString()
-      encodedRegionName = region.getRegionInfo().getEncodedName()
-
       command(:unassign, regionName, true)
+    end
+
+    define_test "unassign should allow non-encoded region names" do
+      region = command(:locate_region, @test_name, '')
+      encodedRegionName = region.getRegionInfo().getEncodedName()
       command(:unassign, encodedRegionName, true)
     end
   end
@@ -428,7 +431,7 @@ module Hbase
       command(:alter, @test_name, 'MAX_FILESIZE' => 12345678)
       assert_match(/12345678/, admin.describe(@test_name))
     end
-    
+
     define_test "alter should be able to change coprocessor attributes" do
       drop_test_table(@test_name)
       create_test_table(@test_name)