(tomcat) branch 9.0.x updated: Fix test on Windows.

[markt]

This is an automated email from the ASF dual-hosted git repository.

markt pushed a commit to branch 9.0.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git


The following commit(s) were added to refs/heads/9.0.x by this push:
     new fc67ca91a6 Fix test on Windows.
fc67ca91a6 is described below

commit fc67ca91a6dd6371e2cecbdb52a24c81724d7721
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Thu Dec 5 14:37:51 2024 +0000

    Fix test on Windows.
---
 test/org/apache/catalina/valves/TestJDBCAccessLogValve.java | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/test/org/apache/catalina/valves/TestJDBCAccessLogValve.java 
b/test/org/apache/catalina/valves/TestJDBCAccessLogValve.java
index f693ae15bb..9a5ea08848 100644
--- a/test/org/apache/catalina/valves/TestJDBCAccessLogValve.java
+++ b/test/org/apache/catalina/valves/TestJDBCAccessLogValve.java
@@ -33,6 +33,7 @@ import org.junit.runners.Parameterized;
 import org.junit.runners.Parameterized.Parameter;
 
 import org.apache.catalina.Context;
+import org.apache.catalina.startup.SimpleHttpClient;
 import org.apache.catalina.startup.Tomcat;
 import org.apache.catalina.startup.TomcatBaseTest;
 import org.apache.tomcat.util.buf.ByteChunk;
@@ -134,7 +135,11 @@ public class TestJDBCAccessLogValve extends TomcatBaseTest 
{
                 Assert.assertTrue(rs.next());
                 Assert.assertEquals(HttpServletResponse.SC_OK, 
rs.getInt("status"));
                 Assert.assertEquals("/test/index.html", rs.getString("query"));
-                Assert.assertTrue(rs.getLong("bytes") == 934);
+                if (System.lineSeparator().equals(SimpleHttpClient.CRLF)) {
+                    Assert.assertEquals(957, rs.getLong("bytes"));
+                } else {
+                    Assert.assertEquals(934, rs.getLong("bytes"));
+                }
                 Assert.assertTrue(rs.next());
                 Assert.assertEquals(HttpServletResponse.SC_OK, 
rs.getInt("status"));
                 Assert.assertEquals("/test/404.html", rs.getString("query"));


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org