Fix tests that were mocking ILogger

aspnet / KestrelHttpServer

[Archived] A cross platform web server for ASP.NET Core. Project moved to https://github.com/aspnet/AspNetCore

Apache License 2.0

2.62k stars 526 forks source link

Fix tests that were mocking ILogger #3066

Closed pakrym closed 6 years ago

pakrym commented 6 years ago

Use TestSink.MessageLogged instead.

Tratcher commented 6 years ago

Do you have a summary for why it was broken?

pakrym commented 6 years ago

We started using structs as TState in ILogger.Log().

I wasn't able to find an easy way of setting up Mock method where T is an unknown struct but it doesn't matter much because mocks are not required there at all.

halter73 commented 6 years ago

https://github.com/moq/moq4/issues/343