diff options
| author | Thomas Huth <thuth@redhat.com> | 2025-01-22 14:43:13 +0100 |
|---|---|---|
| committer | Thomas Huth <thuth@redhat.com> | 2025-01-30 10:30:36 +0100 |
| commit | aca2c48e4d26426c14499fd3d8a2e333ac57d267 (patch) | |
| tree | f74b99dcf492319aa1e7d36c4bf2169c8fd0a9cd | |
| parent | 871af84dd599fab68c8ed414d9ecbdb2bcfc5801 (diff) | |
| download | focaccia-qemu-aca2c48e4d26426c14499fd3d8a2e333ac57d267.tar.gz focaccia-qemu-aca2c48e4d26426c14499fd3d8a2e333ac57d267.zip | |
tests/functional/qemu_test/decorators: Fix bad check for imports
skipIfMissingImports should use importlib.import_module() for checking whether a module with the name stored in the "impname" variable is available or not, otherwise the code tries to import a module with the name "impname" instead. (This bug hasn't been noticed before since there is another issue with this decorator that will be fixed by the next patch) Suggested-by: Daniel P. Berrangé <berrange@redhat.com> Reviewed-by: Daniel P. Berrangé <berrange@redhat.com> Message-ID: <20250122134315.1448794-2-thuth@redhat.com> Signed-off-by: Thomas Huth <thuth@redhat.com>
| -rw-r--r-- | tests/functional/qemu_test/decorators.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/tests/functional/qemu_test/decorators.py b/tests/functional/qemu_test/decorators.py index df088bc090..08f58f6b40 100644 --- a/tests/functional/qemu_test/decorators.py +++ b/tests/functional/qemu_test/decorators.py @@ -2,6 +2,7 @@ # # Decorators useful in functional tests +import importlib import os import platform from unittest import skipUnless @@ -97,7 +98,7 @@ def skipIfMissingImports(*args): def has_imports(importlist): for impname in importlist: try: - import impname + importlib.import_module(impname) except ImportError: return False return True |