Backport invalid value logging in ChoiceAttribute from #2624 to 8.0.x

sopel-irc · Oct 13, 2024 · 107e57c · 107e57c
1 parent 6ccd615
commit 107e57c
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/sopel/config/types.py b/sopel/config/types.py
@@ -649,7 +649,10 @@ def parse(self, value):
         if value in self.choices:
             return value
         else:
-            raise ValueError('Value must be in {}'.format(self.choices))
+            raise ValueError(
+                '{!r} is not one of the valid choices: {}'
+                .format(value, ', '.join(self.choices))
+            )
 
     def serialize(self, value):
         """Make sure ``value`` is valid and safe to write in the config file.
@@ -662,7 +665,10 @@ def serialize(self, value):
         if value in self.choices:
             return value
         else:
-            raise ValueError('Value must be in {}'.format(self.choices))
+            raise ValueError(
+                '{!r} is not one of the valid choices: {}'
+                .format(value, ', '.join(self.choices))
+            )
 
 
 class FilenameAttribute(BaseValidated):