add some tests for invalid JSON
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>
This commit is contained in:
parent
2e89c06889
commit
7f8fca7c8a
@ -638,11 +638,90 @@ START_TEST(simple_varargs)
|
|||||||
}
|
}
|
||||||
END_TEST
|
END_TEST
|
||||||
|
|
||||||
|
START_TEST(empty_input)
|
||||||
|
{
|
||||||
|
QObject *obj = qobject_from_json("");
|
||||||
|
fail_unless(obj == NULL);
|
||||||
|
}
|
||||||
|
END_TEST
|
||||||
|
|
||||||
|
START_TEST(unterminated_string)
|
||||||
|
{
|
||||||
|
QObject *obj = qobject_from_json("\"abc");
|
||||||
|
fail_unless(obj == NULL);
|
||||||
|
}
|
||||||
|
END_TEST
|
||||||
|
|
||||||
|
START_TEST(unterminated_sq_string)
|
||||||
|
{
|
||||||
|
QObject *obj = qobject_from_json("'abc");
|
||||||
|
fail_unless(obj == NULL);
|
||||||
|
}
|
||||||
|
END_TEST
|
||||||
|
|
||||||
|
START_TEST(unterminated_escape)
|
||||||
|
{
|
||||||
|
QObject *obj = qobject_from_json("\"abc\\\"");
|
||||||
|
fail_unless(obj == NULL);
|
||||||
|
}
|
||||||
|
END_TEST
|
||||||
|
|
||||||
|
START_TEST(unterminated_array)
|
||||||
|
{
|
||||||
|
QObject *obj = qobject_from_json("[32");
|
||||||
|
fail_unless(obj == NULL);
|
||||||
|
}
|
||||||
|
END_TEST
|
||||||
|
|
||||||
|
START_TEST(unterminated_array_comma)
|
||||||
|
{
|
||||||
|
QObject *obj = qobject_from_json("[32,");
|
||||||
|
fail_unless(obj == NULL);
|
||||||
|
}
|
||||||
|
END_TEST
|
||||||
|
|
||||||
|
START_TEST(invalid_array_comma)
|
||||||
|
{
|
||||||
|
QObject *obj = qobject_from_json("[32,}");
|
||||||
|
fail_unless(obj == NULL);
|
||||||
|
}
|
||||||
|
END_TEST
|
||||||
|
|
||||||
|
START_TEST(unterminated_dict)
|
||||||
|
{
|
||||||
|
QObject *obj = qobject_from_json("{'abc':32");
|
||||||
|
fail_unless(obj == NULL);
|
||||||
|
}
|
||||||
|
END_TEST
|
||||||
|
|
||||||
|
START_TEST(unterminated_dict_comma)
|
||||||
|
{
|
||||||
|
QObject *obj = qobject_from_json("{'abc':32,");
|
||||||
|
fail_unless(obj == NULL);
|
||||||
|
}
|
||||||
|
END_TEST
|
||||||
|
|
||||||
|
#if 0
|
||||||
|
START_TEST(invalid_dict_comma)
|
||||||
|
{
|
||||||
|
QObject *obj = qobject_from_json("{'abc':32,}");
|
||||||
|
fail_unless(obj == NULL);
|
||||||
|
}
|
||||||
|
END_TEST
|
||||||
|
|
||||||
|
START_TEST(unterminated_literal)
|
||||||
|
{
|
||||||
|
QObject *obj = qobject_from_json("nul");
|
||||||
|
fail_unless(obj == NULL);
|
||||||
|
}
|
||||||
|
END_TEST
|
||||||
|
#endif
|
||||||
|
|
||||||
static Suite *qjson_suite(void)
|
static Suite *qjson_suite(void)
|
||||||
{
|
{
|
||||||
Suite *suite;
|
Suite *suite;
|
||||||
TCase *string_literals, *number_literals, *keyword_literals;
|
TCase *string_literals, *number_literals, *keyword_literals;
|
||||||
TCase *dicts, *lists, *whitespace, *varargs;
|
TCase *dicts, *lists, *whitespace, *varargs, *errors;
|
||||||
|
|
||||||
string_literals = tcase_create("String Literals");
|
string_literals = tcase_create("String Literals");
|
||||||
tcase_add_test(string_literals, simple_string);
|
tcase_add_test(string_literals, simple_string);
|
||||||
@ -668,6 +747,22 @@ static Suite *qjson_suite(void)
|
|||||||
varargs = tcase_create("Varargs");
|
varargs = tcase_create("Varargs");
|
||||||
tcase_add_test(varargs, simple_varargs);
|
tcase_add_test(varargs, simple_varargs);
|
||||||
|
|
||||||
|
errors = tcase_create("Invalid JSON");
|
||||||
|
tcase_add_test(errors, empty_input);
|
||||||
|
tcase_add_test(errors, unterminated_string);
|
||||||
|
tcase_add_test(errors, unterminated_escape);
|
||||||
|
tcase_add_test(errors, unterminated_sq_string);
|
||||||
|
tcase_add_test(errors, unterminated_array);
|
||||||
|
tcase_add_test(errors, unterminated_array_comma);
|
||||||
|
tcase_add_test(errors, invalid_array_comma);
|
||||||
|
tcase_add_test(errors, unterminated_dict);
|
||||||
|
tcase_add_test(errors, unterminated_dict_comma);
|
||||||
|
#if 0
|
||||||
|
/* FIXME: this print parse error messages on stderr. */
|
||||||
|
tcase_add_test(errors, invalid_dict_comma);
|
||||||
|
tcase_add_test(errors, unterminated_literal);
|
||||||
|
#endif
|
||||||
|
|
||||||
suite = suite_create("QJSON test-suite");
|
suite = suite_create("QJSON test-suite");
|
||||||
suite_add_tcase(suite, string_literals);
|
suite_add_tcase(suite, string_literals);
|
||||||
suite_add_tcase(suite, number_literals);
|
suite_add_tcase(suite, number_literals);
|
||||||
@ -676,6 +771,7 @@ static Suite *qjson_suite(void)
|
|||||||
suite_add_tcase(suite, lists);
|
suite_add_tcase(suite, lists);
|
||||||
suite_add_tcase(suite, whitespace);
|
suite_add_tcase(suite, whitespace);
|
||||||
suite_add_tcase(suite, varargs);
|
suite_add_tcase(suite, varargs);
|
||||||
|
suite_add_tcase(suite, errors);
|
||||||
|
|
||||||
return suite;
|
return suite;
|
||||||
}
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user