Merge pull request #243 from ckadluba/refactoring-cleanup-tests

ckadluba · web-flow · commit dc4ea1cc49af · 2020-03-30T21:27:51.000+02:00
Reorganized test classes
diff --git a/test/Serilog.Sinks.MSSqlServer.Tests/Configuration/Microsoft.Extensions.Configuration/ConfigurationExtensionsTests.cs b/test/Serilog.Sinks.MSSqlServer.Tests/Configuration/Microsoft.Extensions.Configuration/ConfigurationExtensionsTests.cs
@@ -5,12 +5,12 @@
 using Xunit;
 using FluentAssertions;
 using Microsoft.Extensions.Configuration;
-using System;
+using Serilog.Sinks.MSSqlServer.Tests.TestUtils;
 
 namespace Serilog.Sinks.MSSqlServer.Tests
 {
     [Collection("LogTest")]
-    public sealed class ConfigurationExtensionsTests : IDisposable
+    public class ConfigurationExtensionsTests : DatabaseTestsBase
     {
         static string ConnectionStringName = "NamedConnection";
         static string ColumnOptionsSection = "CustomColumnNames";
@@ -79,10 +79,5 @@ public void ColumnOptionsFromConfigSection()
                 infoSchema.Should().Contain(columns => columns.ColumnName == "Id");
             }
         }
-
-        public void Dispose()
-        {
-            DatabaseFixture.DropTable();
-        }
     }
 }
diff --git a/test/Serilog.Sinks.MSSqlServer.Tests/Configuration/System.Configuration/ConfigurationExtensionsTests.cs b/test/Serilog.Sinks.MSSqlServer.Tests/Configuration/System.Configuration/ConfigurationExtensionsTests.cs
@@ -1,10 +1,10 @@
-﻿using System;
-using Dapper;
+﻿using Dapper;
 using Xunit;
 using FluentAssertions;
 using System.Collections.Generic;
 using System.Data.SqlClient;
 using System.Linq;
+using Serilog.Sinks.MSSqlServer.Tests.TestUtils;
 
 // Because System.Configuration is static and config is loaded automatically,
 // the tests alter the static AppConfigSectionName string value exposed by the
@@ -16,7 +16,7 @@
 namespace Serilog.Sinks.MSSqlServer.Tests
 {
     [Collection("LogTest")]
-    public sealed class ConfigurationExtensionsTests : IDisposable
+    public class ConfigurationExtensionsTests : DatabaseTestsBase
     {
         [Fact]
         public void ConnectionStringByName()
@@ -89,10 +89,5 @@ public void CustomizedColumnList()
                 infoSchema.Should().Contain(columns => columns.ColumnName == "CustomColumn");
             }
         }
-
-        public void Dispose()
-        {
-            DatabaseFixture.DropTable();
-        }
     }
 }
diff --git a/test/Serilog.Sinks.MSSqlServer.Tests/CustomStandardColumnNamesTests.cs b/test/Serilog.Sinks.MSSqlServer.Tests/CustomStandardColumnNamesTests.cs
@@ -6,11 +6,12 @@
 using Dapper;
 using Xunit;
 using FluentAssertions;
+using Serilog.Sinks.MSSqlServer.Tests.TestUtils;
 
 namespace Serilog.Sinks.MSSqlServer.Tests
 {
     [Collection("LogTest")]
-    public sealed class CustomStandardColumnNamesTests : IDisposable
+    public class CustomStandardColumnNamesTests : DatabaseTestsBase
     {
         [Fact]
         public void CustomIdColumn()
@@ -264,10 +265,5 @@ public void AuditEventToDefaultStandardColumns()
                 logEvents.Should().Contain(e => e.Message.Contains(loggingInformationMessage));
             }
         }
-
-        public void Dispose()
-        {
-            DatabaseFixture.DropTable();
-        }
     }
 }
diff --git a/test/Serilog.Sinks.MSSqlServer.Tests/IndexingFeaturesTests.cs b/test/Serilog.Sinks.MSSqlServer.Tests/IndexingFeaturesTests.cs
@@ -3,12 +3,13 @@
 using System.Linq;
 using Dapper;
 using FluentAssertions;
