You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tomee.apache.org by rm...@apache.org on 2016/07/04 12:20:22 UTC

tomee git commit: fixing FailingJspTest

Repository: tomee
Updated Branches:
  refs/heads/master 971d5c9bd -> 69edd5835


fixing FailingJspTest


Project: http://git-wip-us.apache.org/repos/asf/tomee/repo
Commit: http://git-wip-us.apache.org/repos/asf/tomee/commit/69edd583
Tree: http://git-wip-us.apache.org/repos/asf/tomee/tree/69edd583
Diff: http://git-wip-us.apache.org/repos/asf/tomee/diff/69edd583

Branch: refs/heads/master
Commit: 69edd5835b14c372abe7d70e7cf012748886f2fa
Parents: 971d5c9
Author: Romain manni-Bucau <rm...@gmail.com>
Authored: Mon Jul 4 14:20:05 2016 +0200
Committer: Romain manni-Bucau <rm...@gmail.com>
Committed: Mon Jul 4 14:20:05 2016 +0200

----------------------------------------------------------------------
 .../test/java/org/apache/tomee/embedded/FailingJspTest.java  | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tomee/blob/69edd583/tomee/tomee-embedded/src/test/java/org/apache/tomee/embedded/FailingJspTest.java
----------------------------------------------------------------------
diff --git a/tomee/tomee-embedded/src/test/java/org/apache/tomee/embedded/FailingJspTest.java b/tomee/tomee-embedded/src/test/java/org/apache/tomee/embedded/FailingJspTest.java
index 94cf0e9..d6f4805 100644
--- a/tomee/tomee-embedded/src/test/java/org/apache/tomee/embedded/FailingJspTest.java
+++ b/tomee/tomee-embedded/src/test/java/org/apache/tomee/embedded/FailingJspTest.java
@@ -66,8 +66,14 @@ public class FailingJspTest {
 
             // bug in org.apache.jasper.servlet.JspServletWrapper.destroy()
             tracked2.removeAll(tracked1);
-            assertEquals(String.valueOf(tracked2), 0, tracked2.size());
+            assertEquals(String.valueOf(tracked2), 1, tracked2.size());
         }
+
+        final Collection<Object> tracked2 = getTrackedContexts(ctx);
+
+        // bug in org.apache.jasper.servlet.JspServletWrapper.destroy()
+        tracked2.removeAll(tracked1);
+        assertEquals(String.valueOf(tracked2), 0, tracked2.size());
     }
 
     private Collection<Object> getTrackedContexts(final WebContext ctx) {