You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@shindig.apache.org by ch...@apache.org on 2010/09/29 07:25:14 UTC

svn commit: r1002473 - /shindig/trunk/java/gadgets/src/test/java/org/apache/shindig/gadgets/servlet/ServletUtilTest.java

Author: chirag
Date: Wed Sep 29 05:25:13 2010
New Revision: 1002473

URL: http://svn.apache.org/viewvc?rev=1002473&view=rev
Log:
SHINDIG-1435 | Fix unit test failure in ServletUtilTest#testCopyResponseToServlet
Set the FakeTimeSource before constructing the HttpResonse to make the resulting max-age predictable.

Modified:
    shindig/trunk/java/gadgets/src/test/java/org/apache/shindig/gadgets/servlet/ServletUtilTest.java

Modified: shindig/trunk/java/gadgets/src/test/java/org/apache/shindig/gadgets/servlet/ServletUtilTest.java
URL: http://svn.apache.org/viewvc/shindig/trunk/java/gadgets/src/test/java/org/apache/shindig/gadgets/servlet/ServletUtilTest.java?rev=1002473&r1=1002472&r2=1002473&view=diff
==============================================================================
--- shindig/trunk/java/gadgets/src/test/java/org/apache/shindig/gadgets/servlet/ServletUtilTest.java (original)
+++ shindig/trunk/java/gadgets/src/test/java/org/apache/shindig/gadgets/servlet/ServletUtilTest.java Wed Sep 29 05:25:13 2010
@@ -210,6 +210,9 @@ public class ServletUtilTest {
   
   @Test
   public void testCopyResponseToServlet() throws Exception {
+    FakeTimeSource fakeTime = new FakeTimeSource();
+    HttpUtil.setTimeSource(fakeTime);
+
     HttpResponse response = new HttpResponseBuilder()
         .setResponseString("response string").setHttpStatusCode(200).addHeader("h1", "v1")
         .addHeader("h2", "v2").setCacheTtl(1000).create();