apply review changes

napari_animation/_enum_compat.py

@@ -1,10 +1,11 @@
 """Compatibility layer for enum behavior across Python versions.
 
-Python 3.13 introduced stricter enum handling requiring `enum.member()` wrapper for callable values.
+Python 3.13 introduced stricter enum handling requiring `enum.member()` wrapper for callable values. See https://docs.python.org/3/library/enum.html.
 This compatibility layer detects Python version and conditionally wraps enum values.
 - Python 3.10:
 - Python 3.11 added `member`
-- Python 3.13+: member is required.
+- Python 3.13+: EnumDict is added, and member is required. See
+  https://docs.python.org/3/whatsnew/3.13.html#enum
 
 Enum members have a name and a value.
 """
@@ -29,9 +30,7 @@ def wrap_enum_member(value):
     """Conditionally wrap a value with enum.member if needed."""
     if _NEEDS_ENUM_MEMBER:
         if not _HAS_ENUM_MEMBER:
-            raise RuntimeError(
-                f"Python {sys.version_info[:2]} requires enum.member but it's not available. "
-                "Please upgrade your Python installation."
-            )
+            # python 3.10 does not have member so we cannot wrap
+            return value
+        # Needed for Python 3.11+
         return member(value)
-    return value

napari_animation/_tests/test_enum_compat.py

@@ -110,26 +110,6 @@ def mock_member(value):
             assert wrapped == f"wrapped_{partial_func}"
 
 
-def test_wrap_enum_member_error_case():
-    """Test wrap_enum_member raises error when needed but not available."""
-    import pytest
-
-    def test_func(x):
-        return x * 2
-
-    partial_func = partial(test_func)
-
-    # Simulate Python 3.13+ without enum.member (broken environment)
-    with (
-        patch("napari_animation._enum_compat._NEEDS_ENUM_MEMBER", True),
-        patch("napari_animation._enum_compat._HAS_ENUM_MEMBER", False),
-    ):
-        with pytest.raises(
-            RuntimeError, match="requires enum.member but it's not available"
-        ):
-            wrap_enum_member(partial_func)
-
-
 def test_easing_enum_functionality():
     """Test that the Easing enum works correctly with the compatibility fix."""
     # Test that all easing functions are callable

tox.ini

@@ -1,6 +1,6 @@
 # For more information about tox, see https://tox.readthedocs.io/en/latest/
 [tox]
-envlist = py{310,311,312,313}-{linux,macos,windows}-pyqt5, py{310,311,312,313}-{linux,macos,windows}-pyqt6, py{310}-{linux,macos_x86_64,macos_arm64,windows}-pyside2, py{310}-{linux,macos_x86_64,macos_arm64,windows}-pyside6
+envlist = py{310,311,312,313}-{linux,macos,windows}-pyqt5, py{310,311,312,313}-{linux,macos,windows}-pyqt6, py{310}-{linux,macos,windows}-pyside2, py{310}-{linux,macos,windows}-pyside6
 
 [gh-actions]
 python =
@@ -11,16 +11,14 @@ python =
 
 [gh-actions:env]
 PLATFORM =
-    macos_arm64: darwin-arm64
-    macos_x86_64: darwin-x86_64
-    linux: linux
-    windows: win32
+    ubuntu-latest: linux
+    macos-latest: darwin
+    windows-latest: win32
 
 [testenv]
 platform = 
-    macos_arm64: darwin-arm64
-    macos_x86_64: darwin-x86_64
     linux: linux
+    macos: darwin
     windows: win32
 passenv = 
     CI
@@ -36,23 +34,24 @@ deps =
     pytest-xvfb ; sys_platform == 'linux'
 commands = pytest -v --color=yes --cov=napari_animation --cov-report=xml
 
-[testenv:py{310,311,312,313}-{linux,macos_x86_64,macos_arm64,windows}-pyqt5]
+[testenv:py{310,311,312,313}-{linux,macos,windows}-pyqt5]
 deps =
     {[testenv]deps}
     napari[PyQt5]
 
-[testenv:py{310,311,312,313}-{linux,macos_x86_64,macos_arm64,windows}-pyqt6]
+[testenv:py{310,311,312,313}-{linux,macos,windows}-pyqt6]
 deps =
     {[testenv]deps}
     napari[PyQt6]
 
 # PySide2 unavailable on Mac apple silicon arm64, and for python >=3.11
-[testenv:py{310}-{linux,macos_x86_64,windows}-pyside2]
+[testenv:py{310}-{linux,macos,windows}-pyside2]
 deps =
     {[testenv]deps}
     napari[pyside2]
+    pytest-qt < 4.5.0
 
-[testenv:py{310}-{linux,macos_x86_64,macos_arm64,windows}-pyside6]
+[testenv:py{310}-{linux,macos,windows}-pyside6]
 deps =
     {[testenv]deps}
     napari[PySide6]