Author martin.panter
Recipients SilentGhost, kristjan.jonsson, martin.panter, maubp, serhiy.storchaka
Date 2016-03-12.00:27:18
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1457742438.79.0.0724906853247.issue26499@psf.upfronthosting.co.za>
In-reply-to
Content
To add a second response you would just concatenate it to the existing response. (Your existing test already uses FakeSocket.) The FakeSocket parameter just represents data that would be sent from the server. So:

body = (
   b"HTTP/1.1 200 OK\r\n"
   b"Content-Length: 3\r\n"
   b"\r\n"
   b"abc"
   b"HTTP/1.1 408 Next response should not be read yet\r\n"
)

In fact, see the BasicTest.test_content_length_sync() case, which literally has "extradata" as that last line. I think we just need to adapt or duplicate this test to cover readline() and read1(), not just read(). Maybe also with read1(100), readline(100) cases as well.
History
Date User Action Args
2016-03-12 00:27:18martin.pantersetrecipients: + martin.panter, kristjan.jonsson, SilentGhost, maubp, serhiy.storchaka
2016-03-12 00:27:18martin.pantersetmessageid: <1457742438.79.0.0724906853247.issue26499@psf.upfronthosting.co.za>
2016-03-12 00:27:18martin.panterlinkissue26499 messages
2016-03-12 00:27:18martin.pantercreate