diff --git a/test/Kestrel.FunctionalTests/RequestTests.cs b/test/Kestrel.FunctionalTests/RequestTests.cs index bef16a7333..ff063b35bf 100644 --- a/test/Kestrel.FunctionalTests/RequestTests.cs +++ b/test/Kestrel.FunctionalTests/RequestTests.cs @@ -269,6 +269,7 @@ namespace Microsoft.AspNetCore.Server.Kestrel.FunctionalTests { var connectionStarted = new SemaphoreSlim(0); var connectionReset = new SemaphoreSlim(0); + var loggedHigherThanDebug = false; var mockLogger = new Mock(); mockLogger @@ -286,6 +287,11 @@ namespace Microsoft.AspNetCore.Server.Kestrel.FunctionalTests { connectionReset.Release(); } + + if (logLevel > LogLevel.Debug) + { + loggedHigherThanDebug = true; + } }); var mockLoggerFactory = new Mock(); @@ -315,12 +321,15 @@ namespace Microsoft.AspNetCore.Server.Kestrel.FunctionalTests // and therefore not logged. Assert.True(await connectionReset.WaitAsync(TestConstants.DefaultTimeout)); } + + Assert.False(loggedHigherThanDebug); } [Fact] public async Task ConnectionResetBetweenRequestsIsLoggedAsDebug() { var connectionReset = new SemaphoreSlim(0); + var loggedHigherThanDebug = false; var mockLogger = new Mock(); mockLogger @@ -334,6 +343,11 @@ namespace Microsoft.AspNetCore.Server.Kestrel.FunctionalTests { connectionReset.Release(); } + + if (logLevel > LogLevel.Debug) + { + loggedHigherThanDebug = true; + } }); var mockLoggerFactory = new Mock(); @@ -375,6 +389,8 @@ namespace Microsoft.AspNetCore.Server.Kestrel.FunctionalTests // and therefore not logged. Assert.True(await connectionReset.WaitAsync(TestConstants.DefaultTimeout)); } + + Assert.False(loggedHigherThanDebug); } [Fact] @@ -383,6 +399,7 @@ namespace Microsoft.AspNetCore.Server.Kestrel.FunctionalTests var requestStarted = new SemaphoreSlim(0); var connectionReset = new SemaphoreSlim(0); var connectionClosing = new SemaphoreSlim(0); + var loggedHigherThanDebug = false; var mockLogger = new Mock(); mockLogger @@ -398,6 +415,11 @@ namespace Microsoft.AspNetCore.Server.Kestrel.FunctionalTests { connectionReset.Release(); } + + if (logLevel > LogLevel.Debug) + { + loggedHigherThanDebug = true; + } }); var mockLoggerFactory = new Mock(); @@ -435,6 +457,8 @@ namespace Microsoft.AspNetCore.Server.Kestrel.FunctionalTests Assert.True(await connectionReset.WaitAsync(TestConstants.DefaultTimeout), "Connection reset event should have been logged"); connectionClosing.Release(); } + + Assert.False(loggedHigherThanDebug, "Logged event should not have been higher than debug."); } [Fact]