+using Serilog.Sinks.MSSqlServer.Tests.TestUtils;
 using Xunit;
 
 namespace Serilog.Sinks.MSSqlServer.Tests
 {
     [Collection("LogTest")]
-    public sealed class IndexingFeaturesTests : IDisposable
+    public class IndexingFeaturesTests : DatabaseTestsBase
     {
         [Fact]
         public void NonClusteredDefaultIdPrimaryKey()
@@ -110,10 +111,5 @@ public void ColumnstoreIndex()
                 results.Should().Contain(x => x.name == $"CCI_{DatabaseFixture.LogTableName}");
             }
         }
-
-        public void Dispose()
-        {
-            DatabaseFixture.DropTable();
-        }
     }
 }
diff --git a/test/Serilog.Sinks.MSSqlServer.Tests/LevelAsEnumTests.cs b/test/Serilog.Sinks.MSSqlServer.Tests/LevelAsEnumTests.cs
@@ -4,12 +4,13 @@
 using Dapper;
 using FluentAssertions;
 using Serilog.Events;
+using Serilog.Sinks.MSSqlServer.Tests.TestUtils;
 using Xunit;
 
 namespace Serilog.Sinks.MSSqlServer.Tests
 {
     [Collection("LogTest")]
-    public sealed class LevelAsEnumTests : IDisposable
+    public class LevelAsEnumTests : DatabaseTestsBase
     {
         [Fact]
         public void CanStoreLevelAsEnum()
@@ -130,10 +131,5 @@ public void AuditCanStoreLevelAsString()
                 logEvents.Should().Contain(e => e.Message.Contains(loggingInformationMessage) && e.Level == LogEventLevel.Information.ToString());
             }
         }
-
-        public void Dispose()
-        {
-            DatabaseFixture.DropTable();
-        }
     }
 }
diff --git a/test/Serilog.Sinks.MSSqlServer.Tests/MiscFeaturesTests.cs b/test/Serilog.Sinks.MSSqlServer.Tests/MiscFeaturesTests.cs
@@ -5,12 +5,13 @@
 using System.Linq;
 using Dapper;
 using FluentAssertions;
+using Serilog.Sinks.MSSqlServer.Tests.TestUtils;
 using Xunit;
 
 namespace Serilog.Sinks.MSSqlServer.Tests
 {
     [Collection("LogTest")]
-    public sealed class MiscFeaturesTests : IDisposable
+    public class MiscFeaturesTests : DatabaseTestsBase
     {
         [Fact]
         public void LogEventExcludeAdditionalProperties()
@@ -266,10 +267,5 @@ public void LogEventStoreAsEnum()
                 logEventCount.Should().HaveCount(1);
             }
         }
-
-        public void Dispose()
-        {
-            DatabaseFixture.DropTable();
-        }
     }
 }
diff --git a/test/Serilog.Sinks.MSSqlServer.Tests/PropertiesColumnFilteringTests.cs b/test/Serilog.Sinks.MSSqlServer.Tests/PropertiesColumnFilteringTests.cs
@@ -1,13 +1,14 @@
 ﻿using Dapper;
 using FluentAssertions;
+using Serilog.Sinks.MSSqlServer.Tests.TestUtils;
 using System;
 using System.Data.SqlClient;
 using Xunit;
 
 namespace Serilog.Sinks.MSSqlServer.Tests
 {
     [Collection("LogTest")]
-    public sealed class PropertiesColumnFilteringTests : IDisposable
+    public class PropertiesColumnFilteringTests : DatabaseTestsBase
     {
         [Fact]
         public void FilteredProperties()
@@ -78,10 +79,5 @@ public void FilteredPropertiesWhenAuditing()
                 logEvents.Should().NotContain(e => e.Properties.Contains("BValue"));
             }
         }
-
-        public void Dispose()
-        {
-            DatabaseFixture.DropTable();
-        }
     }
 }
diff --git a/test/Serilog.Sinks.MSSqlServer.Tests/SqlTypesTests.cs b/test/Serilog.Sinks.MSSqlServer.Tests/SqlTypesTests.cs
@@ -1,4 +1,5 @@
-﻿using System;
+﻿using Serilog.Sinks.MSSqlServer.Tests.TestUtils;
+using System;
 using System.Collections.Generic;
 using System.Collections.ObjectModel;
 using System.Data;
@@ -7,7 +8,7 @@
 namespace Serilog.Sinks.MSSqlServer.Tests
 {
     [Collection("LogTest")]
-    public sealed class SqlTypesTests : IDisposable
+    public class SqlTypesTests : DatabaseTestsBase
     {
         // Since the point of these tests are to validate we can write to
         // specific underlying SQL Server column data types, we use audit
@@ -142,10 +143,5 @@ private void Arrange(ICollection<SqlColumn> customColumns)
                 )
                 .CreateLogger();
         }
