From b873c6a9fbe2515d8b46d4e20e333af9eb9be49c Mon Sep 17 00:00:00 2001 From: Abdessamad Derraz <3028866+Abdess@users.noreply.github.com> Date: Fri, 6 Nov 2026 02:20:14 +0100 Subject: [PATCH] fix: skip the report excerpt without the assets The excerpt quotes verify over the whole collection. The PR workflow restores none of the release assets, so the run there printed other counts and failed on every push since the engine batch. The test now names how many assets are absent and skips. --- tests/test_doc_pages.py | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/tests/test_doc_pages.py b/tests/test_doc_pages.py index bb99c68cb..3054134d4 100644 --- a/tests/test_doc_pages.py +++ b/tests/test_doc_pages.py @@ -1783,6 +1783,15 @@ class ReportExcerptIsTheRun(unittest.TestCase): def test_every_quoted_line_is_printed(self): import subprocess # noqa: PLC0415 + from restore_large_files import gitignored_paths # noqa: PLC0415 + + # The excerpt is a run over the whole collection. A checkout without + # the release assets (the PR workflow restores none) prints other + # counts, which says nothing about the page. + absent = [p for p in gitignored_paths(str(ROOT / ".gitignore")) + if not any(ROOT.glob(p))] + if absent: + self.skipTest(f"{len(absent)} release assets absent from this checkout, e.g. {absent[0]}") page = (WIKI / "reading-reports.md").read_text(encoding="utf-8") intro = "From `python scripts/verify.py --platform retrodeck`:" start = page.index("```\n", page.index(intro)) + 4