Merge pull request #3465 from Spinnich/fix/screenscraper-new-3ds-platform-map

fix(ss): map New Nintendo 3DS to ScreenScraper system 17
This commit is contained in:
Georges-Antoine Assi
2026-06-03 11:17:44 -04:00
committed by GitHub
2 changed files with 13 additions and 0 deletions

View File

@@ -997,6 +997,7 @@ SCREENSAVER_PLATFORM_LIST: dict[UPS, SlugToSSId] = {
"name": "Neo-Geo Pocket Color",
},
UPS.N3DS: {"id": 17, "name": "Nintendo 3DS"},
UPS.NEW_NINTENDON3DS: {"id": 17, "name": "Nintendo 3DS"},
UPS.N64: {"id": 14, "name": "Nintendo 64"},
UPS.N64DD: {"id": 122, "name": "Nintendo 64DD"},
UPS.NDS: {"id": 15, "name": "Nintendo DS"},

View File

@@ -667,6 +667,18 @@ class TestAddSsAuthToUrl:
assert download_query.get("systemeid") == ["1"]
class TestGetPlatform:
"""Tests for SSHandler.get_platform — the slug → ScreenScraper system map."""
def test_unmapped_platform_returns_none_ss_id(self):
"""A slug with no ScreenScraper mapping yields ss_id=None (lookup skipped)."""
handler = SSHandler()
platform = handler.get_platform("not-a-real-platform")
assert platform["ss_id"] is None
assert platform["slug"] == "not-a-real-platform"
class TestGetRomType:
def _file(self, ext: str, top_level: bool = True) -> MagicMock:
f = MagicMock()