aboutsummaryrefslogtreecommitdiff
path: root/tests/qemu-iotests/065
diff options
context:
space:
mode:
authorJohn Snow <jsnow@redhat.com>2022-03-21 16:16:14 -0400
committerHanna Reitz <hreitz@redhat.com>2022-03-22 10:14:38 +0100
commit4cf661f2c0540d3644b62c0ed64dff14fdd3bd68 (patch)
tree9f7e405455690f6dfb5dd4be210198c0db09073b /tests/qemu-iotests/065
parent88baae552dad5c0b91973cae16b0f4a927129f51 (diff)
downloadqemu-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-xtests/qemu-iotests/0654
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: