Uploaded image for project: 'XWiki Platform'
  1. XWiki Platform
  2. XWIKI-10828

Speed up XWiki instance starting in functional tests

    Details

    • Type: Task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 6.1
    • Fix Version/s: 6.2-milestone-2
    • Component/s: Test Framework
    • Labels:
      None
    • Difficulty:
      Unknown
    • Documentation:
      N/A
    • Documentation in Release Notes:
      N/A
    • Similar issues:

      Description

      Right now we wait 15 seconds to check if an XWiki instance is already started before starting one. This is too long and not necessary.

      01:37:49.819 [main] INFO  o.x.test.integration.XWikiExecutor - Checking if an XWiki server is already started at [http://localhost:8080]
      01:38:04.880 [main] INFO  o.x.test.integration.XWikiExecutor - No server is responding on [http://localhost:8080/xwiki/bin/get/Main/] after [15] seconds
      01:38:04.880 [main] INFO  o.x.test.integration.XWikiExecutor - Starting XWiki server at [http://localhost:8080]
      01:38:04.903 [main] INFO  o.x.test.integration.XWikiExecutor - Checking that XWiki is up and running...
      01:38:04.909 [Thread-2] INFO  o.x.t.i.XWikiLogOutputStream - Starting Jetty on port 8080, please wait...
      ...
      

      Instead do the following:

          /**
           * Start XWiki using the following strategy:
           * <ul>
           *   <li>If the {@link #VERIFY_RUNNING_XWIKI_AT_START} property is set then checks if an XWiki instance is already
           *       running before trying to start XWiki and if so, reuse it and don't start XWiki</li>
           *   <li>If the {@link #VERIFY_RUNNING_XWIKI_AT_START} property is set to false then verify if some XWiki instance
           *       is already running by verifying if the port is free and fail if so. Otherwise start XWiki.</li>
           * </ul>
           */
      

        Attachments

          Activity

            People

            • Assignee:
              vmassol Vincent Massol
              Reporter:
              vmassol Vincent Massol
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: