diff options
author | jaseg <git@jaseg.de> | 2023-07-04 16:20:11 +0200 |
---|---|---|
committer | jaseg <git@jaseg.de> | 2023-07-04 16:20:11 +0200 |
commit | 8f7b2893dc9b6bc1b0c7d6f6476e98c8c27a948c (patch) | |
tree | c9927e1f36c363a63e6479218b5dc5d7cd2d3156 /gerbonara/tests | |
parent | e696c09eac8f997fc82bbe281e9b6c76dc02308a (diff) | |
download | gerbonara-8f7b2893dc9b6bc1b0c7d6f6476e98c8c27a948c.tar.gz gerbonara-8f7b2893dc9b6bc1b0c7d6f6476e98c8c27a948c.tar.bz2 gerbonara-8f7b2893dc9b6bc1b0c7d6f6476e98c8c27a948c.zip |
Fix failing symbol tests
Diffstat (limited to 'gerbonara/tests')
-rw-r--r-- | gerbonara/tests/conftest.py | 2 | ||||
-rw-r--r-- | gerbonara/tests/test_kicad_symbols.py | 12 |
2 files changed, 12 insertions, 2 deletions
diff --git a/gerbonara/tests/conftest.py b/gerbonara/tests/conftest.py index 4eba5e3..ea16217 100644 --- a/gerbonara/tests/conftest.py +++ b/gerbonara/tests/conftest.py @@ -58,7 +58,7 @@ def pytest_generate_tests(metafunc): lib_dir = Path(lib_dir).expanduser() if not lib_dir.is_dir(): raise ValueError(f'Path "{lib_dir}" given by KICAD_FOOTPRINTS environment variable does not exist or is not a directory.') - mod_files = list(lib_dir.glob('**/*.kicad_mod')) + mod_files = list(lib_dir.glob('*.pretty/*.kicad_mod')) else: raise ValueError('Either --kicad-footprint-files command line parameter or KICAD_FOOTPRINTS environment variable must be given.') metafunc.parametrize('kicad_mod_file', mod_files, ids=list(map(str, mod_files))) diff --git a/gerbonara/tests/test_kicad_symbols.py b/gerbonara/tests/test_kicad_symbols.py index 0a6c595..f7ed148 100644 --- a/gerbonara/tests/test_kicad_symbols.py +++ b/gerbonara/tests/test_kicad_symbols.py @@ -6,12 +6,14 @@ from ..cad.kicad.sexp import build_sexp from ..cad.kicad.sexp_mapper import sexp from ..cad.kicad.symbols import Library +from .utils import tmpfile + def test_parse(kicad_library_file): Library.open(kicad_library_file) -def test_round_trip(kicad_library_file): +def test_round_trip(kicad_library_file, tmpfile): print('========== Stage 1 load ==========') orig_lib = Library.open(kicad_library_file) print('========== Stage 1 save ==========') @@ -31,7 +33,11 @@ def test_round_trip(kicad_library_file): original = re.sub(r'\) \)', '))', original) original = re.sub(r'\) \)', '))', original) original = re.sub(r'\) \)', '))', original) + tmpfile('Processed original', '.kicad_sym').write_text(original) + stage1 = re.sub(r'\(', '\n(', re.sub(r'\s+', ' ', stage1_sexp)) + tmpfile('Processed stage 1 output', '.kicad_sym').write_text(stage1) + for original, stage1 in zip_longest(original.splitlines(), stage1.splitlines()): if original.startswith('(version'): continue @@ -54,6 +60,10 @@ def test_round_trip(kicad_library_file): # There is some disagreement as to whether rotation angles are ints or floats, and the spec doesn't say. return + if 'hide' in original or 'hide' in stage1: + # KiCad changed the position of the hide token inside text effects between versions. + return + assert original == stage1 |