Clean up some tests

Author: BrettJaner

src/Serilog.Sinks.MSSqlServer/Sinks/MSSqlServer/ColumnOptions/ColumnOptions.cs

@@ -90,6 +90,7 @@ public ICollection<StandardColumn> Store
 
         /// <summary>
         /// Indicates if triggers should be disabled when inserting log entries.
+        /// Only applies when SqlBulkCopy is used.
         /// </summary>
         public bool DisableTriggers { get; set; }
 

test/Serilog.Sinks.MSSqlServer.Tests/Sinks/MSSqlServer/Platform/SqlInsertBatchWriterTests.cs

@@ -73,7 +73,7 @@ public async Task WriteBatchCallsSqlConnectionFactoryCreate()
         }
 
         [Fact]
-        public async Task WriteBatchCallsSqlConnectionWrapperOpen()
+        public async Task WriteBatchCallsSqlConnectionWrapperOpenAsync()
         {
             // Arrange
             var logEvents = CreateLogEvents();
@@ -95,7 +95,7 @@ public async Task WriteBatchCallsSqlConnectionWrappeCreateCommand()
             await _sut.WriteBatch(logEvents, null);
 
             // Assert
-            _sqlConnectionWrapperMock.Verify(c => c.CreateCommand(), Times.AtLeastOnce);
+            _sqlConnectionWrapperMock.Verify(c => c.CreateCommand(), Times.Exactly(logEvents.Count));
         }
 
         [Fact]
@@ -169,7 +169,7 @@ public async Task WriteBatchCallsSqlCommandWrapperExecuteNonQueryAsync()
             await _sut.WriteBatch(logEvents, null);
 
             // Assert
-            _sqlCommandWrapperMock.Verify(c => c.ExecuteNonQueryAsync(), Times.AtLeastOnce);
+            _sqlCommandWrapperMock.Verify(c => c.ExecuteNonQueryAsync(), Times.Exactly(logEvents.Count));
         }
 
         [Fact]
@@ -182,7 +182,7 @@ public async Task WriteBatchCallsSqlCommandWrapperDispose()
             await _sut.WriteBatch(logEvents, null);
 
             // Assert
-            _sqlCommandWrapperMock.Verify(c => c.Dispose(), Times.AtLeastOnce);
+            _sqlCommandWrapperMock.Verify(c => c.Dispose(), Times.Exactly(logEvents.Count));
         }
 
         [Fact]
@@ -247,7 +247,7 @@ public async Task WriteBatchRethrowsIfSqlCommandAddParameterThrows()
         }
 
         [Fact]
-        public async Task WriteBatchRethrowsIfSqlCommandExecuteNonQueryThrows()
+        public async Task WriteBatchRethrowsIfSqlCommandExecuteNonQueryAsyncThrows()
         {
             // Arrange
             _sqlCommandWrapperMock.Setup(c => c.ExecuteNonQueryAsync()).Callback(() => throw new InvalidOperationException());