Uploaded image for project: 'WildFly'
  1. WildFly
  2. WFLY-12108

Add a better timeout for the ConsoleAccessLogTestCase

    XMLWordPrintable

Details

    Description

      The ConsoleAccessLogTestCase had a potential for race conditions only waiting 1 second for messages to be logged. The test has now failed twice and CI which indicates that is not a long enough timeout. The test should be updated with a better timeout routine.

      Due to this the testPredicate() should be removed. It always ran the risk of getting a false positive due to the possible race between when the log is written vs when it is read. Since the timeout will be increased I see no reason to leave this test. If there are concerns with the predicate not working a test should be written in Undertow and likely already exists.

      Attachments

        Activity

          People

            jperkins-rhn James Perkins
            jperkins-rhn James Perkins
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: