Merge pull request #896 from supercaracal/fix-a-unstable-test-case

Fix unstable test cases
redis · Mar 30, 2020 · 45666da · 45666da
2 parents 7127f3b + a165deb
commit 45666da
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/test/distributed_remote_server_control_commands_test.rb b/test/distributed_remote_server_control_commands_test.rb
@@ -28,11 +28,13 @@ def test_info
 
   def test_info_commandstats
     target_version "2.5.7" do
-      r.nodes.each { |n| n.config(:resetstat) }
-      r.ping # Executed on every node
+      r.nodes.each do |n|
+        n.config(:resetstat)
+        n.config(:get, :port)
+      end
 
       r.info(:commandstats).each do |info|
-        assert_equal "1", info["ping"]["calls"]
+        assert_equal '2', info['config']['calls'] # CONFIG RESETSTAT + CONFIG GET = twice
       end
     end
   end

diff --git a/test/remote_server_control_commands_test.rb b/test/remote_server_control_commands_test.rb
@@ -27,10 +27,10 @@ def test_info
   def test_info_commandstats
     target_version "2.5.7" do
       r.config(:resetstat)
-      r.ping
+      r.config(:get, :port)
 
       result = r.info(:commandstats)
-      assert_equal "1", result["ping"]["calls"]
+      assert_equal '2', result['config']['calls']
     end
   end