-
-        public void Dispose()
-        {
-            DatabaseFixture.DropTable();
-        }
     }
 }
diff --git a/test/Serilog.Sinks.MSSqlServer.Tests/TestUtils/DapperQueryTemplates.cs b/test/Serilog.Sinks.MSSqlServer.Tests/TestUtils/DapperQueryTemplates.cs
@@ -1,7 +1,7 @@
 ﻿using Serilog.Events;
 using System;
 
-namespace Serilog.Sinks.MSSqlServer.Tests
+namespace Serilog.Sinks.MSSqlServer.Tests.TestUtils
 {
     public class CustomStandardLogColumns
     {
diff --git a/test/Serilog.Sinks.MSSqlServer.Tests/TestUtils/DatabaseFixture.cs b/test/Serilog.Sinks.MSSqlServer.Tests/TestUtils/DatabaseFixture.cs
@@ -2,9 +2,8 @@
 using System.Data.SqlClient;
 using System.Linq;
 using Dapper;
-using Xunit;
 
-namespace Serilog.Sinks.MSSqlServer.Tests
+namespace Serilog.Sinks.MSSqlServer.Tests.TestUtils
 {
     public sealed class DatabaseFixture : IDisposable
     {
@@ -83,7 +82,4 @@ private static void CreateDatabase()
             }
         }
     }
-
-    [CollectionDefinition("LogTest", DisableParallelization = true)]
-    public class PatientSecureFixtureCollection : ICollectionFixture<DatabaseFixture> { }
 }
diff --git a/test/Serilog.Sinks.MSSqlServer.Tests/TestUtils/DatabaseTestsBase.cs b/test/Serilog.Sinks.MSSqlServer.Tests/TestUtils/DatabaseTestsBase.cs
@@ -0,0 +1,24 @@
+﻿using System;
+
+namespace Serilog.Sinks.MSSqlServer.Tests.TestUtils
+{
+    public abstract class DatabaseTestsBase : IDisposable
+    {
+        private bool _disposedValue;
+
+        protected virtual void Dispose(bool disposing)
+        {
+            if (!_disposedValue)
+            {
+                DatabaseFixture.DropTable();
+                _disposedValue = true;
+            }
+        }
+
+        public void Dispose()
+        {
+            Dispose(disposing: true);
+            GC.SuppressFinalize(this);
+        }
+    }
+}
diff --git a/test/Serilog.Sinks.MSSqlServer.Tests/TestUtils/PatientSecureFixtureCollection.cs b/test/Serilog.Sinks.MSSqlServer.Tests/TestUtils/PatientSecureFixtureCollection.cs
@@ -0,0 +1,7 @@
+﻿using Xunit;
+
+namespace Serilog.Sinks.MSSqlServer.Tests.TestUtils
+{
+    [CollectionDefinition("LogTest", DisableParallelization = true)]
+    public class PatientSecureFixtureCollection : ICollectionFixture<DatabaseFixture> { }
+}
diff --git a/test/Serilog.Sinks.MSSqlServer.Tests/TimeStampTests.cs b/test/Serilog.Sinks.MSSqlServer.Tests/TimeStampTests.cs
@@ -3,12 +3,13 @@
 using System.Data.SqlClient;
 using Dapper;
 using FluentAssertions;
+using Serilog.Sinks.MSSqlServer.Tests.TestUtils;
 using Xunit;
 
 namespace Serilog.Sinks.MSSqlServer.Tests
 {
     [Collection("LogTest")]
-    public sealed class TimeStampTests : IDisposable
+    public class TimeStampTests : DatabaseTestsBase
     {
         [Trait("Bugfix", "#187")]
         [Fact]
@@ -94,10 +95,5 @@ public void CanStoreDateTimeOffsetWithUtcTimeZone()
                 logEvents.Should().Contain(e => e.TimeStamp.Offset == new TimeSpan(0));
             }
         }
-
-        public void Dispose()
-        {
-            DatabaseFixture.DropTable();
-        }
     }
 }
diff --git a/test/Serilog.Sinks.MSSqlServer.Tests/TriggersOnLogTableTests.cs b/test/Serilog.Sinks.MSSqlServer.Tests/TriggersOnLogTableTests.cs
@@ -2,13 +2,16 @@
 using System.Data.SqlClient;
 using Dapper;
 using FluentAssertions;
