[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v2 17/23] tests/qapi-schema: Rewrite simple union TestIfUnion to
From: |
Markus Armbruster |
Subject: |
[PATCH v2 17/23] tests/qapi-schema: Rewrite simple union TestIfUnion to be flat |
Date: |
Fri, 17 Sep 2021 16:31:28 +0200 |
Simple unions predate flat unions. Having both complicates the QAPI
schema language and the QAPI generator. We haven't been using simple
unions in new code for a long time, because they are less flexible and
somewhat awkward on the wire.
To prepare for their removal, rewrite TestIfUnion to be flat.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
---
tests/qapi-schema/qapi-schema-test.json | 4 +++-
tests/qapi-schema/qapi-schema-test.out | 16 ++++++----------
2 files changed, 9 insertions(+), 11 deletions(-)
diff --git a/tests/qapi-schema/qapi-schema-test.json
b/tests/qapi-schema/qapi-schema-test.json
index eae43f41c4..ef17ab1aae 100644
--- a/tests/qapi-schema/qapi-schema-test.json
+++ b/tests/qapi-schema/qapi-schema-test.json
@@ -229,8 +229,10 @@
'if': 'TEST_IF_ENUM' }
{ 'union': 'TestIfUnion',
+ 'base': { 'type': 'TestIfEnum' },
+ 'discriminator': 'type',
'data': { 'foo': 'TestStruct',
- 'bar': { 'type': 'str', 'if': 'TEST_IF_UNION_BAR'} },
+ 'bar': { 'type': 'UserDefZero', 'if': 'TEST_IF_ENUM_BAR'} },
'if': { 'all': ['TEST_IF_UNION', 'TEST_IF_STRUCT'] } }
{ 'command': 'test-if-union-cmd',
diff --git a/tests/qapi-schema/qapi-schema-test.out
b/tests/qapi-schema/qapi-schema-test.out
index e43073d795..07e4161331 100644
--- a/tests/qapi-schema/qapi-schema-test.out
+++ b/tests/qapi-schema/qapi-schema-test.out
@@ -257,19 +257,15 @@ enum TestIfEnum
member bar
if TEST_IF_ENUM_BAR
if TEST_IF_ENUM
-object q_obj_TestStruct-wrapper
- member data: TestStruct optional=False
-enum TestIfUnionKind
- member foo
- member bar
- if TEST_IF_UNION_BAR
+object q_obj_TestIfUnion-base
+ member type: TestIfEnum optional=False
if {'all': ['TEST_IF_UNION', 'TEST_IF_STRUCT']}
object TestIfUnion
- member type: TestIfUnionKind optional=False
+ base q_obj_TestIfUnion-base
tag type
- case foo: q_obj_TestStruct-wrapper
- case bar: q_obj_str-wrapper
- if TEST_IF_UNION_BAR
+ case foo: TestStruct
+ case bar: UserDefZero
+ if TEST_IF_ENUM_BAR
if {'all': ['TEST_IF_UNION', 'TEST_IF_STRUCT']}
object q_obj_test-if-union-cmd-arg
member union-cmd-arg: TestIfUnion optional=False
--
2.31.1
- [PATCH v2 23/23] test-clone-visitor: Correct an accidental rename, (continued)
- [PATCH v2 23/23] test-clone-visitor: Correct an accidental rename, Markus Armbruster, 2021/09/17
- [PATCH v2 13/23] test-qobject-output-visitor: Wean off UserDefListUnion, Markus Armbruster, 2021/09/17
- [PATCH v2 20/23] tests/qapi-schema: Purge simple unions from tests, Markus Armbruster, 2021/09/17
- [PATCH v2 21/23] qapi: Drop simple unions, Markus Armbruster, 2021/09/17
- [PATCH v2 08/23] qapi: Convert simple union SocketAddressLegacy to flat one, Markus Armbruster, 2021/09/17
- [PATCH v2 14/23] test-clone-visitor: Wean off UserDefListUnion, Markus Armbruster, 2021/09/17
- [PATCH v2 18/23] test-clone-visitor: Wean off __org.qemu_x-Union1, Markus Armbruster, 2021/09/17
- [PATCH v2 22/23] tests/qapi-schema: Rename flat-union-* test cases to union-*, Markus Armbruster, 2021/09/17
- [PATCH v2 02/23] qapi: Stop enforcing "type name should not end in 'Kind', Markus Armbruster, 2021/09/17
- [PATCH v2 10/23] qapi: Convert simple union TransactionAction to flat one, Markus Armbruster, 2021/09/17
- [PATCH v2 17/23] tests/qapi-schema: Rewrite simple union TestIfUnion to be flat,
Markus Armbruster <=
- [PATCH v2 12/23] test-qobject-input-visitor: Wean off UserDefListUnion, Markus Armbruster, 2021/09/17
- [PATCH v2 04/23] qapi: Convert simple union InputEvent to flat one, Markus Armbruster, 2021/09/17