From 0badd8a43d252a7a92f903e32e5ab760d4f4af43 Mon Sep 17 00:00:00 2001 From: Michal Privoznik Date: Thu, 19 Sep 2013 13:56:30 +0200 Subject: [PATCH] tests: Learn qemuMonitorTestNew optional greeting Currently, when creating a new mocked monitor, the greeting can't be chosen. This is crucial for next patches, because some info as qemu version is obtained in the greeting message. Signed-off-by: Michal Privoznik --- tests/qemuhotplugtest.c | 2 +- tests/qemumonitortestutils.c | 10 ++++++---- tests/qemumonitortestutils.h | 5 +++-- 3 files changed, 10 insertions(+), 7 deletions(-) diff --git a/tests/qemuhotplugtest.c b/tests/qemuhotplugtest.c index 89480afc54..5148a21223 100644 --- a/tests/qemuhotplugtest.c +++ b/tests/qemuhotplugtest.c @@ -250,7 +250,7 @@ testQemuHotplug(const void *data) /* Now is the best time to feed the spoofed monitor with predefined * replies. */ - if (!(test_mon = qemuMonitorTestNew(true, driver.xmlopt, vm, &driver))) + if (!(test_mon = qemuMonitorTestNew(true, driver.xmlopt, vm, &driver, NULL))) goto cleanup; tmp = test->mon; diff --git a/tests/qemumonitortestutils.c b/tests/qemumonitortestutils.c index bca338541c..f38cce6641 100644 --- a/tests/qemumonitortestutils.c +++ b/tests/qemumonitortestutils.c @@ -873,7 +873,8 @@ qemuMonitorTestPtr qemuMonitorTestNew(bool json, virDomainXMLOptionPtr xmlopt, virDomainObjPtr vm, - virQEMUDriverPtr driver) + virQEMUDriverPtr driver, + const char *greeting) { qemuMonitorTestPtr test = NULL; virDomainChrSourceDef src; @@ -893,9 +894,10 @@ qemuMonitorTestNew(bool json, virObjectLock(test->mon); - if (qemuMonitorTestAddReponse(test, json ? - QEMU_JSON_GREETING : - QEMU_TEXT_GREETING) < 0) + if (!greeting) + greeting = json ? QEMU_JSON_GREETING : QEMU_TEXT_GREETING; + + if (qemuMonitorTestAddReponse(test, greeting) < 0) goto error; if (qemuMonitorCommonTestInit(test) < 0) diff --git a/tests/qemumonitortestutils.h b/tests/qemumonitortestutils.h index 9e39795d93..3f07f65575 100644 --- a/tests/qemumonitortestutils.h +++ b/tests/qemumonitortestutils.h @@ -61,12 +61,13 @@ int qemuMonitorTestAddItemParams(qemuMonitorTestPtr test, ATTRIBUTE_SENTINEL; # define qemuMonitorTestNewSimple(json, xmlopt) \ - qemuMonitorTestNew(json, xmlopt, NULL, NULL) + qemuMonitorTestNew(json, xmlopt, NULL, NULL, NULL) qemuMonitorTestPtr qemuMonitorTestNew(bool json, virDomainXMLOptionPtr xmlopt, virDomainObjPtr vm, - virQEMUDriverPtr driver); + virQEMUDriverPtr driver, + const char *greeting); qemuMonitorTestPtr qemuMonitorTestNewAgent(virDomainXMLOptionPtr xmlopt);