]> xenbits.xensource.com Git - libvirt.git/commitdiff
qemumonitorjsontest: Work around deprecation of 'vcpu' argument of 'trace-event-get...
authorPeter Krempa <pkrempa@redhat.com>
Tue, 6 Jun 2023 09:03:39 +0000 (11:03 +0200)
committerPeter Krempa <pkrempa@redhat.com>
Tue, 6 Jun 2023 12:23:23 +0000 (14:23 +0200)
'trace-event-get-state' was used for testing schema validation as it had
simple arguments. Now 'vcpu' is optional and deprecated. Fix the test so
that it won't break with upcoming qemu-8.1.

Drop the 'all-attrs' case, as it's not not really testing anything
special and for the 'missing mandatory attr' case use an empty object.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Michal Privoznik <mprivozn@redhat.com>
tests/qemumonitorjsontest.c

index 7e79b2e7579911ad0976fbdc06dbd230e564c07f..8ef4ea7056c3cd98a83151132059546df2b7312b 100644 (file)
@@ -3021,12 +3021,10 @@ mymain(void)
 
     DO_TEST_QAPI_VALIDATE("string", "trace-event-get-state/arg-type", true,
                           "{\"name\":\"test\"}");
-    DO_TEST_QAPI_VALIDATE("all attrs", "trace-event-get-state/arg-type", true,
-                          "{\"name\":\"test\", \"vcpu\":123}");
     DO_TEST_QAPI_VALIDATE("attr type mismatch", "trace-event-get-state/arg-type", false,
                           "{\"name\":123}");
     DO_TEST_QAPI_VALIDATE("missing mandatory attr", "trace-event-get-state/arg-type", false,
-                          "{\"vcpu\":123}");
+                          "{}");
     DO_TEST_QAPI_VALIDATE("attr name not present", "trace-event-get-state/arg-type", false,
                           "{\"name\":\"test\", \"blah\":123}");
     DO_TEST_QAPI_VALIDATE("variant", "blockdev-add/arg-type", true,