diff options
| author | Markus Armbruster <armbru@redhat.com> | 2017-05-22 18:42:14 +0200 |
|---|---|---|
| committer | Markus Armbruster <armbru@redhat.com> | 2017-05-31 16:04:05 +0200 |
| commit | 8168ca8ea3699b9fca4d8c948c7fa6ecdedc4a97 (patch) | |
| tree | aabf333374f1e7995ea807d068fb745f0baab59c /tests/test-qobject-output-visitor.c | |
| parent | 8339fa266ccd03a43d4a9f196f1098fa3c6924f2 (diff) | |
| download | qemu-8168ca8ea3699b9fca4d8c948c7fa6ecdedc4a97.tar.gz | |
tests/qapi-schema: Avoid 'str' in alternate test cases
The next commit is going to make alternate members of type 'str'
conflict with other scalar types. Would break a few test cases that
don't actually require 'str'. Flip them from 'str' to 'bool' or
'EnumOne'.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Message-Id: <1495471335-23707-4-git-send-email-armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Diffstat (limited to 'tests/test-qobject-output-visitor.c')
| -rw-r--r-- | tests/test-qobject-output-visitor.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/test-qobject-output-visitor.c b/tests/test-qobject-output-visitor.c index 94b9518e40..4e8fdf1397 100644 --- a/tests/test-qobject-output-visitor.c +++ b/tests/test-qobject-output-visitor.c @@ -406,12 +406,12 @@ static void test_visitor_out_alternate(TestOutputVisitorData *data, visitor_reset(data); tmp = g_new0(UserDefAlternate, 1); tmp->type = QTYPE_QSTRING; - tmp->u.s = g_strdup("hello"); + tmp->u.e = ENUM_ONE_VALUE1; visit_type_UserDefAlternate(data->ov, NULL, &tmp, &error_abort); qstr = qobject_to_qstring(visitor_get(data)); g_assert(qstr); - g_assert_cmpstr(qstring_get_str(qstr), ==, "hello"); + g_assert_cmpstr(qstring_get_str(qstr), ==, "value1"); qapi_free_UserDefAlternate(tmp); |