diff options
author | jaseg <git@jaseg.de> | 2024-11-06 14:49:36 +0100 |
---|---|---|
committer | jaseg <git@jaseg.de> | 2024-11-06 14:49:36 +0100 |
commit | 1a854b1812400867117584f6bbde7bd1c75bb118 (patch) | |
tree | 15618edcd42340cf40ccc80bba0ae8dc3e4a058a /gerbonara/tests | |
parent | 091ee84910bdd6e30988ee7c9061b6d474f588db (diff) | |
download | gerbonara-1a854b1812400867117584f6bbde7bd1c75bb118.tar.gz gerbonara-1a854b1812400867117584f6bbde7bd1c75bb118.tar.bz2 gerbonara-1a854b1812400867117584f6bbde7bd1c75bb118.zip |
Tests: Make kicad docker image configurable
Diffstat (limited to 'gerbonara/tests')
-rw-r--r-- | gerbonara/tests/conftest.py | 4 | ||||
-rw-r--r-- | gerbonara/tests/image_support.py | 14 |
2 files changed, 13 insertions, 5 deletions
diff --git a/gerbonara/tests/conftest.py b/gerbonara/tests/conftest.py index 242ec04..40cf756 100644 --- a/gerbonara/tests/conftest.py +++ b/gerbonara/tests/conftest.py @@ -8,7 +8,7 @@ from itertools import chain import pytest -from .image_support import ImageDifference, run_cargo_cmd, bulk_populate_kicad_fp_export_cache +from .image_support import ImageDifference, run_cargo_cmd, bulk_populate_kicad_fp_export_cache, KICAD_CONTAINER def pytest_assertrepr_compare(op, left, right): if isinstance(left, ImageDifference) or isinstance(right, ImageDifference): @@ -56,7 +56,7 @@ def pytest_configure(config): raise ValueError(f'Path "{lib_dir}" given by KICAD_FOOTPRINTS environment variable does not exist or is not a directory.') print('Updating podman image') - subprocess.run(['podman', 'pull', 'registry.hub.docker.com/kicad/kicad:nightly'], check=True) + subprocess.run(['podman', 'pull', KICAD_CONTAINER], check=True) print('Checking and bulk re-building KiCad footprint library cache') with multiprocessing.pool.ThreadPool() as pool: # use thread pool here since we're only monitoring podman processes diff --git a/gerbonara/tests/image_support.py b/gerbonara/tests/image_support.py index efb962a..5f6db04 100644 --- a/gerbonara/tests/image_support.py +++ b/gerbonara/tests/image_support.py @@ -40,6 +40,8 @@ from PIL import Image cachedir = Path(__file__).parent / 'image_cache' cachedir.mkdir(exist_ok=True) +KICAD_CONTAINER = os.environ.get('KICAD_CONTAINER', 'registry.hub.docker.com/kicad/kicad:nightly') + @total_ordering class ImageDifference: def __init__(self, value, histogram): @@ -169,7 +171,7 @@ def kicad_fp_export(mod_file, out_svg): '--userns=keep-id', # To allow container to read from bind mount '--mount', f'type=bind,src={pretty_dir},dst=/{pretty_dir.name}', '--mount', f'type=bind,src={tmpdir},dst=/out', - 'registry.hub.docker.com/kicad/kicad:nightly', + KICAD_CONTAINER, 'kicad-cli', 'fp', 'export', 'svg', '--output', '/out', '--footprint', fp_name, f'/{pretty_dir.name}'] subprocess.run(cmd, check=True) #, stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL) out_file = Path(tmpdir) / f'{fp_name}.svg' @@ -199,9 +201,15 @@ def bulk_populate_kicad_fp_export_cache(pretty_dir): '--userns=keep-id', # To allow container to read from bind mount '--mount', f'type=bind,src={pretty_dir},dst=/{pretty_dir.name}', '--mount', f'type=bind,src={tmpdir},dst=/out', - 'registry.hub.docker.com/kicad/kicad:nightly', + KICAD_CONTAINER, 'kicad-cli', 'fp', 'export', 'svg', '--output', '/out', f'/{pretty_dir.name}'] - subprocess.run(cmd, check=True, stdout=subprocess.DEVNULL) + + try: + subprocess.run(cmd, check=True, stdout=subprocess.DEVNULL) + + except subprocess.CalledProcessError as e: + print('Error running command with command line:', ' '.join(e.cmd), file=sys.stderr) + raise e for fn in mod_files: out_file = Path(tmpdir) / fn.with_suffix('.svg').name |