+using Serilog.Sinks.MSSqlServer.Tests.TestUtils;
 using Xunit;
 
 namespace Serilog.Sinks.MSSqlServer.Tests
 {
     [Collection("LogTest")]
-    public sealed class TriggersOnLogTableTests : IDisposable
+    public class TriggersOnLogTableTests : DatabaseTestsBase
     {
+        private bool _disposedValue;
+
         [Fact]
         public void TestTriggerOnLogTableFire()
         {
@@ -136,10 +139,14 @@ AFTER INSERT
             }
         }
 
-        public void Dispose()
+        protected override void Dispose(bool disposing)
         {
-            DatabaseFixture.DropTable();
-            DatabaseFixture.DropTable(logTriggerTableName);
+            base.Dispose(disposing);
+            if (!_disposedValue)
+            {
+                DatabaseFixture.DropTable(logTriggerTableName);
+                _disposedValue = true;
+            }
         }
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -5,12 +5,12 @@`
`5`	`5`	`using Xunit;`
`6`	`6`	`using FluentAssertions;`
`7`	`7`	`using Microsoft.Extensions.Configuration;`
`8`		`-using System;`
	`8`	`+using Serilog.Sinks.MSSqlServer.Tests.TestUtils;`
`9`	`9`
`10`	`10`	`namespace Serilog.Sinks.MSSqlServer.Tests`
`11`	`11`	`{`
`12`	`12`	`[Collection("LogTest")]`
`13`		`- public sealed class ConfigurationExtensionsTests : IDisposable`
	`13`	`+ public class ConfigurationExtensionsTests : DatabaseTestsBase`
`14`	`14`	`{`
`15`	`15`	`static string ConnectionStringName = "NamedConnection";`
`16`	`16`	`static string ColumnOptionsSection = "CustomColumnNames";`
`@@ -79,10 +79,5 @@ public void ColumnOptionsFromConfigSection()`
`79`	`79`	`infoSchema.Should().Contain(columns => columns.ColumnName == "Id");`
`80`	`80`	`}`
`81`	`81`	`}`
`82`		`-`
`83`		`- public void Dispose()`
`84`		`- {`
`85`		`- DatabaseFixture.DropTable();`
`86`		`- }`
`87`	`82`	`}`
`88`	`83`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,10 +1,10 @@`
`1`		`-using System;`
`2`		`-using Dapper;`
	`1`	`+using Dapper;`
`3`	`2`	`using Xunit;`
`4`	`3`	`using FluentAssertions;`
`5`	`4`	`using System.Collections.Generic;`
`6`	`5`	`using System.Data.SqlClient;`
`7`	`6`	`using System.Linq;`
	`7`	`+using Serilog.Sinks.MSSqlServer.Tests.TestUtils;`
`8`	`8`
`9`	`9`	`// Because System.Configuration is static and config is loaded automatically,`
`10`	`10`	`// the tests alter the static AppConfigSectionName string value exposed by the`
`@@ -16,7 +16,7 @@`
`16`	`16`	`namespace Serilog.Sinks.MSSqlServer.Tests`
`17`	`17`	`{`
`18`	`18`	`[Collection("LogTest")]`
`19`		`- public sealed class ConfigurationExtensionsTests : IDisposable`
	`19`	`+ public class ConfigurationExtensionsTests : DatabaseTestsBase`
`20`	`20`	`{`
`21`	`21`	`[Fact]`
`22`	`22`	`public void ConnectionStringByName()`
`@@ -89,10 +89,5 @@ public void CustomizedColumnList()`
`89`	`89`	`infoSchema.Should().Contain(columns => columns.ColumnName == "CustomColumn");`
`90`	`90`	`}`
`91`	`91`	`}`
`92`		`-`
`93`		`- public void Dispose()`
`94`		`- {`
`95`		`- DatabaseFixture.DropTable();`
`96`		`- }`
`97`	`92`	`}`
`98`	`93`	`}`
Original file line number	Diff line number	Diff line change
`@@ -6,11 +6,12 @@`
`6`	`6`	`using Dapper;`
`7`	`7`	`using Xunit;`
`8`	`8`	`using FluentAssertions;`
	`9`	`+using Serilog.Sinks.MSSqlServer.Tests.TestUtils;`
`9`	`10`
`10`	`11`	`namespace Serilog.Sinks.MSSqlServer.Tests`
`11`	`12`	`{`
`12`	`13`	`[Collection("LogTest")]`
`13`		`- public sealed class CustomStandardColumnNamesTests : IDisposable`
	`14`	`+ public class CustomStandardColumnNamesTests : DatabaseTestsBase`
