diff options
| author | Markus Armbruster | 2018-08-06 08:53:35 +0200 |
|---|---|---|
| committer | Markus Armbruster | 2018-08-16 08:42:06 +0200 |
| commit | 82cab70bd884a77fbe4e738725c32c88daca70d6 (patch) | |
| tree | 0e121a24b1b8b3c55c8f1fe21b9dca6d60bfad84 /tests/qapi-schema/args-array-empty.err | |
| parent | cpu-plug-test: Don't pass integers as strings to device_add (diff) | |
| download | qemu-82cab70bd884a77fbe4e738725c32c88daca70d6.tar.gz qemu-82cab70bd884a77fbe4e738725c32c88daca70d6.tar.xz qemu-82cab70bd884a77fbe4e738725c32c88daca70d6.zip | |
tests: Clean up string interpolation around qtest_qmp_device_add()
Leaving interpolation into JSON to qmp() is more robust than building
QMP input manually, as explained in the commit before previous.
qtest_qmp_device_add() and its wrappers interpolate into JSON as
follows:
* qtest_qmp_device_add() interpolates members into a JSON object.
* So do its wrappers qpci_plug_device_test() and usb_test_hotplug().
* usb_test_hotplug() additionally interpolates strings and numbers
into JSON strings.
Clean them up:
* Have qtest_qmp_device_add() take its extra device properties as
arguments for qdict_from_jsonf_nofail() instead of a string
containing JSON members.
* Drop qpci_plug_device_test(), use qtest_qmp_device_add()
directly.
* Change usb_test_hotplug() parameter @port to string, to avoid
interpolation. Interpolate @hcd_id separately.
Bonus: gets rid of a non-literal format string. A step towards
compile-time format string checking without triggering
-Wformat-nonliteral.
Cc: Thomas Huth <thuth@redhat.com>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20180806065344.7103-15-armbru@redhat.com>
Diffstat (limited to 'tests/qapi-schema/args-array-empty.err')
0 files changed, 0 insertions, 0 deletions