From 6207e81d9c50ab73acc1c5df784d7693c6c2fb35 Mon Sep 17 00:00:00 2001 From: Abdessamad Derraz <3028866+Abdess@users.noreply.github.com> Date: Fri, 9 Oct 2026 22:45:58 +0200 Subject: [PATCH] refactor: read the grouping where it lives --- scripts/verify.py | 2 +- tests/test_region.py | 10 ++++++++++ 2 files changed, 11 insertions(+), 1 deletion(-) diff --git a/scripts/verify.py b/scripts/verify.py index 6b2ab01b..93c8fcec 100644 --- a/scripts/verify.py +++ b/scripts/verify.py @@ -901,7 +901,7 @@ def verify_platform( # pass reads. Grouping the platform files here and the core extras # there let the two answer differently on one request: the report kept # every core extra a region run withdraws from the pack. - from generate_pack import platform_region_groups + from packextras import platform_region_groups region_groups, region_extra_dests = platform_region_groups( config, diff --git a/tests/test_region.py b/tests/test_region.py index af95d7e8..87f33786 100644 --- a/tests/test_region.py +++ b/tests/test_region.py @@ -576,6 +576,16 @@ class TestReportAndBuilderNarrowTogether(unittest.TestCase): "or emulator_region_drops", ) + def test_the_verifier_reads_the_grouping_where_it_lives(self): + """Imported through the builder's CLI module, the verdict of a + --region run depended on that module compiling, and seven scripts + entered the site workflow's trigger paths for one function.""" + source = (Path(__file__).resolve().parent.parent / "scripts" / "verify.py").read_text( + encoding="utf-8" + ) + self.assertNotIn("from generate_pack import platform_region_groups", source) + self.assertIn("from packextras import platform_region_groups", source) + def test_emulator_report_keeps_what_the_emulator_pack_keeps(self): """O2EM: --region us left the Videopac BIOSes in the pack, not in the report.""" repo = Path(__file__).resolve().parent.parent