[jira] Created: (CONTINUUM-2268) Continuum Webapp Tests Capture Screenshot for AbstractSeleniumTest.java

Continuum Webapp Tests Capture Screenshot for AbstractSeleniumTest.java
-----------------------------------------------------------------------

                 Key: CONTINUUM-2268
                 URL: http://jira.codehaus.org/browse/CONTINUUM-2268
             Project: Continuum
          Issue Type: Test
          Components: Testing
         Environment: Linux
            Reporter: Bernard Lago
         Attachments: ContinuumTestCaptureAssertionError.patch

The attached patch includes the following changes:
Changed continuum-webapp-tests/pom.xml selenium version from 0.9.2 to 1.0.1
Added a captureEntirePageScreenshot to AbstractSeleniumTest.java
Added a catch block for every failed assertion to do a screen capture and save them to target/screenshots directory
Formatted the filename of the captured file as "methodName(className.java:lineNumber)-time.png"

Additional Info:

Maven version: 2.0.10
Java version: 1.5.0_16
OS name: "linux" version: "2.6.27-14-generic" arch: "i386" Family: "unix"

Patch created at revision 785116.

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




--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

     [ http://jira.codehaus.org/browse/CONTINUUM-2268?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Bernard Lago updated CONTINUUM-2268:
------------------------------------

    Attachment: CaptureScreenshotForAbstractSeleniumTest.patch

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] ------------------------------------------------------------------------

> Continuum Webapp Tests Capture Screenshot for AbstractSeleniumTest.java
> -----------------------------------------------------------------------
>
>                 Key: CONTINUUM-2268
>                 URL: http://jira.codehaus.org/browse/CONTINUUM-2268
>             Project: Continuum
>          Issue Type: Test
>          Components: Testing
>         Environment: Linux
>            Reporter: Bernard Lago
>         Attachments: CaptureScreenshotForAbstractSeleniumTest.patch, ContinuumTestCaptureAssertionError.patch
>
>
> The attached patch includes the following changes:
> Changed continuum-webapp-tests/pom.xml selenium version from 0.9.2 to 1.0.1
> Added a captureEntirePageScreenshot to AbstractSeleniumTest.java
> Added a catch block for every failed assertion to do a screen capture and save them to target/screenshots directory
> Formatted the filename of the captured file as "methodName(className.java:lineNumber)-time.png"
> Additional Info:
> Maven version: 2.0.10
> Java version: 1.5.0_16
> OS name: "linux" version: "2.6.27-14-generic" arch: "i386" Family: "unix"
> Patch created at revision 785116.
> Build Results:
> Results :
> Tests run: 167, Failures: 0, Errors: 0, Skipped: 0

