You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tapestry.apache.org by hl...@apache.org on 2011/09/23 19:06:11 UTC

svn commit: r1174886 - /tapestry/tapestry5/trunk/tapestry-test/src/main/java/org/apache/tapestry5/test/ErrorReporterImpl.java

Author: hlship
Date: Fri Sep 23 17:06:11 2011
New Revision: 1174886

URL: http://svn.apache.org/viewvc?rev=1174886&view=rev
Log:
TAP5-1423: Selenium test failures attempt to take a screenshot, which fails on most browsers besides firefox

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

Modified: tapestry/tapestry5/trunk/tapestry-test/src/main/java/org/apache/tapestry5/test/ErrorReporterImpl.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-test/src/main/java/org/apache/tapestry5/test/ErrorReporterImpl.java?rev=1174886&r1=1174885&r2=1174886&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-test/src/main/java/org/apache/tapestry5/test/ErrorReporterImpl.java (original)
+++ tapestry/tapestry5/trunk/tapestry-test/src/main/java/org/apache/tapestry5/test/ErrorReporterImpl.java Fri Sep 23 17:06:11 2011
@@ -110,10 +110,16 @@ public class ErrorReporterImpl implement
 
         System.err.println("Writing current page screenshot to: " + capture);
 
-        commandProcessor.doCommand("captureEntirePageScreenshot", new String[]
-                {capture.getAbsolutePath(), "background=white"});
+        try
+        {
+            commandProcessor.doCommand("captureEntirePageScreenshot", new String[]
+                    {capture.getAbsolutePath(), "background=white"});
 
-        outputPaths.add(capture);
+            outputPaths.add(capture);
+        } catch (Exception ex)
+        {
+            System.err.println(ex.getMessage());
+        }
     }
 
 }