Improve debugging of RESPValue

Motivation:

Reading output from `RESPValue` existentials is entirely verbose and does not provide a human-readable description of what is being represented.

Modifications:

- Add conformance to `CustomStringConvertible` for `RESPValue`
- Remove redundant logging of arguments in `RedisConnection.send`

Result:

String representations of `RESPValue` should now be more readable for human to understand.

Author: Mordil

Sources/RedisNIO/RESP/RESPValue.swift

@@ -81,6 +81,20 @@ extension RESPValue: ExpressibleByIntegerLiteral {
     }
 }
 
+// MARK: Custom String Convertible
+
+extension RESPValue: CustomStringConvertible {
+    public var description: String {
+        switch self {
+        case .integer, .simpleString, .bulkString: return self.string!
+        case .null: return "NULL"
+        case let .array(elements): return "[\(elements.map({ $0.description }).joined(separator: ","))]"
+        case let .error(e): return e.message
+        default: return ""
+        }
+    }
+}
+
 // MARK: Computed Values
 
 extension RESPValue {

Sources/RedisNIO/RedisClient.swift

@@ -176,7 +176,7 @@ public final class RedisConnection: RedisClient {
             guard case let .failure(error) = result else { return }
             self.logger.error("\(error.localizedDescription)")
         }
-        logger.debug("Sending command \"\(command)\" with \(arguments) encoded as \(args)")
+        logger.debug("Sending command \"\(command)\" with \(arguments)")
 
         guard sendCommandsImmediately else {
             return channel.write(context).flatMap { promise.futureResult }