diff options
| author | Daniel P. Berrangé <berrange@redhat.com> | 2024-11-29 17:31:02 +0000 |
|---|---|---|
| committer | Thomas Huth <thuth@redhat.com> | 2024-12-11 09:19:12 +0100 |
| commit | f29c96d0f5760f72220623500ee659f1fc2d1542 (patch) | |
| tree | e23123a59f9c56c25684e25abdd18c4ef6ab7987 | |
| parent | 6d509f637e2ee98b1ed106018b69780a4dd3c255 (diff) | |
| download | qemu-f29c96d0f5760f72220623500ee659f1fc2d1542.zip qemu-f29c96d0f5760f72220623500ee659f1fc2d1542.tar.gz qemu-f29c96d0f5760f72220623500ee659f1fc2d1542.tar.bz2 | |
tests/functional: remove pointless with statement
The xorriso command directly writes to 'filename', so the surrounding
'with' statement is pointless.
Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Message-ID: <20241129173120.761728-5-berrange@redhat.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>
| -rwxr-xr-x | tests/functional/test_ppc64_hv.py | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/tests/functional/test_ppc64_hv.py b/tests/functional/test_ppc64_hv.py index 312248b..d97b62e 100755 --- a/tests/functional/test_ppc64_hv.py +++ b/tests/functional/test_ppc64_hv.py @@ -72,10 +72,9 @@ class HypervisorTest(QemuSystemTest): cwd = os.getcwd() os.chdir(self.workdir) - with open(filename, "w") as outfile: - cmd = "xorriso -osirrox on -indev %s -cpx %s %s" % (iso, path, filename) - subprocess.run(cmd.split(), - stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL) + cmd = "xorriso -osirrox on -indev %s -cpx %s %s" % (iso, path, filename) + subprocess.run(cmd.split(), + stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL) os.chmod(filename, 0o600) os.chdir(cwd) |