You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2013/04/25 16:21:27 UTC
svn commit: r1475791 -
/tomcat/trunk/test/org/apache/catalina/core/TestAsyncContextImpl.java
Author: markt
Date: Thu Apr 25 14:21:27 2013
New Revision: 1475791
URL: http://svn.apache.org/r1475791
Log:
Fix failing unit test now that a RuntimeException in an AsyncListener is caught and logged.
Modified:
tomcat/trunk/test/org/apache/catalina/core/TestAsyncContextImpl.java
Modified: tomcat/trunk/test/org/apache/catalina/core/TestAsyncContextImpl.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/core/TestAsyncContextImpl.java?rev=1475791&r1=1475790&r2=1475791&view=diff
==============================================================================
--- tomcat/trunk/test/org/apache/catalina/core/TestAsyncContextImpl.java (original)
+++ tomcat/trunk/test/org/apache/catalina/core/TestAsyncContextImpl.java Thu Apr 25 14:21:27 2013
@@ -460,15 +460,8 @@ public class TestAsyncContextImpl extend
expected.append("requestDestroyed");
} else if (completeOnTimeout.booleanValue()) {
expected.append("onTimeout-");
- if (dispatchUrl == null) {
- expected.append("onComplete-");
- expected.append("requestDestroyed");
- } else {
- // Error - no further output
- // There is no onComplete- since the complete event would be
- // fired during post processing but since there is an error that
- // never happens.
- }
+ expected.append("onComplete-");
+ expected.append("requestDestroyed");
} else {
expected.append("onTimeout-");
if (dispatchUrl != null) {
@@ -487,11 +480,6 @@ public class TestAsyncContextImpl extend
alv.validateAccessLog(1, 500, TimeoutServlet.ASYNC_TIMEOUT,
TimeoutServlet.ASYNC_TIMEOUT + TIMEOUT_MARGIN +
REQUEST_TIME);
- } else if (completeOnTimeout.booleanValue() && dispatchUrl != null) {
- // This error is written into Host-level AccessLogValve only
- alvGlobal.validateAccessLog(1, 500, 0, TimeoutServlet.ASYNC_TIMEOUT
- + TIMEOUT_MARGIN + REQUEST_TIME);
- alv.validateAccessLog(0, 500, 0, 0);
} else {
alvGlobal.validateAccessLog(1, 200, TimeoutServlet.ASYNC_TIMEOUT,
TimeoutServlet.ASYNC_TIMEOUT + TIMEOUT_MARGIN +
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org