From c6681c0a631863f49d947d321df2c94995a76f7f Mon Sep 17 00:00:00 2001
From: Abdessamad Derraz <3028866+Abdess@users.noreply.github.com>
Date: Tue, 6 Oct 2026 09:48:15 +0200
Subject: [PATCH] fix: compute coverage from the profiles the site reads

---
 scripts/generate_readme.py      | 10 +++++++++-
 scripts/generate_site.py        |  3 ++-
 tests/test_contributing_text.py | 21 +++++++++++++++++++++
 3 files changed, 32 insertions(+), 2 deletions(-)

diff --git a/scripts/generate_readme.py b/scripts/generate_readme.py
index 2cf5d127..db360d29 100644
--- a/scripts/generate_readme.py
+++ b/scripts/generate_readme.py
@@ -15,6 +15,7 @@ import json
 import os
 import sys
 from datetime import datetime, timezone
+from pathlib import Path
 
 sys.path.insert(0, os.path.dirname(__file__))
 from common import (
@@ -33,17 +34,24 @@ import release_record
 from verify import verify_platform
 
 
+# The repository's profiles wherever the script is run from: a bare
+# "emulators" read the working directory, and a run elsewhere counted 0 cores.
+_EMULATORS_DIR = str(Path(__file__).resolve().parent.parent / "emulators")
+
+
 def compute_coverage(
     platform_name: str,
     platforms_dir: str,
     db: dict,
     data_registry: dict | None = None,
     supplemental_names: set[str] | None = None,
+    emulators_dir: str = _EMULATORS_DIR,
 ) -> dict:
     config = load_platform_config(platform_name, platforms_dir)
     result = verify_platform(
         config,
         db,
+        emulators_dir,
         data_dir_registry=data_registry,
         supplemental_names=supplemental_names,
     )
@@ -59,7 +67,7 @@ def compute_coverage(
     core_missing = len(undeclared) - core_present
     # Files a profile marks unsourceable never reach the undeclared list, so
     # the gap they represent has to be counted back in.
-    profiles = load_emulator_profiles("emulators")
+    profiles = load_emulator_profiles(emulators_dir)
     unsourceable_names = {
         f.get("name", "")
         for emu in resolve_platform_cores(config, profiles)
diff --git a/scripts/generate_site.py b/scripts/generate_site.py
index 0daf4f5e..ae9da32c 100644
--- a/scripts/generate_site.py
+++ b/scripts/generate_site.py
@@ -3587,7 +3587,8 @@ def main():
     for name in sorted(platform_names):
         try:
             cov = compute_coverage(
-                name, args.platforms_dir, db, data_registry, suppl_names
+                name, args.platforms_dir, db, data_registry, suppl_names,
+                emulators_dir=args.emulators_dir,
             )
             coverages[name] = cov
             print(
diff --git a/tests/test_contributing_text.py b/tests/test_contributing_text.py
index c64846d5..84a345c0 100644
--- a/tests/test_contributing_text.py
+++ b/tests/test_contributing_text.py
@@ -31,5 +31,26 @@ class OneContributingText(unittest.TestCase):
         )
 
 
+
+class CoverageReadsTheGivenProfiles(unittest.TestCase):
+    """generate_site --emulators-dir changed the emulator pages but not the
+    platform coverage, which read ./emulators whatever was given."""
+
+    def test_the_directory_reaches_verify(self):
+        from unittest import mock  # noqa: PLC0415
+
+        seen: list[str] = []
+
+        def fake_verify(config, db, emulators_dir, **_kwargs):
+            seen.append(emulators_dir)
+            return {"status_counts": {}, "total_files": 0, "undeclared_files": [], "details": []}
+
+        with mock.patch.object(generate_readme, "verify_platform", fake_verify), \
+                mock.patch.object(generate_readme, "load_emulator_profiles", return_value={}):
+            generate_readme.compute_coverage(
+                "retroarch", str(REPO_ROOT / "platforms"), {"files": {}}, emulators_dir="elsewhere"
+            )
+        self.assertEqual(seen, ["elsewhere"])
+
 if __name__ == "__main__":
     unittest.main()