Details
-
Bug
-
Resolution: Done
-
Major
-
None
-
None
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.