You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tapestry.apache.org by hl...@apache.org on 2010/07/12 20:47:35 UTC

svn commit: r963428 - /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/test/PageTester.java

Author: hlship
Date: Mon Jul 12 18:47:35 2010
New Revision: 963428

URL: http://svn.apache.org/viewvc?rev=963428&view=rev
Log:
TAP5-1197: Ensure that PageTester cleans up the thread after each simulated request, to ensure no bleed-through of per-thread data from one request to the next

Modified:
    tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/test/PageTester.java

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/test/PageTester.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/test/PageTester.java?rev=963428&r1=963427&r2=963428&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/test/PageTester.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/test/PageTester.java Mon Jul 12 18:47:35 2010
@@ -144,6 +144,8 @@ public class PageTester
      */
     public void shutdown()
     {
+        registry.cleanupThread();
+        
         registry.shutdown();
     }
 
@@ -210,7 +212,10 @@ public class PageTester
             {
                 throw new RuntimeException(ex);
             }
-
+            finally
+            {
+                registry.cleanupThread();
+            }
         }
 
     }
@@ -290,7 +295,10 @@ public class PageTester
             {
                 throw new RuntimeException(ex);
             }
-
+            finally
+            {
+                registry.cleanupThread();
+            }
         }
 
     }