tests/qapi-schema: Cover empty union base
authorMarkus Armbruster <armbru@redhat.com>
Wed, 15 Mar 2017 12:56:57 +0000 (13:56 +0100)
committerMarkus Armbruster <armbru@redhat.com>
Thu, 16 Mar 2017 06:13:02 +0000 (07:13 +0100)
The new test case shows off qapi.py choking on an empty union base.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <1489582656-31133-9-git-send-email-armbru@redhat.com>

tests/Makefile.include
tests/qapi-schema/union-base-empty.err [new file with mode: 0644]
tests/qapi-schema/union-base-empty.exit [new file with mode: 0644]
tests/qapi-schema/union-base-empty.json [new file with mode: 0644]
tests/qapi-schema/union-base-empty.out [new file with mode: 0644]

index 16e0a9f615b8b3fddbb29903ef0958485d7e4548..3e640a846cf75e6b1890a643b44315df8eff02b8 100644 (file)
@@ -476,6 +476,7 @@ qapi-schema += unclosed-list.json
 qapi-schema += unclosed-object.json
 qapi-schema += unclosed-string.json
 qapi-schema += unicode-str.json
+qapi-schema += union-base-empty.json
 qapi-schema += union-base-no-discriminator.json
 qapi-schema += union-branch-case.json
 qapi-schema += union-clash-branches.json
diff --git a/tests/qapi-schema/union-base-empty.err b/tests/qapi-schema/union-base-empty.err
new file mode 100644 (file)
index 0000000..26bfa07
--- /dev/null
@@ -0,0 +1,10 @@
+Traceback (most recent call last):
+  File "tests/qapi-schema/test-qapi.py", line 56, in <module>
+    schema = QAPISchema(sys.argv[1])
+  File "scripts/qapi.py", line 1487, in __init__
+    self.exprs = check_exprs(parser.exprs)
+  File "scripts/qapi.py", line 921, in check_exprs
+    check_union(expr, info)
+  File "scripts/qapi.py", line 738, in check_union
+    assert base_members
+AssertionError
diff --git a/tests/qapi-schema/union-base-empty.exit b/tests/qapi-schema/union-base-empty.exit
new file mode 100644 (file)
index 0000000..d00491f
--- /dev/null
@@ -0,0 +1 @@
+1
diff --git a/tests/qapi-schema/union-base-empty.json b/tests/qapi-schema/union-base-empty.json
new file mode 100644 (file)
index 0000000..d1843d3
--- /dev/null
@@ -0,0 +1,9 @@
+# Flat union with empty base and therefore without discriminator
+
+{ 'struct': 'Empty', 'data': { } }
+
+{ 'union': 'TestUnion',
+  'base': 'Empty',
+  'discriminator': 'type',
+  'data': { 'value1': 'int',
+            'value2': 'str' } }
diff --git a/tests/qapi-schema/union-base-empty.out b/tests/qapi-schema/union-base-empty.out
new file mode 100644 (file)
index 0000000..e69de29