aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDaniel P. Berrangé <berrange@redhat.com>2024-12-17 15:59:27 +0000
committerThomas Huth <thuth@redhat.com>2024-12-17 19:39:53 +0100
commitc54edc71093bf85550245250d1f5922465848e83 (patch)
treeb74567c20ef6261a3b468810e90da958b8acb511
parentdd6dfc012c914d3e81be1da4b0ca7328949bdd08 (diff)
downloadqemu-c54edc71093bf85550245250d1f5922465848e83.zip
qemu-c54edc71093bf85550245250d1f5922465848e83.tar.gz
qemu-c54edc71093bf85550245250d1f5922465848e83.tar.bz2
tests/functional: drop 'tesseract_available' helper
Platforms we target have new enough tesseract that it suffices to merely check if the binary exists. Signed-off-by: Daniel P. Berrangé <berrange@redhat.com> Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Message-ID: <20241217155953.3950506-7-berrange@redhat.com> Signed-off-by: Thomas Huth <thuth@redhat.com>
-rw-r--r--tests/functional/qemu_test/tesseract.py12
-rwxr-xr-xtests/functional/test_m68k_nextcube.py8
2 files changed, 4 insertions, 16 deletions
diff --git a/tests/functional/qemu_test/tesseract.py b/tests/functional/qemu_test/tesseract.py
index ef18331..1b78180 100644
--- a/tests/functional/qemu_test/tesseract.py
+++ b/tests/functional/qemu_test/tesseract.py
@@ -7,17 +7,7 @@
import logging
-from . import has_cmd, run_cmd
-
-def tesseract_available(expected_version):
- (has_tesseract, _) = has_cmd('tesseract')
- if not has_tesseract:
- return False
- (stdout, stderr, ret) = run_cmd([ 'tesseract', '--version'])
- if ret:
- return False
- version = stdout.split()[1]
- return int(version.split('.')[0]) >= expected_version
+from . import run_cmd
def tesseract_ocr(image_path, tesseract_args=''):
console_logger = logging.getLogger('console')
diff --git a/tests/functional/test_m68k_nextcube.py b/tests/functional/test_m68k_nextcube.py
index 0124622..e6e8d4f 100755
--- a/tests/functional/test_m68k_nextcube.py
+++ b/tests/functional/test_m68k_nextcube.py
@@ -13,7 +13,8 @@ import time
from qemu_test import QemuSystemTest, Asset
from unittest import skipUnless
-from qemu_test.tesseract import tesseract_available, tesseract_ocr
+from qemu_test import has_cmd
+from qemu_test.tesseract import tesseract_ocr
PIL_AVAILABLE = True
try:
@@ -53,10 +54,7 @@ class NextCubeMachine(QemuSystemTest):
self.assertEqual(width, 1120)
self.assertEqual(height, 832)
- # Tesseract 4 adds a new OCR engine based on LSTM neural networks. The
- # new version is faster and more accurate than version 3. The drawback is
- # that it is still alpha-level software.
- @skipUnless(tesseract_available(4), 'tesseract OCR tool not available')
+ @skipUnless(*has_cmd('tesseract'))
def test_bootrom_framebuffer_ocr_with_tesseract(self):
self.set_machine('next-cube')
screenshot_path = os.path.join(self.workdir, "dump.ppm")