fix subscription callback invoke and pubsub tests

The subscription look callbacks were getting invoked with the `SubscriptionMessage` struct. But the docs do not mention that, and seem to suggest that the callback would receive the actual message, which seems natural. The callbacks in tests seem to be expecting the actual message too.

Updated the callback invocation to pass the actual message instead of `SubscriptionMessage` struct.

Author: tanmaykm

src/client.jl

@@ -165,9 +165,9 @@ function subscription_loop(conn::SubscriptionConnection, err_callback::Function)
             reply = convert_reply(reply)
             message = SubscriptionMessage(reply)
             if message.message_type == SubscriptionMessageType.Message
-                conn.callbacks[message.channel](message)
+                conn.callbacks[message.channel](message.message)
             elseif message.message_type == SubscriptionMessageType.Pmessage
-                conn.pcallbacks[message.channel](message)
+                conn.pcallbacks[message.channel](message.message)
             end
         catch err
             err_callback(err)

test/redis_tests.jl

@@ -396,7 +396,7 @@ end
         return nothing
     end
     x = Any[]
-    function f(y::String)
+    function f(y::AbstractString)
         push!(x, y)
     end
     subs = open_subscription(conn, handleException) #handleException is called when an exception occurs