[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 28/33] tests/qapi-schema: Cover two more syntax error
From: |
Markus Armbruster |
Subject: |
[Qemu-devel] [PULL 28/33] tests/qapi-schema: Cover two more syntax errors |
Date: |
Fri, 4 Sep 2015 16:21:35 +0200 |
Syntax error coverage should now be complete.
Signed-off-by: Markus Armbruster <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
---
tests/Makefile | 1 +
tests/qapi-schema/leading-comma-list.err | 1 +
tests/qapi-schema/leading-comma-list.exit | 1 +
tests/qapi-schema/leading-comma-list.json | 2 ++
tests/qapi-schema/leading-comma-list.out | 0
tests/qapi-schema/leading-comma-object.err | 1 +
tests/qapi-schema/leading-comma-object.exit | 1 +
tests/qapi-schema/leading-comma-object.json | 2 ++
tests/qapi-schema/leading-comma-object.out | 0
9 files changed, 9 insertions(+)
create mode 100644 tests/qapi-schema/leading-comma-list.err
create mode 100644 tests/qapi-schema/leading-comma-list.exit
create mode 100644 tests/qapi-schema/leading-comma-list.json
create mode 100644 tests/qapi-schema/leading-comma-list.out
create mode 100644 tests/qapi-schema/leading-comma-object.err
create mode 100644 tests/qapi-schema/leading-comma-object.exit
create mode 100644 tests/qapi-schema/leading-comma-object.json
create mode 100644 tests/qapi-schema/leading-comma-object.out
diff --git a/tests/Makefile b/tests/Makefile
index b8d445e..597ca90 100644
--- a/tests/Makefile
+++ b/tests/Makefile
@@ -237,6 +237,7 @@ check-qapi-schema-y := $(addprefix tests/qapi-schema/, \
missing-colon.json missing-comma-list.json missing-comma-object.json \
nested-struct-data.json non-objects.json \
qapi-schema-test.json quoted-structural-chars.json \
+ leading-comma-list.json leading-comma-object.json \
trailing-comma-list.json trailing-comma-object.json \
unclosed-list.json unclosed-object.json unclosed-string.json \
duplicate-key.json union-invalid-base.json union-bad-branch.json \
diff --git a/tests/qapi-schema/leading-comma-list.err
b/tests/qapi-schema/leading-comma-list.err
new file mode 100644
index 0000000..f5c870b
--- /dev/null
+++ b/tests/qapi-schema/leading-comma-list.err
@@ -0,0 +1 @@
+tests/qapi-schema/leading-comma-list.json:2:13: Expected "{", "[", "]",
string, boolean or "null"
diff --git a/tests/qapi-schema/leading-comma-list.exit
b/tests/qapi-schema/leading-comma-list.exit
new file mode 100644
index 0000000..d00491f
--- /dev/null
+++ b/tests/qapi-schema/leading-comma-list.exit
@@ -0,0 +1 @@
+1
diff --git a/tests/qapi-schema/leading-comma-list.json
b/tests/qapi-schema/leading-comma-list.json
new file mode 100644
index 0000000..c5ba501
--- /dev/null
+++ b/tests/qapi-schema/leading-comma-list.json
@@ -0,0 +1,2 @@
+{ 'enum': 'Status',
+ 'data': [ , 'good', 'bad', 'ugly' ] }
diff --git a/tests/qapi-schema/leading-comma-list.out
b/tests/qapi-schema/leading-comma-list.out
new file mode 100644
index 0000000..e69de29
diff --git a/tests/qapi-schema/leading-comma-object.err
b/tests/qapi-schema/leading-comma-object.err
new file mode 100644
index 0000000..f767b95
--- /dev/null
+++ b/tests/qapi-schema/leading-comma-object.err
@@ -0,0 +1 @@
+tests/qapi-schema/leading-comma-object.json:1:3: Expected string or "}"
diff --git a/tests/qapi-schema/leading-comma-object.exit
b/tests/qapi-schema/leading-comma-object.exit
new file mode 100644
index 0000000..d00491f
--- /dev/null
+++ b/tests/qapi-schema/leading-comma-object.exit
@@ -0,0 +1 @@
+1
diff --git a/tests/qapi-schema/leading-comma-object.json
b/tests/qapi-schema/leading-comma-object.json
new file mode 100644
index 0000000..c89023f
--- /dev/null
+++ b/tests/qapi-schema/leading-comma-object.json
@@ -0,0 +1,2 @@
+{ , 'enum': 'Status',
+ 'data': [ 'good', 'bad', 'ugly' ] }
diff --git a/tests/qapi-schema/leading-comma-object.out
b/tests/qapi-schema/leading-comma-object.out
new file mode 100644
index 0000000..e69de29
--
2.4.3
- [Qemu-devel] [PULL 02/33] qapi: Clean up cgen() and mcgen(), (continued)
- [Qemu-devel] [PULL 02/33] qapi: Clean up cgen() and mcgen(), Markus Armbruster, 2015/09/04
- [Qemu-devel] [PULL 22/33] qapi-commands: Fix gen_err_check(e) for e and e != 'local_err', Markus Armbruster, 2015/09/04
- [Qemu-devel] [PULL 08/33] qapi: Generate a nicer struct for flat unions, Markus Armbruster, 2015/09/04
- [Qemu-devel] [PULL 12/33] tests/qapi-schema: Document alternate's enum lacks visit function, Markus Armbruster, 2015/09/04
- [Qemu-devel] [PULL 16/33] qapi: Document flaws in checking of names, Markus Armbruster, 2015/09/04
- [Qemu-devel] [PULL 30/33] qapi: Fix errors for non-string, non-dictionary members, Markus Armbruster, 2015/09/04
- [Qemu-devel] [PULL 29/33] tests/qapi-schema: Cover non-string, non-dictionary members, Markus Armbruster, 2015/09/04
- [Qemu-devel] [PULL 24/33] qapi-commands: Don't feed output of mcgen() to mcgen() again, Markus Armbruster, 2015/09/04
- [Qemu-devel] [PULL 28/33] tests/qapi-schema: Cover two more syntax errors,
Markus Armbruster <=
- [Qemu-devel] [PULL 19/33] qapi-tests: New tests for union, alternate command arguments, Markus Armbruster, 2015/09/04
- [Qemu-devel] [PULL 09/33] qapi-visit: Fix generated code when schema has forward refs, Markus Armbruster, 2015/09/04
- [Qemu-devel] [PULL 21/33] qapi: Command returning anonymous type doesn't work, outlaw, Markus Armbruster, 2015/09/04
- [Qemu-devel] [PULL 20/33] qapi: Fix to reject union command and event arguments, Markus Armbruster, 2015/09/04
- [Qemu-devel] [PULL 32/33] docs/qapi-code-gen.txt: Fix QAPI schema examples, Markus Armbruster, 2015/09/04
- [Qemu-devel] [PULL 33/33] qapi: Generators crash when --output-dir isn't given, fix, Markus Armbruster, 2015/09/04
- [Qemu-devel] [PULL 31/33] qapi: Simplify error reporting for array types, Markus Armbruster, 2015/09/04
- [Qemu-devel] [PULL 10/33] qapi-visit: Replace list implicit_structs by set, Markus Armbruster, 2015/09/04
- [Qemu-devel] [PULL 17/33] tests/qapi-schema: Restore test case for flat union base bug, Markus Armbruster, 2015/09/04
- [Qemu-devel] [PULL 26/33] qapi: Generated code cleanup, Markus Armbruster, 2015/09/04