summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--Lib/test/test_telnetlib.py8
-rw-r--r--Misc/NEWS2
2 files changed, 2 insertions, 8 deletions
diff --git a/Lib/test/test_telnetlib.py b/Lib/test/test_telnetlib.py
index e40c1fe..04cb7fe 100644
--- a/Lib/test/test_telnetlib.py
+++ b/Lib/test/test_telnetlib.py
@@ -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):
diff --git a/Misc/NEWS b/Misc/NEWS
index 43e2763..9b1e5d1 100644
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -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.