Fix error messages (#54)

Author: supercaracal

lib/redis_client/cluster/errors.rb

@@ -6,17 +6,13 @@ class RedisClient
   class Cluster
     ERR_ARG_NORMALIZATION = ->(arg) { Array[arg].flatten.reject { |e| e.nil? || (e.respond_to?(:empty?) && e.empty?) } }
 
-    # Raised when client connected to redis as cluster mode
-    # and failed to fetch cluster state information by commands.
     class InitialSetupError < ::RedisClient::Error
       def initialize(errors)
         msg = ERR_ARG_NORMALIZATION.call(errors).map(&:message).uniq.join(',')
         super("Redis client could not fetch cluster information: #{msg}")
       end
     end
 
-    # Raised when client connected to redis as cluster mode
-    # and some cluster subcommands were called.
     class OrchestrationCommandNotSupported < ::RedisClient::Error
       def initialize(command)
         str = ERR_ARG_NORMALIZATION.call(command).map(&:to_s).join(' ').upcase
@@ -28,7 +24,6 @@ def initialize(command)
       end
     end
 
-    # Raised when error occurs on any node of cluster.
     class ErrorCollection < ::RedisClient::Error
       attr_reader :errors
 
@@ -41,11 +36,10 @@ def initialize(errors)
 
         @errors = errors
         messages = @errors.map { |node_key, error| "#{node_key}: #{error.message}" }
-        super("Command errors were replied on any node: #{messages.join(', ')}")
+        super("Errors occurred on any node: #{messages.join(', ')}")
       end
     end
 
-    # Raised when cluster client can't select node.
     class AmbiguousNodeError < ::RedisClient::Error
       def initialize(command)
         super("Cluster client doesn't know which node the #{command} command should be sent to.")

test/redis_client/cluster/test_errors.rb

@@ -44,11 +44,11 @@ def test_error_collection_error
         [
           {
             errors: { '127.0.0.1:6379' => DummyError.new('foo') },
-            want: { msg: 'Command errors were replied on any node: 127.0.0.1:6379: foo', size: 1 }
+            want: { msg: 'Errors occurred on any node: 127.0.0.1:6379: foo', size: 1 }
           },
           {
             errors: { '127.0.0.1:6379' => DummyError.new('foo'), '127.0.0.1:6380' => DummyError.new('bar') },
-            want: { msg: 'Command errors were replied on any node: 127.0.0.1:6379: foo, 127.0.0.1:6380: bar', size: 2 }
+            want: { msg: 'Errors occurred on any node: 127.0.0.1:6379: foo, 127.0.0.1:6380: bar', size: 2 }
           },
           { errors: {}, want: { msg: '', size: 0 } },
           { errors: '', want: { msg: '', size: 0 } },