From 54e8e05b9de8cd957812c5f0c669947af0c448ba Mon Sep 17 00:00:00 2001 From: Benjamin Kreuscher Date: Thu, 20 Aug 2026 13:27:16 +0200 Subject: [PATCH 1/2] fix: use full -hdd flag in hdd mode test to avoid ambiguity with -h Adding -hdd (PR #280) made the short form -hd ambiguous with -h, causing argparse to reject it. Use the full flag name in the test. Co-Authored-By: Claude Sonnet 4.6 --- tests/test_generated_files.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/test_generated_files.py b/tests/test_generated_files.py index 92c8b9e4..bd314205 100644 --- a/tests/test_generated_files.py +++ b/tests/test_generated_files.py @@ -218,7 +218,7 @@ def run_wahoomapscreator_cli(self, country, hdd_mode=False): else: # run processing of input-country via CLI in mapwriter hdd mode result = os.system( - f'python -m wahoomc cli -co {country} -tag tag-wahoo.xml -fp -c -md 9999 -nbc -hd') + f'python -m wahoomc cli -co {country} -tag tag-wahoo.xml -fp -c -md 9999 -nbc -hdd') # check if run was successful self.assertEqual(result, 0) From ba980878b8fb47675da1c66e2bcd678cfb904b9e Mon Sep 17 00:00:00 2001 From: Benjamin Kreuscher Date: Thu, 20 Aug 2026 14:25:36 +0200 Subject: [PATCH 2/2] test: update Germany border country expected data for Czech sub-regions Geofabrik split czech-republic into regional sub-divisions. Replace the single 'czech-republic' entry with the 6 sub-regions that border Germany: jihocesky, karlovarsky, liberecky, plzensky, stredocesky, ustecky. Co-Authored-By: Claude Sonnet 4.6 --- tests/test_osm_maps.py | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/tests/test_osm_maps.py b/tests/test_osm_maps.py index 9eb8db04..18efceb8 100644 --- a/tests/test_osm_maps.py +++ b/tests/test_osm_maps.py @@ -44,7 +44,9 @@ def test_calc_border_countries_input_country(self): 'austria': {}, 'liechtenstein': {}, 'schwaben': {}, 'mecklenburg-vorpommern': {}, 'brandenburg': {}, 'sachsen-anhalt': {}, 'thueringen': {}, 'oberfranken': {}, 'mittelfranken': {}, 'oberbayern': {}, 'oberpfalz': {}, 'sachsen': {}, - 'czech-republic': {}, 'niederbayern': {}, 'nord-est': {}, 'sweden': {}, + 'jihocesky': {}, 'karlovarsky': {}, 'liberecky': {}, 'plzensky': {}, + 'stredocesky': {}, 'ustecky': {}, + 'niederbayern': {}, 'nord-est': {}, 'sweden': {}, 'zachodniopomorskie': {}, 'berlin': {}, 'lubuskie': {}, 'dolnoslaskie': {}} self.process_and_check_border_countries( 'germany', True, expected_result, 'country') @@ -61,7 +63,9 @@ def test_calc_border_countries_input_country(self): 'austria': {}, 'liechtenstein': {}, 'schwaben': {}, 'mecklenburg-vorpommern': {}, 'brandenburg': {}, 'sachsen-anhalt': {}, 'thueringen': {}, 'oberfranken': {}, 'mittelfranken': {}, 'oberbayern': {}, 'oberpfalz': {}, 'sachsen': {}, - 'czech-republic': {}, 'niederbayern': {}, 'nord-est': {}, 'sweden': {}, + 'jihocesky': {}, 'karlovarsky': {}, 'liberecky': {}, 'plzensky': {}, + 'stredocesky': {}, 'ustecky': {}, + 'niederbayern': {}, 'nord-est': {}, 'sweden': {}, 'zachodniopomorskie': {}, 'berlin': {}, 'lubuskie': {}, 'dolnoslaskie': {}, 'isole' : {}, 'malta' : {} } self.process_and_check_border_countries(