FileLoggerProcessorTests.RollsTextFiles

[Tratcher]

Failing Test(s)

Microsoft.AspNetCore.HttpLogging.FileLoggerProcessorTests.RollsTextFiles

Error Message

   Assert.Equal() Failure
          ↓ (pos 0)
Expected: Message two\n
Actual:   
          ↑ (pos 0)
  Stack Trace:
     at Microsoft.AspNetCore.HttpLogging.FileLoggerProcessorTests.RollsTextFiles() in /_/src/Middleware/HttpLogging/test/FileLoggerProcessorTests.cs:line 117

aspnetcore/src/Middleware/HttpLogging/test/FileLoggerProcessorTests.cs

Line 117 in 69eb4b3

Assert.Equal("Message two" + Environment.NewLine, File.ReadAllText(fileName2));

Build

https://dev.azure.com/dnceng/public/_build/results?buildId=1231336&view=ms.vss-test-web.build-test-results-tab&runId=36670362&resultId=110218&paneView=debug
(Alpine.312.Amd64.Open)Ubuntu.1804.Amd64.Open

[ghost]

Thanks for contacting us.

We're moving this issue to the Next sprint planning milestone for future evaluation / consideration. We would like to keep this around to collect more feedback, which can help us with prioritizing this work. We will re-evaluate this issue, during our next planning meeting(s).
If we later determine, that the issue has no community involvement, or it's very rare and low-impact issue, we will close it - so that the team can focus on more important and high impact issues.
To learn more about what to expect next and how this issue will be handled you can read more about our triage process here.

[wtgodbe]

Interesting, the file exists but was empty. Not one I ever saw during development - I can take a look after feature work is done

[HaoK]

Doesn't look like this has happened recently, 30 day clean run, should we just reenable this so we can see if/when it fails again?

[HaoK]

[wtgodbe]

Yeah, should be fine to unquarantine - we had a community PR to improve these tests a while back