Solved a potential deadlock in test_telnetlib.py. Related to issue #11812

This commit is contained in:
Jesus Cea 2011-11-08 15:54:42 +01:00
parent 53703d9d72
commit 71094b5d5b
2 changed files with 2 additions and 8 deletions

View File

@ -15,7 +15,6 @@ def server(evt, serv, dataq=None):
1) set evt to true to let the parent know we are ready
2) [optional] if is not False, write the list of data from dataq.get()
to the socket.
3) set evt to true to let the parent know we're done
"""
serv.listen(5)
evt.set()
@ -40,7 +39,6 @@ def server(evt, serv, dataq=None):
conn.close()
finally:
serv.close()
evt.set()
class GeneralTests(TestCase):
@ -52,11 +50,8 @@ class GeneralTests(TestCase):
self.thread = threading.Thread(target=server, args=(self.evt,self.sock))
self.thread.start()
self.evt.wait()
self.evt.clear()
time.sleep(.1)
def tearDown(self):
self.evt.wait()
self.thread.join()
def testBasic(self):
@ -105,11 +100,8 @@ def _read_setUp(self):
self.thread = threading.Thread(target=server, args=(self.evt,self.sock, self.dataq))
self.thread.start()
self.evt.wait()
self.evt.clear()
time.sleep(.1)
def _read_tearDown(self):
self.evt.wait()
self.thread.join()
class ReadTests(TestCase):

View File

@ -359,6 +359,8 @@ Tests
- Skip network tests when getaddrinfo() returns EAI_AGAIN, meaning a temporary
failure in name resolution.
- Solved a potential deadlock in test_telnetlib.py. Related to issue #11812.
- Avoid failing in test_robotparser when mueblesmoraleda.com is flaky and
an overzealous DNS service (e.g. OpenDNS) redirects to a placeholder
Web site.