fixed and added test for Redis::Distributed#inspect

Author: marcoow

lib/redis/distributed.rb

@@ -659,7 +659,7 @@ def inspect
       node_info = nodes.map do |node|
         "#{node.id} (Redis v#{node.info['redis_version']})"
       end
-      "<Redis client v#{Redis::VERSION} connected to #{node_info.join(', ')}>"
+      "#<Redis client v#{Redis::VERSION} connected to #{node_info.join(', ')}>"
     end
 
   protected

test/distributed_internals_test.rb

@@ -13,6 +13,15 @@
 end
 
 $TEST_PIPELINING = false
-$TEST_INSPECT    = false
 
 load File.expand_path("./lint/internals.rb", File.dirname(__FILE__))
+
+test "provides a meaningful inspect" do |r, _|
+  nodes = ["redis://localhost:6379/15", *NODES]
+  @r = Redis::Distributed.new nodes
+
+  node_info = nodes.map do |node|
+    "#{node} (Redis v#{@r.info.first["redis_version"]})"
+  end
+  assert "#<Redis client v#{Redis::VERSION} connected to #{node_info.join(', ')}>" == @r.inspect
+end

test/distributed_test.rb

@@ -57,4 +57,3 @@
     r.not_yet_implemented_command
   end
 end
-

test/internals_test.rb

@@ -12,10 +12,13 @@
 end
 
 $TEST_PIPELINING = true
-$TEST_INSPECT    = true
 
 load File.expand_path("./lint/internals.rb", File.dirname(__FILE__))
 
+test "provides a meaningful inspect" do |r, _|
+  assert "#<Redis client v#{Redis::VERSION} connected to redis://127.0.0.1:6379/15 (Redis v#{r.info["redis_version"]})>" == r.inspect
+end
+
 test "Redis.current" do
   Redis.current.set("foo", "bar")
 

test/lint/internals.rb

@@ -27,10 +27,6 @@
   end
 end
 
-test "provides a meaningful inspect" do |r, _|
-  assert "#<Redis client v#{Redis::VERSION} connected to redis://127.0.0.1:6379/15 (Redis v#{r.info["redis_version"]})>" == r.inspect
-end if $TEST_INSPECT
-
 test "raises on protocol errors" do
   redis_mock(:ping => lambda { |*_| "foo" }) do
     assert_raise(Redis::ProtocolError) do