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 2023/02/17 14:59:35 UTC

[tomcat] branch 9.0.x updated: Fix failing test. RequestFacade has been updated to throw ISE not NPE

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 7f4fa39a67 Fix failing test. RequestFacade has been updated to throw ISE not NPE
7f4fa39a67 is described below

commit 7f4fa39a675fc1a43334efbaf79d8e7a12580637
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Fri Feb 17 14:58:36 2023 +0000

    Fix failing test. RequestFacade has been updated to throw ISE not NPE
---
 test/org/apache/catalina/core/TestAsyncContextImpl.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test/org/apache/catalina/core/TestAsyncContextImpl.java b/test/org/apache/catalina/core/TestAsyncContextImpl.java
index af65c88829..762371cad3 100644
--- a/test/org/apache/catalina/core/TestAsyncContextImpl.java
+++ b/test/org/apache/catalina/core/TestAsyncContextImpl.java
@@ -377,7 +377,7 @@ public class TestAsyncContextImpl extends TomcatBaseTest {
                                     // really checking here is that it does not
                                     // return true.
                                     result.append(req.isAsyncStarted());
-                                } catch (NullPointerException npe) {
+                                } catch (IllegalStateException ise) {
                                     result.append("false");
                                 }
                                 done = true;


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