Continuum Webapp Tests Capture Screenshot for AbstractSeleniumTest.java

Please disregard the first attachment and use CaptureScreenshotForAbstractSeleniumTest.patch
Minor code changes were made to this new one.

Additional Info:
Patch created at revision 785468

Results :

Tests run: 167, Failures: 0, Errors: 0, Skipped: 0

[INFO] [cargo:stop {execution: stop-container}]
[INFO] [install:install]
[INFO] Installing /home/blago/Documents/apache/continuum/trunk/continuum-webapp-test/pom.xml to /home/blago/.m2/repository/org/apache/continuum/continuum-webapp-test/1.4.0-SNAPSHOT/continuum-webapp-test-1.4.0-SNAPSHOT.pom
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESSFUL
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 23 minutes 21 seconds
[INFO] Finished at: Wed Jun 17 12:05:44 GMT+08:00 2009
[INFO] Final Memory: 32M/113M
[INFO] ------------------------------------------------------------------------

Thanks, Bernard, this is really neat! Applied on trunk r785845

I'm leaving the issue open for documentation, src/site/apt/index.apt (or a another page if that gets too long) needs to talk about this.

It's good idea.

I am searching information about logs inTestNG and I think we can replace try/catch in AbstractSeleniumTest for TestNG listeners [1] and then we take screenshot. What do you think about it?

[1] http://testng.org/doc/documentation-main.html#logging-listeners.

I will change filename format because use ':' and this character is not allowed in filename for Windows.

Thanks for pointing out the use of ":"
I haven't tried this on a windows box. Please do the necessary changes. Thanks again!
Still reading about the TestNG Listeners. I will try it out and see if it will work.

Jose,

I tried the example from the link you posted about TestNG Listeners.
I created a class that subclasses TestListenerAdapter and created a method that will be called for every test failure. I also modified pom.xml's maven-surefire-plugin to use the new Listener class.
For the said method, I am having issues creating the selenium object that will call the captureEntirePageScreenshot method. If I create a new selenium object, then I will have to pass another set of parameters. (url, browser, etc) and it would be like calling the open() method of AbstractSeleniumTest class again.
It seems complicated to use this inside the Listener class. A possible workaround is to write a new capture screenshot method inside the Listener class and not use selenium's capture method. (btw, I prefer using selenium's capture method)
Maybe you can suggest how to implement this using Listeners.

Bernard

Fixed in 787880r.

I declare listener CaptureScreenShotsListener in pom and move capture process to this listener. I delete try/catch from selenium class.

I think this issue is finished.

Thank Bernard.