From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:54156) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Zk0Vx-0004tA-Sa for qemu-devel@nongnu.org; Wed, 07 Oct 2015 22:01:10 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Zk0Vv-00038B-Ic for qemu-devel@nongnu.org; Wed, 07 Oct 2015 22:01:01 -0400 Received: from mx1.redhat.com ([209.132.183.28]:38678) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Zk0Vv-000387-BN for qemu-devel@nongnu.org; Wed, 07 Oct 2015 22:00:59 -0400 Received: from int-mx09.intmail.prod.int.phx2.redhat.com (int-mx09.intmail.prod.int.phx2.redhat.com [10.5.11.22]) by mx1.redhat.com (Postfix) with ESMTPS id DCC59461E7 for ; Thu, 8 Oct 2015 02:00:58 +0000 (UTC) From: Eric Blake Date: Wed, 7 Oct 2015 20:00:49 -0600 Message-Id: <1444269650-4117-15-git-send-email-eblake@redhat.com> In-Reply-To: <1444269650-4117-1-git-send-email-eblake@redhat.com> References: <1444269650-4117-1-git-send-email-eblake@redhat.com> Subject: [Qemu-devel] [PATCH v6 14/15] qapi: More tests of input arrays List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: armbru@redhat.com Our testsuite had no coverage of empty arrays, nor of what happens when the input does not match the expected type. Useful to have, especially if we start changing the visitor contracts. Signed-off-by: Eric Blake --- v6: new patch --- tests/test-qmp-input-visitor.c | 51 ++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 51 insertions(+) diff --git a/tests/test-qmp-input-visitor.c b/tests/test-qmp-input-visitor.c index 8dbc3bf..6972ee8 100644 --- a/tests/test-qmp-input-visitor.c +++ b/tests/test-qmp-input-visitor.c @@ -236,6 +236,12 @@ static void test_visitor_in_list(TestInputVisitorData *data, } qapi_free_UserDefOneList(head); + head = NULL; + + /* An empty list is valid */ + v = visitor_input_test_init(data, "[]"); + visit_type_UserDefOneList(v, &head, NULL, &error_abort); + g_assert(!head); } static void test_visitor_in_any(TestInputVisitorData *data, @@ -716,6 +722,49 @@ static void test_visitor_in_errors(TestInputVisitorData *data, qapi_free_strList(q); } +static void test_visitor_in_wrong_type(TestInputVisitorData *data, + const void *unused) +{ + TestStruct *p = NULL; + Visitor *v; + strList *q = NULL; + int64_t i; + + /* Make sure arrays and structs cannot be confused */ + + v = visitor_input_test_init(data, "[]"); + visit_type_TestStruct(v, &p, NULL, &data->err); + g_assert(data->err); + g_assert(!p); + + v = visitor_input_test_init(data, "{}"); + visit_type_strList(v, &q, NULL, &data->err); + assert(data->err); + assert(!q); + + /* Make sure primitives and struct cannot be confused */ + + v = visitor_input_test_init(data, "1"); + visit_type_TestStruct(v, &p, NULL, &data->err); + g_assert(data->err); + g_assert(!p); + + v = visitor_input_test_init(data, "{}"); + visit_type_int(v, &i, NULL, &data->err); + assert(data->err); + + /* Make sure primitives and arrays cannot be confused */ + + v = visitor_input_test_init(data, "1"); + visit_type_strList(v, &q, NULL, &data->err); + assert(data->err); + assert(!q); + + v = visitor_input_test_init(data, "[]"); + visit_type_int(v, &i, NULL, &data->err); + assert(data->err); +} + int main(int argc, char **argv) { TestInputVisitorData in_visitor_data; @@ -748,6 +797,8 @@ int main(int argc, char **argv) &in_visitor_data, test_visitor_in_alternate); input_visitor_test_add("/visitor/input/errors", &in_visitor_data, test_visitor_in_errors); + input_visitor_test_add("/visitor/input/wrong-type", + &in_visitor_data, test_visitor_in_wrong_type); input_visitor_test_add("/visitor/input/alternate-number", &in_visitor_data, test_visitor_in_alternate_number); input_visitor_test_add("/visitor/input/native_list/int", -- 2.4.3