Issue #3916 - Fix whitespace between boundary and part headers
+ Updating existing testcase Signed-off-by: Joakim Erdfelt <joakim.erdfelt@gmail.com>
This commit is contained in:
parent
36b42ca65d
commit
784fabf2ae
|
@ -150,12 +150,28 @@ public class DefaultServletRangesTest
|
||||||
String boundary = body.substring(0, body.indexOf("\r\n"));
|
String boundary = body.substring(0, body.indexOf("\r\n"));
|
||||||
assertResponseContains("206 Partial", response);
|
assertResponseContains("206 Partial", response);
|
||||||
assertResponseContains("Content-Type: multipart/byteranges; boundary=", response);
|
assertResponseContains("Content-Type: multipart/byteranges; boundary=", response);
|
||||||
assertResponseContains("Content-Range: bytes 0-9/80", response);
|
|
||||||
assertResponseContains("Content-Range: bytes 20-29/80", response);
|
String section1 = boundary + "\r\n" +
|
||||||
assertResponseContains("Content-Range: bytes 40-49/80", response);
|
"Content-Type: text/plain\r\n" +
|
||||||
assertResponseContains(DATA.substring(0, 10), response);
|
"Content-Range: bytes 0-9/80\r\n" +
|
||||||
assertResponseContains(DATA.substring(20, 30), response);
|
"\r\n" +
|
||||||
assertResponseContains(DATA.substring(40, 50), response);
|
DATA.substring(0, 10) + "\r\n";
|
||||||
|
assertResponseContains(section1, response);
|
||||||
|
|
||||||
|
String section2 = boundary + "\r\n" +
|
||||||
|
"Content-Type: text/plain\r\n" +
|
||||||
|
"Content-Range: bytes 20-29/80\r\n" +
|
||||||
|
"\r\n" +
|
||||||
|
DATA.substring(20, 30) + "\r\n";
|
||||||
|
assertResponseContains(section2, response);
|
||||||
|
|
||||||
|
String section3 = boundary + "\r\n" +
|
||||||
|
"Content-Type: text/plain\r\n" +
|
||||||
|
"Content-Range: bytes 40-49/80\r\n" +
|
||||||
|
"\r\n" +
|
||||||
|
DATA.substring(40, 50) + "\r\n";
|
||||||
|
assertResponseContains(section3, response);
|
||||||
|
|
||||||
assertTrue(body.endsWith(boundary + "--\r\n"));
|
assertTrue(body.endsWith(boundary + "--\r\n"));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue