diff options
| author | Markus Armbruster <armbru@redhat.com> | 2019-09-14 17:34:59 +0200 |
|---|---|---|
| committer | Markus Armbruster <armbru@redhat.com> | 2019-09-24 14:07:23 +0200 |
| commit | dec0012ef8d644b5dde1b68ee8dab3f8c12e0b6b (patch) | |
| tree | 61c6ab6e7f0576311d94783c99045f117c73e14f /tests/qapi-schema/union-branch-if-invalid.err | |
| parent | c2c7065e1752a3be1b437d1ea359cde35d28ee3b (diff) | |
qapi: Fix missing 'if' checks in struct, union, alternate 'data'
Commit 87adbbffd4..3e270dcacc "qapi: Add 'if' to (implicit
struct|union|alternate) members" (v4.0.0) neglected test coverage, and
promptly failed to check the conditions. Review fail.
Recent commit "tests/qapi-schema: Demonstrate insufficient 'if'
checking" added test coverage, demonstrating the bug. Fix it by add
the missing check_if().
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Message-Id: <20190914153506.2151-13-armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Diffstat (limited to 'tests/qapi-schema/union-branch-if-invalid.err')
| -rw-r--r-- | tests/qapi-schema/union-branch-if-invalid.err | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/tests/qapi-schema/union-branch-if-invalid.err b/tests/qapi-schema/union-branch-if-invalid.err index e69de29bb2..607edee382 100644 --- a/tests/qapi-schema/union-branch-if-invalid.err +++ b/tests/qapi-schema/union-branch-if-invalid.err @@ -0,0 +1 @@ +tests/qapi-schema/union-branch-if-invalid.json:4: 'if' condition '' makes no sense |