`14`	`15`	`{`
`15`	`16`	`[Fact]`
`16`	`17`	`public void CustomIdColumn()`
`@@ -264,10 +265,5 @@ public void AuditEventToDefaultStandardColumns()`
`264`	`265`	`logEvents.Should().Contain(e => e.Message.Contains(loggingInformationMessage));`
`265`	`266`	`}`
`266`	`267`	`}`
`267`		`-`
`268`		`- public void Dispose()`
`269`		`- {`
`270`		`- DatabaseFixture.DropTable();`
`271`		`- }`
`272`	`268`	`}`
`273`	`269`	`}`
Original file line number	Diff line number	Diff line change
`@@ -3,12 +3,13 @@`
`3`	`3`	`using System.Linq;`
`4`	`4`	`using Dapper;`
`5`	`5`	`using FluentAssertions;`
	`6`	`+using Serilog.Sinks.MSSqlServer.Tests.TestUtils;`
`6`	`7`	`using Xunit;`
`7`	`8`
`8`	`9`	`namespace Serilog.Sinks.MSSqlServer.Tests`
`9`	`10`	`{`
`10`	`11`	`[Collection("LogTest")]`
`11`		`- public sealed class IndexingFeaturesTests : IDisposable`
	`12`	`+ public class IndexingFeaturesTests : DatabaseTestsBase`
`12`	`13`	`{`
`13`	`14`	`[Fact]`
`14`	`15`	`public void NonClusteredDefaultIdPrimaryKey()`
`@@ -110,10 +111,5 @@ public void ColumnstoreIndex()`
`110`	`111`	`results.Should().Contain(x => x.name == $"CCI_{DatabaseFixture.LogTableName}");`
`111`	`112`	`}`
`112`	`113`	`}`
`113`		`-`
`114`		`- public void Dispose()`
`115`		`- {`
`116`		`- DatabaseFixture.DropTable();`
`117`		`- }`
`118`	`114`	`}`
`119`	`115`	`}`
Original file line number	Diff line number	Diff line change
`@@ -4,12 +4,13 @@`
`4`	`4`	`using Dapper;`
`5`	`5`	`using FluentAssertions;`
`6`	`6`	`using Serilog.Events;`
	`7`	`+using Serilog.Sinks.MSSqlServer.Tests.TestUtils;`
`7`	`8`	`using Xunit;`
`8`	`9`
`9`	`10`	`namespace Serilog.Sinks.MSSqlServer.Tests`
`10`	`11`	`{`
`11`	`12`	`[Collection("LogTest")]`
`12`		`- public sealed class LevelAsEnumTests : IDisposable`
	`13`	`+ public class LevelAsEnumTests : DatabaseTestsBase`
`13`	`14`	`{`
`14`	`15`	`[Fact]`
`15`	`16`	`public void CanStoreLevelAsEnum()`
`@@ -130,10 +131,5 @@ public void AuditCanStoreLevelAsString()`
`130`	`131`	`logEvents.Should().Contain(e => e.Message.Contains(loggingInformationMessage) && e.Level == LogEventLevel.Information.ToString());`
`131`	`132`	`}`
`132`	`133`	`}`
`133`		`-`
`134`		`- public void Dispose()`
`135`		`- {`
`136`		`- DatabaseFixture.DropTable();`
`137`		`- }`
`138`	`134`	`}`
`139`	`135`	`}`
Original file line number	Diff line number	Diff line change
`@@ -5,12 +5,13 @@`
`5`	`5`	`using System.Linq;`
`6`	`6`	`using Dapper;`
`7`	`7`	`using FluentAssertions;`
	`8`	`+using Serilog.Sinks.MSSqlServer.Tests.TestUtils;`
`8`	`9`	`using Xunit;`
`9`	`10`
`10`	`11`	`namespace Serilog.Sinks.MSSqlServer.Tests`
`11`	`12`	`{`
`12`	`13`	`[Collection("LogTest")]`
`13`		`- public sealed class MiscFeaturesTests : IDisposable`
	`14`	`+ public class MiscFeaturesTests : DatabaseTestsBase`
