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

Add a better timeout for the ConsoleAccessLogTestCase

    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.

        Gliffy Diagrams

          Attachments

            Activity

              People

              • Assignee:
                jamezp James Perkins
                Reporter:
                jamezp James Perkins
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: