[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 18/22] test-clone-visitor: Wean off __org.qemu_x-Union1
From: |
Eric Blake |
Subject: |
Re: [PATCH 18/22] test-clone-visitor: Wean off __org.qemu_x-Union1 |
Date: |
Mon, 13 Sep 2021 10:32:07 -0500 |
User-agent: |
NeoMutt/20210205-772-2b4c52 |
On Mon, Sep 13, 2021 at 02:39:28PM +0200, Markus Armbruster wrote:
> test_clone_complex3() uses simple union __org.qemu_x-Union1 to cover
> arrays. Use UserDefOneList instead. Unions are still covered by
> test_clone_complex1().
>
> Signed-off-by: Markus Armbruster <armbru@redhat.com>
> ---
> tests/unit/test-clone-visitor.c | 70 ++++++++++++++++++---------------
> 1 file changed, 38 insertions(+), 32 deletions(-)
>
> diff --git a/tests/unit/test-clone-visitor.c b/tests/unit/test-clone-visitor.c
> index 8357a90e60..4048018607 100644
> --- a/tests/unit/test-clone-visitor.c
> +++ b/tests/unit/test-clone-visitor.c
> @@ -153,42 +153,48 @@ static void test_clone_complex2(void)
>
> static void test_clone_complex3(void)
> {
> - __org_qemu_x_Struct2 *src, *dst;
> - __org_qemu_x_Union1List *tmp;
> + UserDefOneList *src, *dst, *tail;
> + UserDefOne *elt;
This unit test loses coverge of RFQDN downstream extensions, but I
think we still adequately cover that elsewhere in the testsuite, and
that it was not the primary focus of this test. Meanwhile, what this
test is really focused on (an accurate clone of a union) is still
preserved.
Reviewed-by: Eric Blake <eblake@redhat.com>
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
- Re: [PATCH 17/22] tests/qapi-schema: Rewrite simple union TestIfUnion to be flat, (continued)
- [PATCH 09/22] qapi: Convert simple union ImageInfoSpecific to flat one, Markus Armbruster, 2021/09/13
- [PATCH 19/22] tests/qapi-schema: Drop simple union __org.qemu_x-Union1, Markus Armbruster, 2021/09/13
- [PATCH 13/22] test-qobject-output-visitor: Wean off UserDefListUnion, Markus Armbruster, 2021/09/13
- [PATCH 08/22] qapi: Convert simple union SocketAddressLegacy to flat one, Markus Armbruster, 2021/09/13
- [PATCH 18/22] test-clone-visitor: Wean off __org.qemu_x-Union1, Markus Armbruster, 2021/09/13
- Re: [PATCH 18/22] test-clone-visitor: Wean off __org.qemu_x-Union1,
Eric Blake <=
- [PATCH 21/22] qapi: Drop simple unions, Markus Armbruster, 2021/09/13
- Re: [PATCH 00/22] qapi: Remove simple unions from the schema language, Markus Armbruster, 2021/09/13