Continuum
  1. Continuum
  2. CONTINUUM-2073

Revive the automated Selenium integration tests

    Details

    • Type: Test Test
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.3.1 (Alpha)
    • Fix Version/s: 1.4.0 (Beta)
    • Component/s: None
    • Labels:
      None
    • Complexity:
      Intermediate
    • Number of attachments :
      11

      Description

      Get the automated Selenium tests in the continuum-webapp-module running again.

      1. CONTINUUM-2073-src-test-it.patch
        52 kB
        Marecor M. Baclay
      2. CONTINUUM-2073-src-test-it-2.patch
        69 kB
        Marecor M. Baclay
      3. CONTINUUM-2073-src-test-it-3.patch
        69 kB
        Marecor M. Baclay
      4. CONTINUUM-2073-TestNG
        14 kB
        Jose Morales Martinez
      5. CONTINUUM-2073-TestNG-2.patch
        73 kB
        Jose Morales Martinez
      6. CONTINUUM-2073-TestNG-3.patch
        122 kB
        Jose Morales Martinez
      7. CONTINUUM-2073-TestNG-4.patch
        61 kB
        Jose Morales Martinez
      8. CONTINUUM-2073-TestNG-5.patch
        2 kB
        Marecor M. Baclay
      9. CONTINUUM-2073-TestNG-6.patch
        0.9 kB
        Marecor M. Baclay
      10. CONTINUUM-2073-TestNG-7.patch
        9 kB
        Tin Pardo
      11. continuum-sandbox-build-queue-test-data-testng.patch
        5 kB
        Tin Pardo

        Activity

        Hide
        Wendy Smoak added a comment - - edited

        CONTINUUM-2073-TestNG-6 patch applied in r774938. Thanks!

        I still get 9 failures locally, but I'll force a build on vmbuild.a.o and see if this clears up the 1 failure there.

        ... and it still fails on vmbuild:
        http://vmbuild.apache.org/continuum/buildResult.action?buildId=191419&projectGroupId=69&projectId=2284&projectName=Continuum+Webapp+Test

        Show
        Wendy Smoak added a comment - - edited CONTINUUM-2073 -TestNG-6 patch applied in r774938. Thanks! I still get 9 failures locally, but I'll force a build on vmbuild.a.o and see if this clears up the 1 failure there. ... and it still fails on vmbuild: http://vmbuild.apache.org/continuum/buildResult.action?buildId=191419&projectGroupId=69&projectId=2284&projectName=Continuum+Webapp+Test
        Hide
        Marecor M. Baclay added a comment -

        Oops, i guess CONTINUUM-2073-TestNG-6.patch does not fix the problem on vmbuild. It has different error:

        Results :

        Failed tests:
        initializeContinuum(org.apache.continuum.web.test.AboutTest)

        Tests run: 153, Failures: 1, Errors: 0, Skipped: 152

        Wendy, if you won't mind, can you attach the ../continuum-webapp-test/target/surefire-reports/TestSuite.txt file in here. I'd like to see why it failed there. Thanks!

        Show
        Marecor M. Baclay added a comment - Oops, i guess CONTINUUM-2073 -TestNG-6.patch does not fix the problem on vmbuild. It has different error: Results : Failed tests: initializeContinuum(org.apache.continuum.web.test.AboutTest) Tests run: 153, Failures: 1, Errors: 0, Skipped: 152 Wendy, if you won't mind, can you attach the ../continuum-webapp-test/target/surefire-reports/TestSuite.txt file in here. I'd like to see why it failed there. Thanks!
        Hide
        Jose Morales Martinez added a comment -

        You can view surefire report in:

        http://vmbuild.apache.org:8080/continuum/surefireReport.action?projectId=2284&projectGroupId=69&buildId=191185

        I can't reply this error. I have checked code and I have not found a reason for this error.

        Show
        Jose Morales Martinez added a comment - You can view surefire report in: http://vmbuild.apache.org:8080/continuum/surefireReport.action?projectId=2284&projectGroupId=69&buildId=191185 I can't reply this error. I have checked code and I have not found a reason for this error.
        Hide
        Tin Pardo added a comment -

        Attached CONTINUUM-2073-TestNG-7.patch

        Modifications:

        • QueueTest.java - added a method to verify if a build actively running is displayed in the queue page.
        • AbstractSeleniumTest.java - added 3 methods (clickAndWait, waitForTextPresent, waitForElementPresent)
        • AbstractContinuumTest.java - modified the value of Thread.sleep on waitAddProject method to accomodate a project which builds in a longer time.
        • AbstractBuildQueueTest.java
        • testng.properties - replaced the properties for testAddMavenTwoProject since the new method I added in the QueueTest.java requires a project which builds for a long time.

        Also attached is the "continuum-sandbox-build-queue-test-data-testng.patch" for the sample project used in this patch

        Here's the result of the test run locally:

        Tests run: 149, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 792.766 sec

        Results :

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

        [INFO] [cargo:stop

        {execution: stop-container}

        ]
        [INFO] [install:install]
        [INFO] Installing D:\workspace\continuum2\target\pom-transformed.xml to C:\Docum
        ents and Settings\tintin\.m2\repository\org\apache\continuum\continuum-webapp-te
        st\1.4.0-SNAPSHOT\continuum-webapp-test-1.4.0-SNAPSHOT.pom
        [INFO] ------------------------------------------------------------------------
        [INFO] BUILD SUCCESSFUL
        [INFO] ------------------------------------------------------------------------
        [INFO] Total time: 17 minutes 21 seconds
        [INFO] Finished at: Mon May 25 13:18:12 CST 2009
        [INFO] Final Memory: 41M/254M
        [INFO] ------------------------------------------------------------------------

        Show
        Tin Pardo added a comment - Attached CONTINUUM-2073 -TestNG-7.patch Modifications: QueueTest.java - added a method to verify if a build actively running is displayed in the queue page. AbstractSeleniumTest.java - added 3 methods (clickAndWait, waitForTextPresent, waitForElementPresent) AbstractContinuumTest.java - modified the value of Thread.sleep on waitAddProject method to accomodate a project which builds in a longer time. AbstractBuildQueueTest.java testng.properties - replaced the properties for testAddMavenTwoProject since the new method I added in the QueueTest.java requires a project which builds for a long time. Also attached is the "continuum-sandbox-build-queue-test-data-testng.patch" for the sample project used in this patch Here's the result of the test run locally: Tests run: 149, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 792.766 sec Results : Tests run: 149, Failures: 0, Errors: 0, Skipped: 0 [INFO] [cargo:stop {execution: stop-container} ] [INFO] [install:install] [INFO] Installing D:\workspace\continuum2\target\pom-transformed.xml to C:\Docum ents and Settings\tintin\.m2\repository\org\apache\continuum\continuum-webapp-te st\1.4.0-SNAPSHOT\continuum-webapp-test-1.4.0-SNAPSHOT.pom [INFO] ------------------------------------------------------------------------ [INFO] BUILD SUCCESSFUL [INFO] ------------------------------------------------------------------------ [INFO] Total time: 17 minutes 21 seconds [INFO] Finished at: Mon May 25 13:18:12 CST 2009 [INFO] Final Memory: 41M/254M [INFO] ------------------------------------------------------------------------
        Hide
        Wendy Smoak added a comment -

        I noticed another couple of patches here and went ahead and applied patch #7 and continum-build-queue-test-data for the sandbox). Thanks, Tin

        After this, I still get failures locally: Tests run: 149, Failures: 12, Errors: 0, Skipped: 6

        We've already closed this issue as the tests are definitely alive again. Please open a new issue with a more specific description if you have additional tests or changes to contribute.

        Show
        Wendy Smoak added a comment - I noticed another couple of patches here and went ahead and applied patch #7 and continum-build-queue-test-data for the sandbox). Thanks, Tin After this, I still get failures locally: Tests run: 149, Failures: 12, Errors: 0, Skipped: 6 We've already closed this issue as the tests are definitely alive again. Please open a new issue with a more specific description if you have additional tests or changes to contribute.

          People

          • Assignee:
            Jose Morales Martinez
            Reporter:
            Wendy Smoak
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: