]> git.ipfire.org Git - thirdparty/Python/cpython.git/commitdiff
bpo-41682: fixed flaky test test_sendfile_close_peer_in_the_middle_of_receiving ...
authorMiss Islington (bot) <31488909+miss-islington@users.noreply.github.com>
Tue, 25 Jan 2022 17:40:34 +0000 (09:40 -0800)
committerGitHub <noreply@github.com>
Tue, 25 Jan 2022 17:40:34 +0000 (18:40 +0100)
(cherry picked from commit 1c705fda8f9902906edd26d46acb0433b0b098e1)

Co-authored-by: Kumar Aditya <59607654+kumaraditya303@users.noreply.github.com>
Co-authored-by: Kumar Aditya <59607654+kumaraditya303@users.noreply.github.com>
Lib/test/test_asyncio/test_sendfile.py

index 0ba966cb5ccf486a6222208849f5bebbcdb887d9..8e34ab49383901e19191f39232b430effee26619 100644 (file)
@@ -87,9 +87,13 @@ class MyProto(asyncio.Protocol):
 
 class SendfileBase:
 
-      # 128 KiB plus small unaligned to buffer chunk
-    DATA = b"SendfileBaseData" * (1024 * 8 + 1)
-
+    # 256 KiB plus small unaligned to buffer chunk
+    # Newer versions of Windows seems to have increased its internal 
+    # buffer and tries to send as much of the data as it can as it 
+    # has some form of buffering for this which is less than 256KiB
+    # on newer server versions and Windows 11.
+    # So DATA should be larger than 256 KiB to make this test reliable.
+    DATA = b"x" * (1024 * 256 + 1)
     # Reduce socket buffer size to test on relative small data sets.
     BUF_SIZE = 4 * 1024   # 4 KiB
 
@@ -451,8 +455,6 @@ class SendfileMixin(SendfileBase):
     # themselves).
     @unittest.skipIf(sys.platform.startswith('sunos'),
                      "Doesn't work on Solaris")
-    @unittest.skipIf(sys.platform == "win32",
-                     "It is flaky on Windows and needs to be fixed")  # TODO: bpo-41682
     def test_sendfile_close_peer_in_the_middle_of_receiving(self):
         srv_proto, cli_proto = self.prepare_sendfile(close_after=1024)
         with self.assertRaises(ConnectionError):