client: do not wrap call args into array

All connection methods do not wrap keys and tuples into arrays - it's
responsibility of user to follow IPROTO format. The only exception is
method `call` (by mistake, I believe) - it wraps arguments into array
and it doesn't allow to pass raw MsgPack to this method since
`mpp::as_raw` tag does not work with any other tags. The commit makes
method `call` work like other methods - it won't wrap `args` by itself
anymore.

Closes #108

Author: drewdzzz

src/Client/RequestEncoder.hpp

@@ -318,7 +318,7 @@ RequestEncoder<BUFFER>::encodeCall(const std::string &func, const T &args)
 	encodeHeader(Iproto::CALL);
 	mpp::encode(m_Buf, mpp::as_map(std::forward_as_tuple(
 		MPP_AS_CONST(Iproto::FUNCTION_NAME), func,
-		MPP_AS_CONST(Iproto::TUPLE), mpp::as_arr(args))));
+		MPP_AS_CONST(Iproto::TUPLE), args)));
 	uint32_t request_size = (m_Buf.end() - request_start) - PREHEADER_SIZE;
 	++request_start;
 	request_start.set(__builtin_bswap32(request_size));

test/ClientTest.cpp

@@ -588,6 +588,7 @@ single_conn_call(Connector<BUFFER, NetProvider> &client)
 	const static char *return_multi   = "remote_multi";
 	const static char *return_nil     = "remote_nil";
 	const static char *return_map     = "remote_map";
+	const static char *echo		  = "remote_echo";
 
 	Connection<Buf_t, NetProvider> conn(client);
 	int rc = test_connect(client, conn, localhost, port);
@@ -671,6 +672,15 @@ single_conn_call(Connector<BUFFER, NetProvider> &client)
 	printResponse<BUFFER>(*response,
 		std::make_tuple(std::make_tuple(std::make_pair("key", int()))));
 
+	TEST_CASE("call remote_echo with raw arguments");
+	/* [1, 2, 3] as a raw MsgPack. */
+	const char raw_data[4] = {static_cast<char>(0x93), 1, 2, 3};
+	rid_t f11 = conn.call(echo, mpp::as_raw(raw_data));
+	client.wait(conn, f11, WAIT_TIMEOUT);
+	fail_unless(conn.futureIsReady(f11));
+	response = conn.getResponse(f11);
+	printResponse<BUFFER>(*response, std::make_tuple(std::make_tuple(0, 0, 0)));
+
 	client.close(conn);
 }