Improve naming of guards for enum member

napari_animation/_enum_compat.py

@@ -17,19 +17,20 @@
     from enum import member
 
     # member is importable (Python 3.11+)
-    _HAS_ENUM_MEMBER = True
+    _ENUM_MEMBER_AVAILABLE = True
 except ImportError:
     # Python 3.10 has no member support
-    _HAS_ENUM_MEMBER = False
+    _ENUM_MEMBER_AVAILABLE = False
 
 # Check if enum.member is required (Python 3.13+)
-_NEEDS_ENUM_MEMBER = sys.version_info >= (3, 13)
+_ENUM_MEMBER_REQUIRED = sys.version_info >= (3, 13)
 
 
 def wrap_enum_member(value):
     """Conditionally wrap a value with enum.member if needed."""
-    if _NEEDS_ENUM_MEMBER:
-        # Needed for Python 3.11+
+    # Enum member is required for Python 3.13+, and available for 3.11+
+    if _ENUM_MEMBER_REQUIRED or _ENUM_MEMBER_AVAILABLE:
         return member(value)
-    # python 3.10 common case
-    return value
+    # For python 3.10 and older, just return the value directly
+    if not _ENUM_MEMBER_AVAILABLE:
+        return value

napari_animation/_tests/test_enum_compat.py

@@ -4,8 +4,8 @@
 from functools import partial
 
 from napari_animation._enum_compat import (
-    _HAS_ENUM_MEMBER,
-    _NEEDS_ENUM_MEMBER,
+    _ENUM_MEMBER_AVAILABLE,
+    _ENUM_MEMBER_REQUIRED,
     wrap_enum_member,
 )
 from napari_animation.easing import Easing
@@ -17,14 +17,14 @@
 def test_enum_member_detection():
     """Test that enum.member availability is correctly detected."""
     if sys.version_info >= (3, 11):
-        assert _HAS_ENUM_MEMBER is True
+        assert _ENUM_MEMBER_AVAILABLE is True
     else:
-        assert _HAS_ENUM_MEMBER is False
+        assert _ENUM_MEMBER_AVAILABLE is False
 
     if sys.version_info >= (3, 13):
-        assert _NEEDS_ENUM_MEMBER is True
+        assert _ENUM_MEMBER_REQUIRED is True
     else:
-        assert _NEEDS_ENUM_MEMBER is False
+        assert _ENUM_MEMBER_REQUIRED is False
 
 
 def test_wrap_enum_member_basic():