Merge pull request #293 from ckadluba/fix-code-analysis-warning-string-contains

ckadluba · web-flow · commit a575c2b9ecb5 · 2020-06-06T22:41:52.000+02:00
Fixed code analysis warning
diff --git a/test/Serilog.Sinks.MSSqlServer.Tests/Serilog.Sinks.MSSqlServer.Tests.csproj b/test/Serilog.Sinks.MSSqlServer.Tests/Serilog.Sinks.MSSqlServer.Tests.csproj
@@ -3,6 +3,7 @@
   <PropertyGroup>
     <TargetFrameworks>net452;netcoreapp2.0;netcoreapp2.2</TargetFrameworks>
     <TargetFrameworks Condition=" '$(OS)' != 'Windows_NT' ">netstandard2.0</TargetFrameworks>
+    <TreatWarningsAsErrors>true</TreatWarningsAsErrors>
     <AssemblyName>Serilog.Sinks.MSSqlServer.Tests</AssemblyName>
     <AssemblyOriginatorKeyFile>../../assets/Serilog.snk</AssemblyOriginatorKeyFile>
     <SignAssembly>true</SignAssembly>
diff --git a/test/Serilog.Sinks.MSSqlServer.Tests/TestUtils/DatabaseTestsBase.cs b/test/Serilog.Sinks.MSSqlServer.Tests/TestUtils/DatabaseTestsBase.cs
@@ -109,7 +109,7 @@ protected static void VerifyCustomLogMessageWasWritten(string expectedMessage)
             {
                 var logEvents = conn.Query<CustomStandardLogColumns>($"SELECT CustomMessage FROM {DatabaseFixture.LogTableName}");
 
-                logEvents.Should().Contain(e => e.CustomMessage.Contains(expectedMessage));
+                logEvents.Should().Contain(e => e.CustomMessage == expectedMessage);
             }
         }
 

Original file line number	Diff line number	Diff line change
`@@ -109,7 +109,7 @@ protected static void VerifyCustomLogMessageWasWritten(string expectedMessage)`
`109`	`109`	`{`
`110`	`110`	`var logEvents = conn.Query<CustomStandardLogColumns>($"SELECT CustomMessage FROM {DatabaseFixture.LogTableName}");`
`111`	`111`
`112`		`- logEvents.Should().Contain(e => e.CustomMessage.Contains(expectedMessage));`
	`112`	`+ logEvents.Should().Contain(e => e.CustomMessage == expectedMessage);`
`113`	`113`	`}`
`114`	`114`	`}`
`115`	`115`