From: Guido van Rossum <guido@python.org>
Date: Thu, 8 Aug 2002 20:28:34 +0000 (+0000)
Subject: testSendAll(): loop until all data is read; this was necessary at
X-Git-Tag: v2.3c1~4608
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=e531e296fa178a3a8fc8ec708035e5255f327227;p=python

testSendAll(): loop until all data is read; this was necessary at
least on OS/2 (see note on SF patch 555085 by A I MacIntyre) but
looks like the test *could* fail on any other platform too -- there's
no guarantee that recv() reads all data.
---

diff --git a/Lib/test/test_socket.py b/Lib/test/test_socket.py
index 69113fdebb..3583d41b09 100644
--- a/Lib/test/test_socket.py
+++ b/Lib/test/test_socket.py
@@ -378,13 +378,13 @@ class BasicTCPTest(SocketConnectedTest):
 
     def testSendAll(self):
         # Testing sendall() with a 2048 byte string over TCP
+        msg = ''
         while 1:
             read = self.cli_conn.recv(1024)
             if not read:
                 break
-            self.assert_(len(read) == 1024, "Error performing sendall.")
-            read = filter(lambda x: x == 'f', read)
-            self.assert_(len(read) == 1024, "Error performing sendall.")
+            msg += read
+        self.assertEqual(msg, 'f' * 2048)
 
     def _testSendAll(self):
         big_chunk = 'f' * 2048