Make platform and group IDs resolution case-insensitive

Author: kdeldycke

changelog.md

@@ -5,6 +5,7 @@
 > [!IMPORTANT]
 > This version is not released yet and is under active development.
 
+- Make resolution of platform and group IDs case-insensitive.
 - Drop supports for Python 3.10.
 
 ## [2.1.0 (2025-02-20)](https://github.com/kdeldycke/extra-platforms/compare/v2.0.0...v2.1.0)

extra_platforms/group.py

@@ -124,6 +124,11 @@ def _extract_platforms(other: _TNestedSources) -> Iterator[Platform]:
             yield other
         elif isinstance(other, Group):
             yield from other.platforms
+        elif isinstance(other, str):
+            # Prevent circular import.
+            from .operations import platforms_from_ids
+
+            yield platforms_from_ids(other)
         elif isinstance(other, Iterable):
             for item in flatten_iter(other):
                 yield from Group._extract_platforms(item)

extra_platforms/operations.py

@@ -48,7 +48,7 @@ def platforms_from_ids(*platform_ids: str) -> set[Platform]:
         If you want to reduce the returned set and removes as much overlaps as
         possible, you can use the ``extra_platforms.reduce()`` function on the results.
     """
-    ids = frozenset(platform_ids)
+    ids = frozenset((s.lower() for s in platform_ids))
     unrecognized_ids = ids - ALL_IDS
     if unrecognized_ids:
         raise ValueError(f"Unrecognized IDs: {', '.join(sorted(unrecognized_ids))}")
@@ -69,7 +69,7 @@ def groups_from_ids(*group_ids: str) -> set[Group]:
         If you want to reduce the returned set and removes as much overlaps as
         possible, you can use the ``extra_platforms.reduce()`` function on the results.
     """
-    ids = frozenset(group_ids)
+    ids = frozenset((s.lower() for s in group_ids))
     unrecognized_ids = ids - ALL_GROUP_IDS
     if unrecognized_ids:
         raise ValueError(

tests/test_operations.py

@@ -16,6 +16,9 @@
 
 from __future__ import annotations
 
+from random import choice
+from typing import Iterable
+
 import pytest
 
 from extra_platforms import (
@@ -108,32 +111,54 @@ def test_unique_ids():
     assert ALL_GROUP_IDS.isdisjoint(ALL_PLATFORM_IDS)
 
 
-def test_platforms_from_ids():
-    for platform_id in ALL_PLATFORM_IDS:
-        platforms = platforms_from_ids(platform_id)
-        assert platforms
-        assert len(platforms) == 1
-        platform = platforms.pop()
-        assert platform.id == platform_id
-        assert platform in ALL_PLATFORMS.platforms
-
-    for group_id in ALL_GROUP_IDS:
-        platforms = platforms_from_ids(group_id)
-        assert platforms
-        assert len(platforms) >= 1
-        groups = groups_from_ids(group_id)
-        assert len(groups) == 1
-        group = groups.pop()
-        assert platforms == set(group.platforms)
-
-
-def test_groups_from_ids():
-    for group_id in ALL_GROUP_IDS:
-        groups = groups_from_ids(group_id)
-        assert len(groups) == 1
-        group = groups.pop()
-        assert group.id == group_id
-        assert group in ALL_GROUPS
+def randomize_case(strings: Iterable[str]) -> set[str]:
+    test_strings = set()
+    for string in strings:
+        test_strings.add(string)
+        for str_func in (
+            str.upper,
+            str.lower,
+            str.title,
+            str.capitalize,
+            str.casefold,
+            str.swapcase,
+        ):
+            test_strings.add(str_func(string))
+        test_strings.add(
+            "".join(choice((str.upper, str.lower))(char) for char in string)
+        )
+    return test_strings
+
+
+@pytest.mark.parametrize("platform_id", randomize_case(ALL_PLATFORM_IDS))
+def test_platforms_from_ids(platform_id):
+    platforms = platforms_from_ids(platform_id)
+    assert platforms
+    assert len(platforms) == 1
+    platform = platforms.pop()
+    assert platform.id == platform_id.lower()
+    assert platform in ALL_PLATFORMS.platforms
+
+
+@pytest.mark.parametrize("group_id", randomize_case(ALL_GROUP_IDS))
+def test_platforms_from_ids_group_resolve(group_id):
+    """platforms_from_ids() can also resolve group IDs."""
+    platforms = platforms_from_ids(group_id)
+    assert platforms
+    assert len(platforms) >= 1
+    groups = groups_from_ids(group_id)
+    assert len(groups) == 1
+    group = groups.pop()
+    assert platforms == set(group.platforms)
+
+
+@pytest.mark.parametrize("group_id", randomize_case(ALL_GROUP_IDS))
+def test_groups_from_ids(group_id):
+    groups = groups_from_ids(group_id)
+    assert len(groups) == 1
+    group = groups.pop()
+    assert group.id == group_id.lower()
+    assert group in ALL_GROUPS
 
 
 @pytest.mark.parametrize(