mirror of
				https://github.com/python/cpython.git
				synced 2025-10-31 21:51:50 +00:00 
			
		
		
		
	bpo-37100: Fix test_coroutines with -Werror (GH-13756)
test_coroutines: test_unawaited_warning_when_module_broken() now uses support.check_warnings() to catch expected RuntimeWarning.
This commit is contained in:
		
							parent
							
								
									7f4ae1b2cc
								
							
						
					
					
						commit
						3cf7ea1272
					
				
					 1 changed files with 4 additions and 4 deletions
				
			
		|  | @ -2250,7 +2250,8 @@ async def corofn(): | |||
|         try: | ||||
|             warnings._warn_unawaited_coroutine = lambda coro: 1/0 | ||||
|             with support.catch_unraisable_exception() as cm, \ | ||||
|                  support.captured_stderr() as stream: | ||||
|                  support.check_warnings((r'coroutine .* was never awaited', | ||||
|                                          RuntimeWarning)): | ||||
|                 # only store repr() to avoid keeping the coroutine alive | ||||
|                 coro = corofn() | ||||
|                 coro_repr = repr(coro) | ||||
|  | @ -2261,13 +2262,12 @@ async def corofn(): | |||
| 
 | ||||
|                 self.assertEqual(repr(cm.unraisable.object), coro_repr) | ||||
|                 self.assertEqual(cm.unraisable.exc_type, ZeroDivisionError) | ||||
|                 self.assertIn("was never awaited", stream.getvalue()) | ||||
| 
 | ||||
|             del warnings._warn_unawaited_coroutine | ||||
|             with support.captured_stderr() as stream: | ||||
|             with support.check_warnings((r'coroutine .* was never awaited', | ||||
|                                          RuntimeWarning)): | ||||
|                 corofn() | ||||
|                 support.gc_collect() | ||||
|             self.assertIn("was never awaited", stream.getvalue()) | ||||
| 
 | ||||
|         finally: | ||||
|             warnings._warn_unawaited_coroutine = orig_wuc | ||||
|  |  | |||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue
	
	 Victor Stinner
						Victor Stinner