提交 38788a32 编写于 作者: M Martin Kletzander

tests: Use libvirt properly with initialization and error dispatching

We were using "complicated" error printing in virnetservertest even
though we could've just dispatched the error.  Also add some good
practices that might come in handy (the code may fail without proper
initialization and event loop).
Signed-off-by: NMartin Kletzander <mkletzan@redhat.com>
上级 73ad20e9
...@@ -119,6 +119,8 @@ testCreateServer(const char *host, int family) ...@@ -119,6 +119,8 @@ testCreateServer(const char *host, int family)
goto error; goto error;
cleanup: cleanup:
if (!srv)
virDispatchError(NULL);
virObjectUnref(cln1); virObjectUnref(cln1);
virObjectUnref(cln2); virObjectUnref(cln2);
virObjectUnref(svc1); virObjectUnref(svc1);
...@@ -235,14 +237,8 @@ static int testExecRestart(const void *opaque) ...@@ -235,14 +237,8 @@ static int testExecRestart(const void *opaque)
ret = 0; ret = 0;
cleanup: cleanup:
if (ret < 0) { if (ret < 0)
virErrorPtr err = virGetLastError(); virDispatchError(NULL);
/* Rather be safe, we have lot of missing errors */
if (err)
fprintf(stderr, "%s\n", err->message);
else
fprintf(stderr, "%s\n", "Unknown error");
}
fail: fail:
VIR_FREE(infile); VIR_FREE(infile);
VIR_FREE(outfile); VIR_FREE(outfile);
...@@ -264,6 +260,12 @@ mymain(void) ...@@ -264,6 +260,12 @@ mymain(void)
{ {
int ret = 0; int ret = 0;
if (virInitialize() < 0 ||
virEventRegisterDefaultImpl() < 0) {
virDispatchError(NULL);
return EXIT_FAILURE;
}
/* Hack to make it easier to generate new JSON files when /* Hack to make it easier to generate new JSON files when
* the RPC classes change. Just set this env var, save * the RPC classes change. Just set this env var, save
* the generated JSON, and replace the file descriptor * the generated JSON, and replace the file descriptor
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册