[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH for-4.1 4/5] tests/test-hmp: Use qtest_init() in
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [Qemu-block] [PATCH for-4.1 4/5] tests/test-hmp: Use qtest_init() instead of qtest_start() |
Date: |
Tue, 9 Apr 2019 10:09:22 +0100 |
User-agent: |
Mutt/1.11.4 (2019-03-13) |
* Thomas Huth (address@hidden) wrote:
> qtest_start() + qtest_end() should be avoided, since they use the
> global_qtest variable that we want to get rid of in the long run
> Use qtest_init() and qtest_quit() instead.
>
> Signed-off-by: Thomas Huth <address@hidden>
Reviewed-by: Dr. David Alan Gilbert <address@hidden>
> ---
> tests/test-hmp.c | 19 ++++++++++---------
> 1 file changed, 10 insertions(+), 9 deletions(-)
>
> diff --git a/tests/test-hmp.c b/tests/test-hmp.c
> index 54a01824dc..f0d843e34a 100644
> --- a/tests/test-hmp.c
> +++ b/tests/test-hmp.c
> @@ -72,13 +72,13 @@ static const char *hmp_cmds[] = {
> };
>
> /* Run through the list of pre-defined commands */
> -static void test_commands(void)
> +static void test_commands(QTestState *qts)
> {
> char *response;
> int i;
>
> for (i = 0; hmp_cmds[i] != NULL; i++) {
> - response = hmp("%s", hmp_cmds[i]);
> + response = qtest_hmp(qts, "%s", hmp_cmds[i]);
> if (verbose) {
> fprintf(stderr,
> "\texecute HMP command: %s\n"
> @@ -91,11 +91,11 @@ static void test_commands(void)
> }
>
> /* Run through all info commands and call them blindly (without arguments) */
> -static void test_info_commands(void)
> +static void test_info_commands(QTestState *qts)
> {
> char *resp, *info, *info_buf, *endp;
>
> - info_buf = info = hmp("help info");
> + info_buf = info = qtest_hmp(qts, "help info");
>
> while (*info) {
> /* Extract the info command, ignore parameters and description */
> @@ -107,7 +107,7 @@ static void test_info_commands(void)
> if (verbose) {
> fprintf(stderr, "\t%s\n", info);
> }
> - resp = hmp("%s", info);
> + resp = qtest_hmp(qts, "%s", info);
> g_free(resp);
> /* And move forward to the next line */
> info = strchr(endp + 1, '\n');
> @@ -124,14 +124,15 @@ static void test_machine(gconstpointer data)
> {
> const char *machine = data;
> char *args;
> + QTestState *qts;
>
> args = g_strdup_printf("-S -M %s", machine);
> - qtest_start(args);
> + qts = qtest_init(args);
>
> - test_info_commands();
> - test_commands();
> + test_info_commands(qts);
> + test_commands(qts);
>
> - qtest_end();
> + qtest_quit(qts);
> g_free(args);
> g_free((void *)data);
> }
> --
> 2.21.0
>
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK
- [Qemu-block] [PATCH for-4.1 0/5] First batch of global_qtest-removement patches for QEMU 4.1, Thomas Huth, 2019/04/09
- [Qemu-block] [PATCH for-4.1 1/5] tests/tco: Make test independent of global_qtest, Thomas Huth, 2019/04/09
- [Qemu-block] [PATCH for-4.1 2/5] tests/megasas: Make test independent of global_qtest, Thomas Huth, 2019/04/09
- [Qemu-block] [PATCH for-4.1 3/5] tests/qmp-cmd-test: Use qtest_init() instead of qtest_start(), Thomas Huth, 2019/04/09
- [Qemu-block] [PATCH for-4.1 4/5] tests/test-hmp: Use qtest_init() instead of qtest_start(), Thomas Huth, 2019/04/09
- Re: [Qemu-block] [PATCH for-4.1 4/5] tests/test-hmp: Use qtest_init() instead of qtest_start(),
Dr. David Alan Gilbert <=
- [Qemu-block] [PATCH for-4.1 5/5] tests/ide-test: Make test independent of global_qtest, Thomas Huth, 2019/04/09