--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

    [ http://jira.codehaus.org/browse/CONTINUUM-2268?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=180744#action_180744 ]

Wendy Smoak commented on CONTINUUM-2268:
----------------------------------------

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.

> Continuum Webapp Tests Capture Screenshot for AbstractSeleniumTest.java
> -----------------------------------------------------------------------
>
>                 Key: CONTINUUM-2268
>                 URL: http://jira.codehaus.org/browse/CONTINUUM-2268
>             Project: Continuum
>          Issue Type: Test
>          Components: Testing
>         Environment: Linux
>            Reporter: Bernard Lago
>         Attachments: CaptureScreenshotForAbstractSeleniumTest.patch, ContinuumTestCaptureAssertionError.patch
>
>
> The attached patch includes the following changes:
> Changed continuum-webapp-tests/pom.xml selenium version from 0.9.2 to 1.0.1
> Added a captureEntirePageScreenshot to AbstractSeleniumTest.java
> Added a catch block for every failed assertion to do a screen capture and save them to target/screenshots directory
> Formatted the filename of the captured file as "methodName(className.java:lineNumber)-time.png"
> Additional Info:
> Maven version: 2.0.10
> Java version: 1.5.0_16
> OS name: "linux" version: "2.6.27-14-generic" arch: "i386" Family: "unix"
> Patch created at revision 785116.
> Build Results:
> Results :
> Tests run: 167, Failures: 0, Errors: 0, Skipped: 0

--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

    [ http://jira.codehaus.org/browse/CONTINUUM-2268?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=181014#action_181014 ]

Jose Morales Martinez commented on CONTINUUM-2268:
--------------------------------------------------

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.

> Continuum Webapp Tests Capture Screenshot for AbstractSeleniumTest.java
> -----------------------------------------------------------------------
>
>                 Key: CONTINUUM-2268
>                 URL: http://jira.codehaus.org/browse/CONTINUUM-2268
>             Project: Continuum
>          Issue Type: Test
>          Components: Testing
>         Environment: Linux
>            Reporter: Bernard Lago
>         Attachments: CaptureScreenshotForAbstractSeleniumTest.patch, ContinuumTestCaptureAssertionError.patch
>
>
> The attached patch includes the following changes:
> Changed continuum-webapp-tests/pom.xml selenium version from 0.9.2 to 1.0.1
> Added a captureEntirePageScreenshot to AbstractSeleniumTest.java
> Added a catch block for every failed assertion to do a screen capture and save them to target/screenshots directory
> Formatted the filename of the captured file as "methodName(className.java:lineNumber)-time.png"
> Additional Info:
> Maven version: 2.0.10
> Java version: 1.5.0_16
> OS name: "linux" version: "2.6.27-14-generic" arch: "i386" Family: "unix"
> Patch created at revision 785116.
> Build Results:
> Results :
> Tests run: 167, Failures: 0, Errors: 0, Skipped: 0

--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

    [ http://jira.codehaus.org/browse/CONTINUUM-2268?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=181025#action_181025 ]

Jose Morales Martinez commented on CONTINUUM-2268:
--------------------------------------------------

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

> Continuum Webapp Tests Capture Screenshot for AbstractSeleniumTest.java
> -----------------------------------------------------------------------
>
>                 Key: CONTINUUM-2268
>                 URL: http://jira.codehaus.org/browse/CONTINUUM-2268
>             Project: Continuum
>          Issue Type: Test
>          Components: Testing
>         Environment: Linux
>            Reporter: Bernard Lago
>         Attachments: CaptureScreenshotForAbstractSeleniumTest.patch, ContinuumTestCaptureAssertionError.patch
>
>
> The attached patch includes the following changes:
> Changed continuum-webapp-tests/pom.xml selenium version from 0.9.2 to 1.0.1
> Added a captureEntirePageScreenshot to AbstractSeleniumTest.java
> Added a catch block for every failed assertion to do a screen capture and save them to target/screenshots directory
> Formatted the filename of the captured file as "methodName(className.java:lineNumber)-time.png"
> Additional Info:
> Maven version: 2.0.10
> Java version: 1.5.0_16
> OS name: "linux" version: "2.6.27-14-generic" arch: "i386" Family: "unix"
> Patch created at revision 785116.
> Build Results:
> Results :
> Tests run: 167, Failures: 0, Errors: 0, Skipped: 0

--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

    [ http://jira.codehaus.org/browse/CONTINUUM-2268?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=181046#action_181046 ]

Bernard Lago commented on CONTINUUM-2268:
-----------------------------------------

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. :)

> Continuum Webapp Tests Capture Screenshot for AbstractSeleniumTest.java
> -----------------------------------------------------------------------
>
>                 Key: CONTINUUM-2268
>                 URL: http://jira.codehaus.org/browse/CONTINUUM-2268
>             Project: Continuum
>          Issue Type: Test
>          Components: Testing
>         Environment: Linux
>            Reporter: Bernard Lago
>         Attachments: CaptureScreenshotForAbstractSeleniumTest.patch, ContinuumTestCaptureAssertionError.patch
>
>
> The attached patch includes the following changes:
> Changed continuum-webapp-tests/pom.xml selenium version from 0.9.2 to 1.0.1
> Added a captureEntirePageScreenshot to AbstractSeleniumTest.java
> Added a catch block for every failed assertion to do a screen capture and save them to target/screenshots directory
> Formatted the filename of the captured file as "methodName(className.java:lineNumber)-time.png"
> Additional Info:
> Maven version: 2.0.10
> Java version: 1.5.0_16
> OS name: "linux" version: "2.6.27-14-generic" arch: "i386" Family: "unix"
> Patch created at revision 785116.
> Build Results:
> Results :
> Tests run: 167, Failures: 0, Errors: 0, Skipped: 0

--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

    [ http://jira.codehaus.org/browse/CONTINUUM-2268?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=181182#action_181182 ]

Bernard Lago edited comment on CONTINUUM-2268 at 6/23/09 12:39 AM:
-------------------------------------------------------------------

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




      was (Author: blago):
    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



 

> Continuum Webapp Tests Capture Screenshot for AbstractSeleniumTest.java
> -----------------------------------------------------------------------
>
>                 Key: CONTINUUM-2268
>                 URL: http://jira.codehaus.org/browse/CONTINUUM-2268
>             Project: Continuum
>          Issue Type: Test
>          Components: Testing
>         Environment: Linux
>            Reporter: Bernard Lago
>         Attachments: CaptureScreenshotForAbstractSeleniumTest.patch, ContinuumTestCaptureAssertionError.patch
>
>
> The attached patch includes the following changes:
> Changed continuum-webapp-tests/pom.xml selenium version from 0.9.2 to 1.0.1
> Added a captureEntirePageScreenshot to AbstractSeleniumTest.java
> Added a catch block for every failed assertion to do a screen capture and save them to target/screenshots directory
> Formatted the filename of the captured file as "methodName(className.java:lineNumber)-time.png"
> Additional Info:
> Maven version: 2.0.10
> Java version: 1.5.0_16
> OS name: "linux" version: "2.6.27-14-generic" arch: "i386" Family: "unix"
> Patch created at revision 785116.
> Build Results:
> Results :
> Tests run: 167, Failures: 0, Errors: 0, Skipped: 0

--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

    [ http://jira.codehaus.org/browse/CONTINUUM-2268?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=181182#action_181182 ]

Bernard Lago commented on CONTINUUM-2268:
-----------------------------------------

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

> Continuum Webapp Tests Capture Screenshot for AbstractSeleniumTest.java
> -----------------------------------------------------------------------
>
>                 Key: CONTINUUM-2268
>                 URL: http://jira.codehaus.org/browse/CONTINUUM-2268
>             Project: Continuum
>          Issue Type: Test
>          Components: Testing
>         Environment: Linux
>            Reporter: Bernard Lago
>         Attachments: CaptureScreenshotForAbstractSeleniumTest.patch, ContinuumTestCaptureAssertionError.patch
>
>
> The attached patch includes the following changes:
> Changed continuum-webapp-tests/pom.xml selenium version from 0.9.2 to 1.0.1
> Added a captureEntirePageScreenshot to AbstractSeleniumTest.java
> Added a catch block for every failed assertion to do a screen capture and save them to target/screenshots directory
> Formatted the filename of the captured file as "methodName(className.java:lineNumber)-time.png"
> Additional Info:
> Maven version: 2.0.10
> Java version: 1.5.0_16
> OS name: "linux" version: "2.6.27-14-generic" arch: "i386" Family: "unix"
> Patch created at revision 785116.
> Build Results:
> Results :
> Tests run: 167, Failures: 0, Errors: 0, Skipped: 0

--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

     [ http://jira.codehaus.org/browse/CONTINUUM-2268?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jose Morales Martinez closed CONTINUUM-2268.
--------------------------------------------

         Assignee: Jose Morales Martinez
       Resolution: Fixed
    Fix Version/s: 1.4.0

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.

> Continuum Webapp Tests Capture Screenshot for AbstractSeleniumTest.java
> -----------------------------------------------------------------------
>
>                 Key: CONTINUUM-2268
>                 URL: http://jira.codehaus.org/browse/CONTINUUM-2268
>             Project: Continuum
>          Issue Type: Test
>          Components: Testing
>         Environment: Linux
>            Reporter: Bernard Lago
>            Assignee: Jose Morales Martinez
>             Fix For: 1.4.0
>
>         Attachments: CaptureScreenshotForAbstractSeleniumTest.patch, ContinuumTestCaptureAssertionError.patch
>
>
> The attached patch includes the following changes:
> Changed continuum-webapp-tests/pom.xml selenium version from 0.9.2 to 1.0.1
> Added a captureEntirePageScreenshot to AbstractSeleniumTest.java
> Added a catch block for every failed assertion to do a screen capture and save them to target/screenshots directory
> Formatted the filename of the captured file as "methodName(className.java:lineNumber)-time.png"
> Additional Info:
> Maven version: 2.0.10
> Java version: 1.5.0_16
> OS name: "linux" version: "2.6.27-14-generic" arch: "i386" Family: "unix"
> Patch created at revision 785116.
> Build Results:
> Results :
> Tests run: 167, Failures: 0, Errors: 0, Skipped: 0

--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira