[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 4/4] tests: test-hmp: print command execution re
From: |
Eduardo Otubo |
Subject: |
Re: [Qemu-devel] [PATCH 4/4] tests: test-hmp: print command execution result |
Date: |
Wed, 25 Oct 2017 14:03:23 +0200 |
User-agent: |
Mutt/1.8.3+47 (5f034395e53d) (2017-05-23) |
On Mon, Oct 23, 2017 at 05:13:10PM +0200, Vadim Galitsyn wrote:
> Provide HMP monitor command execution result as it would be seen
> by user who established an HMP monitor session.
>
> Currently many commands may silently fail without any sign of that.
> This patch let this info to be printed once test is running in
> verbose mode.
>
> For the future it might be useful to fail the test if command has
> failed, however it would require a bit of rework inside test
> engine itself.
>
> A simple example of silent failure without reporting it would to
> add some non-existent HMP command into 'hmp_cmds' list. In this case
> test will report it successfully passed without error.
>
> Signed-off-by: Vadim Galitsyn <address@hidden>
> Cc: Dr. David Alan Gilbert <address@hidden>
> Cc: address@hidden
> ---
> tests/test-hmp.c | 7 +++++--
> 1 file changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/tests/test-hmp.c b/tests/test-hmp.c
> index 8e21eee61c..1fd8e44eff 100644
> --- a/tests/test-hmp.c
> +++ b/tests/test-hmp.c
> @@ -79,10 +79,13 @@ static void test_commands(void)
> int i;
>
> for (i = 0; hmp_cmds[i] != NULL; i++) {
> + response = hmp("%s", hmp_cmds[i]);
> if (verbose) {
> - fprintf(stderr, "\t%s\n", hmp_cmds[i]);
> + fprintf(stderr,
> + "\texecute HMP command: %s\n"
> + "\tresult : %s\n",
> + hmp_cmds[i], response);
> }
> - response = hmp("%s", hmp_cmds[i]);
> g_free(response);
> }
>
> --
> 2.13.1.394.g41dd433
>
>
Reviewed-by: Eduardo Otubo <address@hidden>
--
Eduardo Otubo
- [Qemu-devel] QMP, HMP: introduce 'writeconfig' command, Vadim Galitsyn, 2017/10/23
- [Qemu-devel] [PATCH 1/4] qmp: introduce 'writeconfig' command, Vadim Galitsyn, 2017/10/23
- [Qemu-devel] [PATCH 2/4] hmp: introduce 'writeconfig' command, Vadim Galitsyn, 2017/10/23
- [Qemu-devel] [PATCH 3/4] tests: test-hmp: extend with 'writeconfig' command, Vadim Galitsyn, 2017/10/23
- [Qemu-devel] [PATCH 4/4] tests: test-hmp: print command execution result, Vadim Galitsyn, 2017/10/23
- Re: [Qemu-devel] [PATCH 4/4] tests: test-hmp: print command execution result,
Eduardo Otubo <=
- Re: [Qemu-devel] QMP, HMP: introduce 'writeconfig' command, Dr. David Alan Gilbert, 2017/10/24