Added unit tests for SqlClient wrapper classes.

Author: ckadluba

test/Serilog.Sinks.MSSqlServer.Tests/Sinks/MSSqlServer/Platform/SqlClient/SqlBulkCopyWrapperTests.cs

@@ -0,0 +1,39 @@
+using System;
+#if NET452
+using System.Data.SqlClient;
+#else
+using Microsoft.Data.SqlClient;
+#endif
+using Serilog.Sinks.MSSqlServer.Platform.SqlClient;
+using Serilog.Sinks.MSSqlServer.Tests.TestUtils;
+using Xunit;
+
+namespace Serilog.Sinks.MSSqlServer.Tests.Sinks.MSSqlServer.Platform.SqlClient
+{
+    [Trait(TestCategory.TraitName, TestCategory.Unit)]
+    public class SqlBulkCopyWrapperTests
+    {
+        [Fact]
+        public void InitializeThrowsIfSqlBulkCopyIsNull()
+        {
+            Assert.Throws<ArgumentNullException>(() => new SqlBulkCopyWrapper(null));
+        }
+
+        [Fact]
+        public void AddSqlBulkCopyColumnMappingDoesNotThrow()
+        {
+            // Arrange
+            using (var connection = new SqlConnection())
+            {
+                using (var sqlBulkCopy = new SqlBulkCopy(connection))
+                {
+                    using (var sut = new SqlBulkCopyWrapper(sqlBulkCopy))
+                    {
+                        // Act (should not throw)
+                        sut.AddSqlBulkCopyColumnMapping("Column", "Column");
+                    }
+                }
+            }
+        }
+    }
+}

test/Serilog.Sinks.MSSqlServer.Tests/Sinks/MSSqlServer/Platform/SqlClient/SqlCommandWrapperTests.cs

@@ -0,0 +1,36 @@
+using System;
+#if NET452
+using System.Data.SqlClient;
+#else
+using Microsoft.Data.SqlClient;
+#endif
+using Xunit;
+using Serilog.Sinks.MSSqlServer.Tests.TestUtils;
+using Serilog.Sinks.MSSqlServer.Platform.SqlClient;
+
+namespace Serilog.Sinks.MSSqlServer.Tests.Sinks.MSSqlServer.Platform.SqlClient
+{
+    [Trait(TestCategory.TraitName, TestCategory.Unit)]
+    public class SqlCommandWrapperTests
+    {
+        [Fact]
+        public void InitializeThrowsIfSqlCommandIsNull()
+        {
+            Assert.Throws<ArgumentNullException>(() => new SqlCommandWrapper(null));
+        }
+
+        [Fact]
+        public void AddParameterDoesNotThrow()
+        {
+            // Arrange
+            using (var sqlCommand = new SqlCommand())
+            {
+                using (var sut = new SqlCommandWrapper(sqlCommand))
+                {
+                    // Act (should not throw)
+                    sut.AddParameter("Parameter", "Value");
+                }
+            }
+        }
+    }
+}

test/Serilog.Sinks.MSSqlServer.Tests/Sinks/MSSqlServer/Platform/SqlClient/SqlConnectionWrapperTests.cs

@@ -0,0 +1,70 @@
+#if NET452
+using System.Data.SqlClient;
+#else
+using Microsoft.Data.SqlClient;
+#endif
+using Xunit;
+using Serilog.Sinks.MSSqlServer.Tests.TestUtils;
+using Serilog.Sinks.MSSqlServer.Platform.SqlClient;
+using System;
+
+namespace Serilog.Sinks.MSSqlServer.Tests.Sinks.MSSqlServer.Platform.SqlClient
+{
+    [Trait(TestCategory.TraitName, TestCategory.Unit)]
+    public class SqlConnectionWrapperTests
+    {
+        [Fact]
+        public void InitializeThrowsIfCalledWithAuthenticationTokenOnDotNetFramework452ButNotOnOtherTargets()
+        {
+#if NET452
+            Assert.Throws<InvalidOperationException>(() => new SqlConnectionWrapper(DatabaseFixture.LogEventsConnectionString, "AuthenticationToken"));
+#else
+            // Should not throw
+            new SqlConnectionWrapper(@"Data Source=(localdb)\MSSQLLocalDB;Initial Catalog=Master", "AuthenticationToken");
+#endif
+        }
+
+        [Fact]
+        public void CreateCommandReturnsSqlCommandWrapper()
+        {
+            // Arrange
+            using (var sut = new SqlConnectionWrapper(DatabaseFixture.LogEventsConnectionString, null))
+            {
+                // Act
+                var result = sut.CreateCommand();
+
+                // Assert
+                Assert.NotNull(result);
+            }
+        }
+
+        [Fact]
+        public void CreateCommandWithParameterReturnsSqlCommandWrapper()
+        {
+            // Arrange
+            using (var sut = new SqlConnectionWrapper(DatabaseFixture.LogEventsConnectionString, null))
+            {
+                // Act
+                var result = sut.CreateCommand("CommandText");
+
+                // Assert
+                Assert.NotNull(result);
+                Assert.Equal("CommandText", result.CommandText);
+            }
+        }
+
+        [Fact]
+        public void CreateSqlBulkCopyReturnsSqlBulkCopyWrapper()
+        {
+            // Arrange
+            using (var sut = new SqlConnectionWrapper(DatabaseFixture.LogEventsConnectionString, null))
+            {
+                // Act
+                var result = sut.CreateSqlBulkCopy(false, "DestinationTableName");
+
+                // Assert
+                Assert.NotNull(result);
+            }
+        }
+    }
+}