diff options
author | John Snow <jsnow@redhat.com> | 2022-03-21 16:16:14 -0400 |
---|---|---|
committer | Hanna Reitz <hreitz@redhat.com> | 2022-03-22 10:14:38 +0100 |
commit | 4cf661f2c0540d3644b62c0ed64dff14fdd3bd68 (patch) | |
tree | 9f7e405455690f6dfb5dd4be210198c0db09073b /tests/qemu-iotests/065 | |
parent | 88baae552dad5c0b91973cae16b0f4a927129f51 (diff) | |
download | qemu-4cf661f2c0540d3644b62c0ed64dff14fdd3bd68.zip qemu-4cf661f2c0540d3644b62c0ed64dff14fdd3bd68.tar.gz qemu-4cf661f2c0540d3644b62c0ed64dff14fdd3bd68.tar.bz2 |
iotests: remove remaining calls to qemu_img_pipe()
As part of moving all python iotest invocations of qemu-img onto a
single qemu_img() implementation, remove a few lingering uses of
qemu_img_pipe() from outside of iotests.py itself.
Several cases here rely on the knowledge that qemu_img_pipe() suppresses
*all* output on a successful case when the command being issued is
'create'.
065: This call's output is inspected, but it appears as if it's expected
to succeed. Replace this call with the checked qemu_img() variant
instead to get better diagnostics if/when qemu-img itself fails.
237: "create" call output isn't actually logged. Use qemu_img_create()
instead, which checks the return code. Remove the empty lines from
the test output.
296: Two calls;
-create: Expected to succeed. Like other create calls, the output
isn't actually logged. Switch to a checked variant
(qemu_img_create) instead. The output for this test is
a mixture of both test styles, so actually replace the
blank line for readability.
-amend: This is expected to fail. Log the output.
After this patch, the only uses of qemu_img_pipe are internal to
iotests.py and will be removed in subsequent patches.
Signed-off-by: John Snow <jsnow@redhat.com>
Reviewed-by: Hanna Reitz <hreitz@redhat.com>
Message-Id: <20220321201618.903471-15-jsnow@redhat.com>
Signed-off-by: Hanna Reitz <hreitz@redhat.com>
Diffstat (limited to 'tests/qemu-iotests/065')
-rwxr-xr-x | tests/qemu-iotests/065 | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/qemu-iotests/065 b/tests/qemu-iotests/065 index 9466ce7..ba94e19 100755 --- a/tests/qemu-iotests/065 +++ b/tests/qemu-iotests/065 @@ -24,7 +24,7 @@ import os import re import json import iotests -from iotests import qemu_img, qemu_img_info, qemu_img_pipe +from iotests import qemu_img, qemu_img_info import unittest test_img = os.path.join(iotests.test_dir, 'test.img') @@ -54,7 +54,7 @@ class TestQemuImgInfo(TestImageInfoSpecific): self.assertEqual(data['data'], self.json_compare) def test_human(self): - data = qemu_img_pipe('info', '--output=human', test_img).split('\n') + data = qemu_img('info', '--output=human', test_img).stdout.split('\n') data = data[(data.index('Format specific information:') + 1) :data.index('')] for field in data: |