Akka.API.Tests.DefaultLogFormatSpec.ShouldUseDefaultLogFormat: racy failures

[Aaronontheweb]

VerifyException : Directory: /home/vsts/work/1/s/src/core/Akka.API.Tests/verify
NotEqual:
  - Received: DefaultLogFormatSpec.ShouldUseDefaultLogFormat.DotNet8_0.received.txt
    Verified: DefaultLogFormatSpec.ShouldUseDefaultLogFormat.DotNet.verified.txt

FileContent:

NotEqual:

Received: DefaultLogFormatSpec.ShouldUseDefaultLogFormat.DotNet8_0.received.txt
Verified: DefaultLogFormatSpec.ShouldUseDefaultLogFormat.DotNet.verified.txt
Compare Result:
+ [DEBUG][DateTime][Thread 0001][EventStream(test)] Logger log1-TestEventListener [Akka.TestKit.TestEventListener] started
  [DEBUG][DateTime][Thread 0001][ActorSystem(test)] This is a test 1 cheese
  [INFO][DateTime][Thread 0001][ActorSystem(test)] This is a test 1
  [WARNING][DateTime][Thread 0001][ActorSystem(test)] This is a test 1
  [ERROR][DateTime][Thread 0001][ActorSystem(test)] This is a test 1

The problem with this spec is that SOMETIMES [DEBUG][DateTime][Thread 0001][EventStream(test)] Logger log1-TestEventListener [Akka.TestKit.TestEventListener] started gets included, other times it doesn't - this is a minor race condition where the xUnit logger sometimes does / does not get added before the TestKit event listener gets added.

We should fix this by having Verify scrub that line for this spec.