You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by ps...@apache.org on 2020/03/31 13:34:31 UTC

[hbase] branch branch-2.2 updated: HBASE-23992 Fix TestAdminShell and TestQuotasShell mistakenly broken by parent commit

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

psomogyi pushed a commit to branch branch-2.2
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/branch-2.2 by this push:
     new 559bc91  HBASE-23992 Fix TestAdminShell and TestQuotasShell mistakenly broken by parent commit
559bc91 is described below

commit 559bc91bc3e3bb5e6612bacbe5b89c34da0edca8
Author: stack <st...@apache.org>
AuthorDate: Sat Mar 14 15:17:20 2020 -0700

    HBASE-23992 Fix TestAdminShell and TestQuotasShell mistakenly broken by parent commit
---
 hbase-shell/src/test/ruby/hbase/admin_test.rb  | 5 +++--
 hbase-shell/src/test/ruby/hbase/quotas_test.rb | 2 +-
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/hbase-shell/src/test/ruby/hbase/admin_test.rb b/hbase-shell/src/test/ruby/hbase/admin_test.rb
index 5206874..8499fe3 100644
--- a/hbase-shell/src/test/ruby/hbase/admin_test.rb
+++ b/hbase-shell/src/test/ruby/hbase/admin_test.rb
@@ -311,7 +311,7 @@ module Hbase
       assert(output.include?('2 row(s)'))
 
       assert(output.include?('QUOTAS'))
-      assert(output.include?('LIMIT => 1G'))
+      assert(output.include?('LIMIT => 1.00G'))
       assert(output.include?('VIOLATION_POLICY => NO_INSERTS'))
       assert(output.include?('TYPE => SPACE'))
       assert(output.include?('1 row(s)'))
@@ -333,12 +333,13 @@ module Hbase
               POLICY => NO_INSERTS,
               NAMESPACE => ns)
       output = capture_stdout { command(:describe_namespace, ns) }
+      puts output
 
       assert(output.include?('DESCRIPTION'))
       assert(output.include?("NAME => '#{ns}'"))
 
       assert(output.include?('QUOTAS'))
-      assert(output.include?('LIMIT => 1G'))
+      assert(output.include?('LIMIT => 1.00G'))
       assert(output.include?('VIOLATION_POLICY => NO_INSERTS'))
       assert(output.include?('TYPE => SPACE'))
       assert(output.include?('1 row(s)'))
diff --git a/hbase-shell/src/test/ruby/hbase/quotas_test.rb b/hbase-shell/src/test/ruby/hbase/quotas_test.rb
index bc51fc41..21c81fa 100644
--- a/hbase-shell/src/test/ruby/hbase/quotas_test.rb
+++ b/hbase-shell/src/test/ruby/hbase/quotas_test.rb
@@ -180,7 +180,7 @@ module Hbase
     define_test 'can set and remove quota' do
       command(:set_quota, TYPE => SPACE, LIMIT => '1G', POLICY => NO_INSERTS, TABLE => @test_name)
       output = capture_stdout{ command(:list_quotas) }
-      assert(output.include?("LIMIT => 1G"))
+      assert(output.include?("LIMIT => 1.00G"))
       assert(output.include?("VIOLATION_POLICY => NO_INSERTS"))
       assert(output.include?("TYPE => SPACE"))
       assert(output.include?("TABLE => #{@test_name}"))