fix show bug, tests for type conversions and show

Author: tanmaykm

src/show.jl

@@ -22,7 +22,7 @@ end
 
 function show(io::IO, f::TAMQPFieldValuePair)
     indent = isa(io, IOContext) ? get(io, :indent, "") : ""
-    print(indent)
+    print(io, indent)
     show(io, f.name)
     print(io, " => ")
     show(io, f.val)
@@ -52,7 +52,7 @@ function show(io::IO, fields::Vector{Pair{Symbol,TAMQPField}})
     idx = 1
     for fld in fields
         (idx > 1) && print(ioc, '\n')
-        print(indent)
+        print(ioc, indent)
         show(ioc, fld.first)
         print(ioc, " => ")
         show(ioc, fld.second)

test/test_coverage.jl

@@ -12,6 +12,7 @@ testlog(msg) = println(msg)
 
 function runtests(;virtualhost="/", host="localhost", port=AMQPClient.AMQP_DEFAULT_PORT, auth_params=AMQPClient.DEFAULT_AUTH_PARAMS)
     verify_spec()
+    test_types()
     @test default_exchange_name("direct") == "amq.direct"
     @test default_exchange_name() == ""
     @test AMQPClient.method_name(AMQPClient.TAMQPMethodPayload(:Basic, :Ack, (1, false))) == "Basic.Ack"
@@ -180,4 +181,25 @@ function verify_spec()
     end
 end
 
+function test_types()
+    d = Dict{String,Any}(
+        "bool"      => 0x1,
+        "int"       => 10,
+        "uint"      => 0x1,
+        "float"     => rand(),
+        "shortstr"  => convert(AMQPClient.TAMQPShortStr, randstring(10)),
+        "longstr"   => convert(AMQPClient.TAMQPLongStr, randstring(1024)))
+    ft = convert(AMQPClient.TAMQPFieldTable, d)
+    iob = IOBuffer()
+    show(iob, ft)
+    @test length(take!(iob)) > 0
+
+    fields = [Pair{Symbol,AMQPClient.TAMQPField}(:bit,          AMQPClient.TAMQPBit(0x1)),
+              Pair{Symbol,AMQPClient.TAMQPField}(:shortstr,     convert(AMQPClient.TAMQPShortStr, randstring(10))),
+              Pair{Symbol,AMQPClient.TAMQPField}(:longstr,      convert(AMQPClient.TAMQPLongStr, randstring(1024))),
+              Pair{Symbol,AMQPClient.TAMQPField}(:fieldtable,   ft)]
+    show(iob, fields)
+    @test length(take!(iob)) > 0
+end
+
 end # module AMQPTestCoverage