[SUREFIRE-1208] Start stream capture before loading tests

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.19
• Fix Version/s: 2.19.1
• Component/s: Junit 4.7+ (parallel) support, Junit 4.x support, TestNG support
• Assignee: Tibor Digana
• Reporter: Tibor Digana
• Created: 12/Dec/15 21:46
• Updated: 12/Dec/15 22:45
• Resolved: 12/Dec/15 21:55

Description

If, when scanClassPath() runs, a class is loaded and that loading process produces a log (via log4j in our case), the logging framework creates its console appender around the current stdout stream, before the stream is replaced to start the redirection of stdout and stderr to a file (when redirectTestOutputToFile is used). The consequences are that stdout gets flooded with logs (i.e. redirectTestOutputToFile doesn't work).

Scanning the classpath after the capture is started, fixed the issue.

I believe that other providers have the same issue.