Re: [Qemu-devel] [PATCHv2] tests: start a /qga/guest-exec test

[Michael Roth]

Quoting address@hidden (2016-06-03 07:27:50)
> From: Marc-André Lureau <address@hidden>
> 
> Test a few guest-exec guest agent commands, added in qemu 2.5.
> 
> Signed-off-by: Marc-André Lureau <address@hidden>

Thanks, applied to qga tree:

  https://github.com/mdroth/qemu/tree/qga

> ---
>  tests/test-qga.c | 81 
> ++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 81 insertions(+)
> 
> diff --git a/tests/test-qga.c b/tests/test-qga.c
> index 72a89de..10b29eb 100644
> --- a/tests/test-qga.c
> +++ b/tests/test-qga.c
> @@ -823,6 +823,84 @@ static void test_qga_fsfreeze_and_thaw(gconstpointer fix)
>      QDECREF(ret);
>  }
> 
> +static void test_qga_guest_exec(gconstpointer fix)
> +{
> +    const TestFixture *fixture = fix;
> +    QDict *ret, *val;
> +    const gchar *out;
> +    guchar *decoded;
> +    int64_t pid, now, exitcode;
> +    gsize len;
> +    bool exited;
> +
> +    /* exec 'echo foo bar' */
> +    ret = qmp_fd(fixture->fd, "{'execute': 'guest-exec', 'arguments': {"
> +                 " 'path': '/bin/echo', 'arg': [ '-n', '\" test_str \"' ],"
> +                 " 'capture-output': true } }");
> +    g_assert_nonnull(ret);
> +    qmp_assert_no_error(ret);
> +    val = qdict_get_qdict(ret, "return");
> +    pid = qdict_get_int(val, "pid");
> +    g_assert_cmpint(pid, >, 0);
> +    QDECREF(ret);
> +
> +    /* wait for completion */
> +    now = g_get_monotonic_time();
> +    do {
> +        ret = qmp_fd(fixture->fd, "{'execute': 'guest-exec-status',"
> +                     " 'arguments': { 'pid': %" PRId64 "  } }", pid);
> +        g_assert_nonnull(ret);
> +        val = qdict_get_qdict(ret, "return");
> +        exited = qdict_get_bool(val, "exited");
> +        if (!exited) {
> +            QDECREF(ret);
> +        }
> +    } while (!exited &&
> +             g_get_monotonic_time() < now + 5 * G_TIME_SPAN_SECOND);
> +    g_assert(exited);
> +
> +    /* check stdout */
> +    exitcode = qdict_get_int(val, "exitcode");
> +    g_assert_cmpint(exitcode, ==, 0);
> +    out = qdict_get_str(val, "out-data");
> +    decoded = g_base64_decode(out, &len);
> +    g_assert_cmpint(len, ==, 12);
> +    g_assert_cmpstr((char *)decoded, ==, "\" test_str \"");
> +    g_free(decoded);
> +    QDECREF(ret);
> +}
> +
> +static void test_qga_guest_exec_invalid(gconstpointer fix)
> +{
> +    const TestFixture *fixture = fix;
> +    QDict *ret, *error;
> +    const gchar *class, *desc;
> +
> +    /* invalid command */
> +    ret = qmp_fd(fixture->fd, "{'execute': 'guest-exec', 'arguments': {"
> +                 " 'path': '/bin/invalid-cmd42' } }");
> +    g_assert_nonnull(ret);
> +    error = qdict_get_qdict(ret, "error");
> +    g_assert_nonnull(error);
> +    class = qdict_get_str(error, "class");
> +    desc = qdict_get_str(error, "desc");
> +    g_assert_cmpstr(class, ==, "GenericError");
> +    g_assert_cmpint(strlen(desc), >, 0);
> +    QDECREF(ret);
> +
> +    /* invalid pid */
> +    ret = qmp_fd(fixture->fd, "{'execute': 'guest-exec-status',"
> +                 " 'arguments': { 'pid': 0 } }");
> +    g_assert_nonnull(ret);
> +    error = qdict_get_qdict(ret, "error");
> +    g_assert_nonnull(error);
> +    class = qdict_get_str(error, "class");
> +    desc = qdict_get_str(error, "desc");
> +    g_assert_cmpstr(class, ==, "GenericError");
> +    g_assert_cmpint(strlen(desc), >, 0);
> +    QDECREF(ret);
> +}
> +
>  int main(int argc, char **argv)
>  {
>      TestFixture fix;
> @@ -853,6 +931,9 @@ int main(int argc, char **argv)
> 
>      g_test_add_data_func("/qga/blacklist", NULL, test_qga_blacklist);
>      g_test_add_data_func("/qga/config", NULL, test_qga_config);
> +    g_test_add_data_func("/qga/guest-exec", &fix, test_qga_guest_exec);
> +    g_test_add_data_func("/qga/guest-exec-invalid", &fix,
> +                         test_qga_guest_exec_invalid);
> 
>      if (g_getenv("QGA_TEST_SIDE_EFFECTING")) {
>          g_test_add_data_func("/qga/fsfreeze-and-thaw", &fix,
> -- 
> 2.7.4
>