[PULL,05/38] tests/functional: simplify 'which' implementation

Commit Message

Thomas Huth Dec. 18, 2024, 11:09 a.m. UTC

From: Daniel P. Berrangé <[e-mail]>

The 'access' check implies the file exists.

Reviewed-by: Thomas Huth <[e-mail]>
Reviewed-by: Richard Henderson <[e-mail]>
Signed-off-by: Daniel P. Berrangé <[e-mail]>
Message-ID: <[e-mail]>
Signed-off-by: Thomas Huth <[e-mail]>
---
 tests/functional/qemu_test/cmd.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Patch

diff --git a/tests/functional/qemu_test/cmd.py b/tests/functional/qemu_test/cmd.py
index 4106f1ee7c..600e0509db 100644
--- a/tests/functional/qemu_test/cmd.py
+++ b/tests/functional/qemu_test/cmd.py
@@ -25,7 +25,7 @@  def which(tool):
     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):
+        if os.access(p, os.X_OK):
             return p
     return None