This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch 8.5.x in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/8.5.x by this push: new e8ab3a2 Polish. Align with master. e8ab3a2 is described below commit e8ab3a28b2b176c85a696a02e50d741172bfc6b4 Author: Mark Thomas <ma...@apache.org> AuthorDate: Wed Jul 24 10:19:37 2019 +0100 Polish. Align with master. --- test/org/apache/catalina/startup/SimpleHttpClient.java | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/test/org/apache/catalina/startup/SimpleHttpClient.java b/test/org/apache/catalina/startup/SimpleHttpClient.java index 1d6e81e..1020985 100644 --- a/test/org/apache/catalina/startup/SimpleHttpClient.java +++ b/test/org/apache/catalina/startup/SimpleHttpClient.java @@ -34,6 +34,8 @@ import java.util.ArrayList; import java.util.List; import java.util.Locale; +import org.junit.Assert; + /** * Simple client for unit testing. It isn't robust, it isn't secure and * should not be used as the basis for production code. Its only purpose @@ -302,7 +304,8 @@ public abstract class SimpleHttpClient { if (wantBody) { if (useContentLength && (contentLength > -1)) { char[] body = new char[contentLength]; - reader.read(body); + int read = reader.read(body); + Assert.assertEquals(contentLength, read); builder.append(body); } else { --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org