From 95b14ee282d0f44ee1e347fb1b0c771e89ebff23 Mon Sep 17 00:00:00 2001 From: "James R. Barlow" Date: Thu, 12 Oct 2023 16:30:16 -0700 Subject: [PATCH] Refactor lossless reconstruction setter into separate function Still messy but good enough as a start. --- src/ocrmypdf/_validation.py | 5 ++++- tests/test_validation.py | 4 +++- 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/src/ocrmypdf/_validation.py b/src/ocrmypdf/_validation.py index 3639ebca..7cf3ecce 100644 --- a/src/ocrmypdf/_validation.py +++ b/src/ocrmypdf/_validation.py @@ -99,6 +99,8 @@ def check_options_output(options: Namespace) -> None: f"`-` to suppress this message." ) + +def set_lossless_reconstruction(options: Namespace) -> None: lossless_reconstruction = False if not any( ( @@ -228,6 +230,7 @@ def _check_plugin_invariant_options(options: Namespace) -> None: check_platform() check_options_metadata(options) check_options_output(options) + set_lossless_reconstruction(options) check_options_sidecar(options) check_options_preprocessing(options) check_options_ocr_behavior(options) @@ -282,7 +285,7 @@ def create_input_file(options: Namespace, work_folder: Path) -> tuple[Path, str] msg = f"File not found - {options.input_file}" if _in_docker(): # pragma: no cover msg += ( - "\nDocker cannot your working directory unless you " + "\nDocker cannot access your working directory unless you " "explicitly share it with the Docker container and set up" "permissions correctly.\n" "You may find it easier to use stdin/stdout:" diff --git a/tests/test_validation.py b/tests/test_validation.py index f0e16541..c29d1637 100644 --- a/tests/test_validation.py +++ b/tests/test_validation.py @@ -71,7 +71,9 @@ def test_tesseract_not_installed(caplog): def test_lossless_redo(): with pytest.raises(BadArgsError): - vd.check_options_output(make_opts(redo_ocr=True, deskew=True)) + options = make_opts(redo_ocr=True, deskew=True) + vd.check_options_output(options) + vd.set_lossless_reconstruction(options) def test_mutex_options():