tests: virjson: Modify logic in testJSONFromString

To allow better testing in case where the string was parsed, modify the
logic so that the regular code path is not included in a conditional
block.
This commit is contained in:
Peter Krempa 2017-07-03 15:02:38 +02:00
parent 255cd95aab
commit 0a2866b42a

View File

@ -27,24 +27,24 @@ testJSONFromString(const void *data)
json = virJSONValueFromString(info->doc);
if (info->pass) {
if (!json) {
if (!json) {
if (info->pass) {
VIR_TEST_VERBOSE("Fail to parse %s\n", info->doc);
ret = -1;
goto cleanup;
} else {
VIR_TEST_DEBUG("Parsed %s\n", info->doc);
}
} else {
if (json) {
VIR_TEST_VERBOSE("Should not have parsed %s\n", info->doc);
ret = -1;
goto cleanup;
} else {
VIR_TEST_DEBUG("Fail to parse %s\n", info->doc);
ret = 0;
goto cleanup;
}
}
if (!info->pass) {
VIR_TEST_VERBOSE("Should not have parsed %s\n", info->doc);
goto cleanup;
}
VIR_TEST_DEBUG("Parsed %s\n", info->doc);
ret = 0;
cleanup: