You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@shindig.apache.org by li...@apache.org on 2010/08/09 13:56:27 UTC

svn commit: r983598 - /shindig/trunk/java/gadgets/src/test/java/org/apache/shindig/gadgets/servlet/MakeRequestHandlerTest.java

Author: lindner
Date: Mon Aug  9 11:56:27 2010
New Revision: 983598

URL: http://svn.apache.org/viewvc?rev=983598&view=rev
Log:
fix non-deterministic test failure

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

Modified: shindig/trunk/java/gadgets/src/test/java/org/apache/shindig/gadgets/servlet/MakeRequestHandlerTest.java
URL: http://svn.apache.org/viewvc/shindig/trunk/java/gadgets/src/test/java/org/apache/shindig/gadgets/servlet/MakeRequestHandlerTest.java?rev=983598&r1=983597&r2=983598&view=diff
==============================================================================
--- shindig/trunk/java/gadgets/src/test/java/org/apache/shindig/gadgets/servlet/MakeRequestHandlerTest.java (original)
+++ shindig/trunk/java/gadgets/src/test/java/org/apache/shindig/gadgets/servlet/MakeRequestHandlerTest.java Mon Aug  9 11:56:27 2010
@@ -544,8 +544,9 @@ public class MakeRequestHandlerTest exte
     expect(request.getParameter(Param.REFRESH.getKey())).andReturn("30").anyTimes();
     replay();
 
+    // not sure why but the following line seems to help this test past deterministically 
+    System.out.println("request started at " + HttpUtilTest.testStartTime);
     MakeRequestHandler.setResponseHeaders(request, recorder, results);
-
     HttpUtilTest.checkCacheControlHeaders(HttpUtilTest.testStartTime, recorder, 30, false);
     assertEquals("attachment;filename=p.txt", recorder.getHeader("Content-Disposition"));
   }