diff options
| author | Daniel P. Berrangé <berrange@redhat.com> | 2024-12-17 15:59:25 +0000 |
|---|---|---|
| committer | Thomas Huth <thuth@redhat.com> | 2024-12-17 19:39:53 +0100 |
| commit | 0da341a78f00d6feae98f38d1dfbe2e9f88d0b93 (patch) | |
| tree | 48591e7113f35c84d88e351dd7c646d280a90d8e /tests/functional/test_ppc64_hv.py | |
| parent | c27f452d61a4741605da2e03c0e6c756dd249f25 (diff) | |
| download | focaccia-qemu-0da341a78f00d6feae98f38d1dfbe2e9f88d0b93.tar.gz focaccia-qemu-0da341a78f00d6feae98f38d1dfbe2e9f88d0b93.zip | |
tests/functional: remove duplicated 'which' function impl
Put the 'which' function into shared code. Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Signed-off-by: Daniel P. Berrangé <berrange@redhat.com> Message-ID: <20241217155953.3950506-5-berrange@redhat.com> Signed-off-by: Thomas Huth <thuth@redhat.com>
Diffstat (limited to 'tests/functional/test_ppc64_hv.py')
| -rwxr-xr-x | tests/functional/test_ppc64_hv.py | 13 |
1 files changed, 1 insertions, 12 deletions
diff --git a/tests/functional/test_ppc64_hv.py b/tests/functional/test_ppc64_hv.py index d97b62e364..7c6f8234f5 100755 --- a/tests/functional/test_ppc64_hv.py +++ b/tests/functional/test_ppc64_hv.py @@ -11,7 +11,7 @@ from unittest import skipIf, skipUnless from qemu_test import QemuSystemTest, Asset -from qemu_test import wait_for_console_pattern, exec_command +from qemu_test import wait_for_console_pattern, exec_command, which import os import time import subprocess @@ -19,17 +19,6 @@ from datetime import datetime deps = ["xorriso"] # dependent tools needed in the test setup/box. -def which(tool): - """ looks up the full path for @tool, returns None if not found - or if @tool does not have executable permissions. - """ - paths=os.getenv('PATH') - for p in paths.split(os.path.pathsep): - p = os.path.join(p, tool) - if os.path.exists(p) and os.access(p, os.X_OK): - return p - return None - def missing_deps(): """ returns True if any of the test dependent tools are absent. """ |