mirror of
				https://github.com/python/cpython.git
				synced 2025-10-28 20:25:04 +00:00 
			
		
		
		
	Issue #19875: Fix random test_getsockaddrarg() failure.
This commit is contained in:
		
						commit
						ab9a446f23
					
				
					 1 changed files with 16 additions and 8 deletions
				
			
		|  | @ -3,6 +3,7 @@ | |||
| 
 | ||||
| import errno | ||||
| import io | ||||
| import itertools | ||||
| import socket | ||||
| import select | ||||
| import tempfile | ||||
|  | @ -1147,17 +1148,24 @@ def testNewAttributes(self): | |||
|         sock.close() | ||||
| 
 | ||||
|     def test_getsockaddrarg(self): | ||||
|         host = '0.0.0.0' | ||||
|         sock = socket.socket() | ||||
|         self.addCleanup(sock.close) | ||||
|         port = support.find_unused_port() | ||||
|         big_port = port + 65536 | ||||
|         neg_port = port - 65536 | ||||
|         sock = socket.socket() | ||||
|         try: | ||||
|             self.assertRaises(OverflowError, sock.bind, (host, big_port)) | ||||
|             self.assertRaises(OverflowError, sock.bind, (host, neg_port)) | ||||
|             sock.bind((host, port)) | ||||
|         finally: | ||||
|             sock.close() | ||||
|         self.assertRaises(OverflowError, sock.bind, (HOST, big_port)) | ||||
|         self.assertRaises(OverflowError, sock.bind, (HOST, neg_port)) | ||||
|         # Since find_unused_port() is inherently subject to race conditions, we | ||||
|         # call it a couple times if necessary. | ||||
|         for i in itertools.count(): | ||||
|             port = support.find_unused_port() | ||||
|             try: | ||||
|                 sock.bind((HOST, port)) | ||||
|             except OSError as e: | ||||
|                 if e.errno != errno.EADDRINUSE or i == 5: | ||||
|                     raise | ||||
|             else: | ||||
|                 break | ||||
| 
 | ||||
|     @unittest.skipUnless(os.name == "nt", "Windows specific") | ||||
|     def test_sock_ioctl(self): | ||||
|  |  | |||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue
	
	 Charles-François Natali
						Charles-François Natali