mirror of
				https://github.com/python/cpython.git
				synced 2025-10-31 13:41:24 +00:00 
			
		
		
		
	[3.9] bpo-39728: Enum: fix duplicate ValueError (GH-22277) (GH-22282)
				
					
				
			fix default `_missing_` to return `None` instead of raising a `ValueError`
Co-authored-by: Andrey Darascheka <andrei.daraschenka@leverx.com>
(cherry picked from commit c95ad7a91f)
Co-authored-by: Ethan Furman <ethan@stoneleaf.us>
			
			
This commit is contained in:
		
							parent
							
								
									4465df6266
								
							
						
					
					
						commit
						a9ba8ba9a7
					
				
					 4 changed files with 21 additions and 2 deletions
				
			
		|  | @ -1832,6 +1832,18 @@ class Dupes(Enum): | |||
|             third = auto() | ||||
|         self.assertEqual([Dupes.first, Dupes.second, Dupes.third], list(Dupes)) | ||||
| 
 | ||||
|     def test_default_missing(self): | ||||
|         class Color(Enum): | ||||
|             RED = 1 | ||||
|             GREEN = 2 | ||||
|             BLUE = 3 | ||||
|         try: | ||||
|             Color(7) | ||||
|         except ValueError as exc: | ||||
|             self.assertTrue(exc.__context__ is None) | ||||
|         else: | ||||
|             raise Exception('Exception not raised.') | ||||
| 
 | ||||
|     def test_missing(self): | ||||
|         class Color(Enum): | ||||
|             red = 1 | ||||
|  | @ -1850,7 +1862,12 @@ def _missing_(cls, item): | |||
|                     # trigger not found | ||||
|                     return None | ||||
|         self.assertIs(Color('three'), Color.blue) | ||||
|         self.assertRaises(ValueError, Color, 7) | ||||
|         try: | ||||
|             Color(7) | ||||
|         except ValueError as exc: | ||||
|             self.assertTrue(exc.__context__ is None) | ||||
|         else: | ||||
|             raise Exception('Exception not raised.') | ||||
|         try: | ||||
|             Color('bad return') | ||||
|         except TypeError as exc: | ||||
|  |  | |||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue
	
	 Ethan Furman
						Ethan Furman