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();
+ }
}
}