aboutsummaryrefslogtreecommitdiff
path: root/tests/qapi-schema/enum-clash-member.out
diff options
context:
space:
mode:
authorEric Blake <eblake@redhat.com>2015-05-04 09:05:04 -0600
committerMarkus Armbruster <armbru@redhat.com>2015-05-05 18:39:00 +0200
commitcf3935907b5df16f667d54ad6761c7e937dcf425 (patch)
treeac11b4cc953cc0d7a605eb589e13128f15cf5803 /tests/qapi-schema/enum-clash-member.out
parentad11dbb93752ffd4bd1d5f31da7e2d9c40a68e8a (diff)
qapi: Better error messages for bad enums
The previous commit demonstrated that the generator had several flaws with less-than-perfect enums: - an enum that listed the same string twice (or two variant strings that map to the same C enumerator) ended up generating an invalid C enum - because the generator adds a _MAX terminator to each enum, the use of an enum member 'max' can also cause this clash - if an enum omits 'data', the generator left a python stack trace rather than a graceful message - an enum that used a non-array 'data' was silently accepted by the parser - an enum that used non-string members in the 'data' member was silently accepted by the parser Add check_enum to cover these situations, and update testcases to match. While valid .json files won't trigger any of these cases, we might as well be nicer to developers that make a typo while trying to add new QAPI code. Signed-off-by: Eric Blake <eblake@redhat.com> Reviewed-by: Markus Armbruster <armbru@redhat.com> Signed-off-by: Markus Armbruster <armbru@redhat.com>
Diffstat (limited to 'tests/qapi-schema/enum-clash-member.out')
-rw-r--r--tests/qapi-schema/enum-clash-member.out3
1 files changed, 0 insertions, 3 deletions
diff --git a/tests/qapi-schema/enum-clash-member.out b/tests/qapi-schema/enum-clash-member.out
index 0814459721..e69de29bb2 100644
--- a/tests/qapi-schema/enum-clash-member.out
+++ b/tests/qapi-schema/enum-clash-member.out
@@ -1,3 +0,0 @@
-[OrderedDict([('enum', 'MyEnum'), ('data', ['one', 'ONE'])])]
-[{'enum_name': 'MyEnum', 'enum_values': ['one', 'ONE']}]
-[]