Marc-André Lureau <marcandre.lur...@redhat.com> writes: > Replace the generated json string with a literal qobject. The later is > easier to deal with, at run time, as well as compile time: #if blocks > can be more easily added than in a json string. > > Signed-off-by: Marc-André Lureau <marcandre.lur...@redhat.com> [...] > diff --git a/tests/test-qobject-input-visitor.c > b/tests/test-qobject-input-visitor.c > index bcf02617dc..1969733971 100644 > --- a/tests/test-qobject-input-visitor.c > +++ b/tests/test-qobject-input-visitor.c > @@ -1247,24 +1247,26 @@ static void > test_visitor_in_fail_alternate(TestInputVisitorData *data, > } > > static void do_test_visitor_in_qmp_introspect(TestInputVisitorData *data, > - const char *schema_json) > + const QLitObject *qlit) > { > SchemaInfoList *schema = NULL; > + QObject *obj = qobject_from_qlit(qlit); > Visitor *v; > > - v = visitor_input_test_init_raw(data, schema_json); > + v = qobject_input_visitor_new(obj); > > visit_type_SchemaInfoList(v, NULL, &schema, &error_abort); > g_assert(schema); > > qapi_free_SchemaInfoList(schema); > + qobject_decref(obj); > }
Are you leaking @v? > > static void test_visitor_in_qmp_introspect(TestInputVisitorData *data, > const void *unused) > { > - do_test_visitor_in_qmp_introspect(data, test_qmp_schema_json); > - do_test_visitor_in_qmp_introspect(data, qmp_schema_json); > + do_test_visitor_in_qmp_introspect(data, &test_qmp_schema_qlit); > + do_test_visitor_in_qmp_introspect(data, &qmp_schema_qlit); > } > > int main(int argc, char **argv) [...]