Commit ec95f614 authored by Markus Armbruster's avatar Markus Armbruster
Browse files

test-qobject-input-visitor: Use strict visitor 



The qobject input visitor comes in a strict and a non-strict variant.
This test is the non-strict variant's last user.  Turns out it relies
on non-strict only in test_visitor_in_null(), and just out of
laziness.  We don't actually test the non-strict behavior.

Clean up test_visitor_in_null(), and switch to the strict variant.
The next commit will drop the non-strict variant.

Signed-off-by: default avatarMarkus Armbruster <armbru@redhat.com>
Reviewed-by: default avatarEric Blake <eblake@redhat.com>
Message-Id: <1488544368-30622-19-git-send-email-armbru@redhat.com>
parent 05601ed2

tests/test-qobject-input-visitor.c

+5 −5
Original line number Diff line number Diff line
@@ -49,7 +49,7 @@ static Visitor *visitor_input_test_init_internal(TestInputVisitorData *data, 
    data->obj = qobject_from_jsonv(json_string, ap);

    g_assert(data->obj);



    data->qiv = qobject_input_visitor_new(data->obj, false);

    data->qiv = qobject_input_visitor_new(data->obj, true);

    g_assert(data->qiv);

    return data->qiv;

}
@@ -290,14 +290,14 @@ static void test_visitor_in_null(TestInputVisitorData *data, 
     * when input is not null.

     */



    v = visitor_input_test_init(data, "{ 'a': null, 'b': '' }");

    v = visitor_input_test_init(data, "{ 'a': null, 'b': '', 'c': null }");

    visit_start_struct(v, NULL, NULL, 0, &error_abort);

    visit_type_null(v, "a", &error_abort);

    visit_type_str(v, "a", &tmp, &err);

    g_assert(!tmp);

    error_free_or_abort(&err);

    visit_type_null(v, "b", &err);

    error_free_or_abort(&err);

    visit_type_str(v, "c", &tmp, &err);

    g_assert(!tmp);

    error_free_or_abort(&err);

    visit_check_struct(v, &error_abort);

    visit_end_struct(v, NULL);

}