Commit b065413b authored by Nick Lewycky's avatar Nick Lewycky Committed by Simon McVittie

marshal-validate: run all the tests instead of just the even ones

Bug: https://bugs.freedesktop.org/show_bug.cgi?id=93908Reviewed-by: default avatarSimon McVittie <simon.mcvittie@collabora.co.uk>
[smcv: Re-enable the failing test that Nick disabled, and fix the
expected result; the result given by our current implementation is
reasonable.]
parent 96ffc2a0
......@@ -61,8 +61,6 @@ run_validity_tests (const ValidityTest *tests,
v, tests[i].data);
_dbus_assert_not_reached ("test failed");
}
++i;
}
}
......@@ -83,7 +81,7 @@ static const ValidityTest signature_tests[] = {
DBUS_INVALID_EXCEEDED_MAXIMUM_STRUCT_RECURSION },
{ ")", DBUS_INVALID_STRUCT_ENDED_BUT_NOT_STARTED },
{ "i)", DBUS_INVALID_STRUCT_ENDED_BUT_NOT_STARTED },
{ "a)", DBUS_INVALID_STRUCT_ENDED_BUT_NOT_STARTED },
{ "a)", DBUS_INVALID_MISSING_ARRAY_ELEMENT_TYPE },
{ "(", DBUS_INVALID_STRUCT_STARTED_BUT_NOT_ENDED },
{ "(i", DBUS_INVALID_STRUCT_STARTED_BUT_NOT_ENDED },
{ "(iiiii", DBUS_INVALID_STRUCT_STARTED_BUT_NOT_ENDED },
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment