Enum: Replace ValueError to AttributeError.

Signed-off-by: Ihor Nehrutsa <Ihor.Nehrutsa@gmail.com>

Author: IhorNehrutsa

python-stdlib/enum/enum.md

@@ -113,5 +113,4 @@ The base class for all enumerations.
     * Raised when attempting to modify an `EnumValue`.
     * Raised when attempting to add new members to an initialized Enum.
     * Raised when a class-level lookup (`Status(999)`) fails.
-* **`ValueError`**:
     * Raised when an instance-level lookup (`s(999)`) fails.

python-stdlib/enum/enum.py

@@ -65,7 +65,7 @@ def _lookup(cls, value):
                 # Wrap static numbers found in class definition
                 return EnumValue(attr, key)
 
-        raise AttributeError(f"{value} is not in {cls.__name__} enum")
+        raise AttributeError(f"{value} is not in {cls.__name__}")
 
     def list_members(self):
         # Returns a list of tuples (name, value) for all members
@@ -103,7 +103,7 @@ def __call__(self, value):
         for member in self:
             if member.value == value:
                 return member
-        raise ValueError(f"no such value: {value}")
+        raise AttributeError(f"{value} is not in {self.__class__.__name__}")
 
     def __setattr__(self, key, value):
         if hasattr(self, '_initialized'):
@@ -139,7 +139,7 @@ class Color(Enum):
 
     # Create instance
     c = Color()
-    print(f"Enum repr c: {c}")
+    print(f"Enum c: {c}")
 
     # Basic access
     print(f"RED: Name={c.RED.name}, Value={c.RED.value}, EnumValue={c.RED}, Call={c.RED()} ")
@@ -162,8 +162,8 @@ class Color(Enum):
 
     try:
         c(999)
-    except ValueError as e:
-        print(f"\nValueError: {c} {e}\n")
+    except AttributeError as e:
+        print(f"\nAttributeError: {e}: {c}\n")
 
     # --- Usage Example 2 ---
     # Define an Enum class

python-stdlib/enum/test_enum.py

@@ -38,7 +38,7 @@ def test_call_reverse_lookup(self):
         self.assertEqual(result.value, 1)
 
         # Перевірка виключення для неіснуючого значення
-        with self.assertRaises(ValueError):
+        with self.assertRaises(AttributeError):
             self.color(999)
 
     def test_is_value(self):

python-stdlib/enum/test_enum_.py

@@ -4,15 +4,15 @@
 import unittest
 from enum import Enum, EnumValue
 
-class TestEnum(unittest.TestCase):
 
+class TestEnum(unittest.TestCase):
     def setUp(self):
         # Define standard Enum classes for testing
         class Color(Enum):
             RED = 'red'
             GREEN = 'green'
             BLUE = 'blue'
-        
+
         class Status(Enum):
             IDLE = 0
             RUNNING = 1
@@ -42,7 +42,7 @@ def test_immutability(self):
         # Test EnumValue immutability
         with self.assertRaises(AttributeError):
             self.color_inst.RED.value = 'blue'
-        
+
         # Test Enum instance immutability (once initialized)
         with self.assertRaises(AttributeError):
             self.color_inst.NEW_COLOR = 'yellow'
@@ -63,24 +63,25 @@ def test_reverse_lookup_via_instance_call(self):
         member = self.color_inst('green')
         self.assertEqual(member.name, 'GREEN')
 
-        with self.assertRaises(ValueError):
+        with self.assertRaises(AttributeError):
             self.color_inst('yellow')
 
     def test_iteration(self):
         """Test that the Enum instance is iterable and returns all members."""
         members = list(self.status_inst)
         names = [m.name for m in members]
         values = [m.value for m in members]
-        
+
         self.assertEqual(len(members), 3)
         self.assertIn('IDLE', names)
         self.assertIn(2, values)
 
+
 #     def test_functional_api(self):
 #         """Test dynamic Enum creation using the Functional API."""
 #         # Dynamic creation via Enum(name, names_dict)
 #         State = Enum(name='State', names={'ON': 1, 'OFF': 0})
-#         
+#
 #         self.assertTrue(hasattr(State, 'ON'))
 #         self.assertEqual(State.ON.value, 1)
 #         self.assertEqual(State.OFF.name, 'OFF')
@@ -92,18 +93,18 @@ def test_iteration(self):
 #         print('c_repr', c_repr)
 #         c_restored = eval(c_repr)
 #         self.assertEqual(self.color_inst, c_restored)
-#         
+#
 #         # Test dynamically created instance
 #         s_dynamic = Enum(name='State', names={'ON': 1, 'OFF': 0})
 #         s_repr = repr(s_dynamic)
 #         s_restored = eval(s_repr)
 #         self.assertEqual(s_dynamic, s_restored)
 
-    def test_len_and_list_members(self):
-        """Test utility functions like __len__ and list_members."""
+    def test_len_and_list(self):
+        """Test utility functions like __len__ and list."""
         self.assertEqual(len(self.color_inst), 3)
-        members_list = self.color_inst.list_members()
-        self.assertEqual(members_list, [('BLUE', 'blue'), ('GREEN', 'green'), ('RED', 'red')])
+        members_list = self.color_inst.list()
+        self.assertEqual(members_list, [self.color_inst.BLUE, self.color_inst.GREEN, self.color_inst.RED])
 
     def test_deletion_prevention(self):
         """Verify that members cannot be deleted."""