`14`	`15`	`{`
`15`	`16`	`[Fact]`
`16`	`17`	`public void LogEventExcludeAdditionalProperties()`
`@@ -266,10 +267,5 @@ public void LogEventStoreAsEnum()`
`266`	`267`	`logEventCount.Should().HaveCount(1);`
`267`	`268`	`}`
`268`	`269`	`}`
`269`		`-`
`270`		`- public void Dispose()`
`271`		`- {`
`272`		`- DatabaseFixture.DropTable();`
`273`		`- }`
`274`	`270`	`}`
`275`	`271`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,13 +1,14 @@`
`1`	`1`	`using Dapper;`
`2`	`2`	`using FluentAssertions;`
	`3`	`+using Serilog.Sinks.MSSqlServer.Tests.TestUtils;`
`3`	`4`	`using System;`
`4`	`5`	`using System.Data.SqlClient;`
`5`	`6`	`using Xunit;`
`6`	`7`
`7`	`8`	`namespace Serilog.Sinks.MSSqlServer.Tests`
`8`	`9`	`{`
`9`	`10`	`[Collection("LogTest")]`
`10`		`- public sealed class PropertiesColumnFilteringTests : IDisposable`
	`11`	`+ public class PropertiesColumnFilteringTests : DatabaseTestsBase`
`11`	`12`	`{`
`12`	`13`	`[Fact]`
`13`	`14`	`public void FilteredProperties()`
`@@ -78,10 +79,5 @@ public void FilteredPropertiesWhenAuditing()`
`78`	`79`	`logEvents.Should().NotContain(e => e.Properties.Contains("BValue"));`
`79`	`80`	`}`
`80`	`81`	`}`
`81`		`-`
`82`		`- public void Dispose()`
`83`		`- {`
`84`		`- DatabaseFixture.DropTable();`
`85`		`- }`
`86`	`82`	`}`
`87`	`83`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,5 @@`
`1`		`-using System;`
	`1`	`+using Serilog.Sinks.MSSqlServer.Tests.TestUtils;`
	`2`	`+using System;`
`2`	`3`	`using System.Collections.Generic;`
`3`	`4`	`using System.Collections.ObjectModel;`
`4`	`5`	`using System.Data;`
`@@ -7,7 +8,7 @@`
`7`	`8`	`namespace Serilog.Sinks.MSSqlServer.Tests`
`8`	`9`	`{`
`9`	`10`	`[Collection("LogTest")]`
`10`		`- public sealed class SqlTypesTests : IDisposable`
	`11`	`+ public class SqlTypesTests : DatabaseTestsBase`
`11`	`12`	`{`
`12`	`13`	`// Since the point of these tests are to validate we can write to`
`13`	`14`	`// specific underlying SQL Server column data types, we use audit`
`@@ -142,10 +143,5 @@ private void Arrange(ICollection<SqlColumn> customColumns)`
`142`	`143`	`)`
`143`	`144`	`.CreateLogger();`
`144`	`145`	`}`
`145`		`-`
`146`		`- public void Dispose()`
`147`		`- {`
`148`		`- DatabaseFixture.DropTable();`
`149`		`- }`
`150`	`146`	`}`
`151`	`147`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,7 @@`
`1`	`1`	`using Serilog.Events;`
`2`	`2`	`using System;`
`3`	`3`
`4`		`-namespace Serilog.Sinks.MSSqlServer.Tests`
	`4`	`+namespace Serilog.Sinks.MSSqlServer.Tests.TestUtils`
`5`	`5`	`{`
`6`	`6`	`public class CustomStandardLogColumns`
`7`	`7`	`{`
Original file line number	Diff line number	Diff line change
`@@ -2,9 +2,8 @@`
`2`	`2`	`using System.Data.SqlClient;`
`3`	`3`	`using System.Linq;`
`4`	`4`	`using Dapper;`
`5`		`-using Xunit;`
`6`	`5`
`7`		`-namespace Serilog.Sinks.MSSqlServer.Tests`
	`6`	`+namespace Serilog.Sinks.MSSqlServer.Tests.TestUtils`
`8`	`7`	`{`
`9`	`8`	`public sealed class DatabaseFixture : IDisposable`
`10`	`9`	`{`
`@@ -83,7 +82,4 @@ private static void CreateDatabase()`
`83`	`82`	`}`
`84`	`83`	`}`
`85`	`84`	`}`
`86`		`-`
`87`		`- [CollectionDefinition("LogTest", DisableParallelization = true)]`
`88`		`- public class PatientSecureFixtureCollection : ICollectionFixture<DatabaseFixture> { }`
`89`	`85`	`}`