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

[hbase] branch branch-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.

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


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

commit bb8e13fc6ec71ab73e14c1916d7a8844f1d3683d
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 3264579..cd3d39a 100644
--- a/hbase-shell/src/test/ruby/hbase/admin_test.rb
+++ b/hbase-shell/src/test/ruby/hbase/admin_test.rb
@@ -339,7 +339,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)'))
@@ -361,12 +361,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}"))