Read new sink option PreventEnlistInTransaction from all supported configuration sources.

Author: ckadluba

src/Serilog.Sinks.MSSqlServer/Configuration/Implementations/Microsoft.Extensions.Configuration/MicrosoftExtensionsSinkOptionsProvider.cs

@@ -25,6 +25,7 @@ private static void ReadTableOptions(IConfigurationSection config, MSSqlServerSi
             SetProperty.IfNotNull<string>(config["tableName"], val => sinkOptions.TableName = val);
             SetProperty.IfNotNull<string>(config["schemaName"], val => sinkOptions.SchemaName = val);
             SetProperty.IfNotNull<bool>(config["autoCreateSqlTable"], val => sinkOptions.AutoCreateSqlTable = val);
+            SetProperty.IfNotNull<bool>(config["preventEnlistInTransaction"], val => sinkOptions.PreventEnlistInTransaction = val);
         }
 
         private static void ReadBatchSettings(IConfigurationSection config, MSSqlServerSinkOptions sinkOptions)

src/Serilog.Sinks.MSSqlServer/Configuration/Implementations/System.Configuration/MSSqlServerConfigurationSection.cs

@@ -157,6 +157,12 @@ public ValueConfigElement AutoCreateSqlTable
             get => (ValueConfigElement)base[nameof(AutoCreateSqlTable)];
         }
 
+        [ConfigurationProperty(nameof(PreventEnlistInTransaction))]
+        public ValueConfigElement PreventEnlistInTransaction
+        {
+            get => (ValueConfigElement)base[nameof(PreventEnlistInTransaction)];
+        }
+
         [ConfigurationProperty(nameof(BatchPostingLimit))]
         public ValueConfigElement BatchPostingLimit
         {

src/Serilog.Sinks.MSSqlServer/Configuration/Implementations/System.Configuration/SystemConfigurationSinkOptionsProvider.cs

@@ -21,6 +21,8 @@ private static void ReadTableOptions(MSSqlServerConfigurationSection config, MSS
             SetProperty.IfProvided<string>(config.SchemaName, nameof(config.SchemaName.Value), value => sinkOptions.SchemaName = value);
             SetProperty.IfProvided<bool>(config.AutoCreateSqlTable, nameof(config.AutoCreateSqlTable.Value),
                 value => sinkOptions.AutoCreateSqlTable = value);
+            SetProperty.IfProvided<bool>(config.PreventEnlistInTransaction, nameof(config.PreventEnlistInTransaction.Value),
+                value => sinkOptions.PreventEnlistInTransaction = value);
         }
 
         private static void ReadBatchSettings(MSSqlServerConfigurationSection config, MSSqlServerSinkOptions sinkOptions)

test/Serilog.Sinks.MSSqlServer.Tests/Configuration/Implementations/Microsoft.Extensions.Configuration/MicrosoftExtensionsSinkOptionsProviderTests.cs

@@ -76,6 +76,20 @@ public void ConfigureSinkOptionsSetsAutoCreateSqlTable()
             Assert.True(result.AutoCreateSqlTable);
         }
 
+        [Fact]
+        public void ConfigureSinkOptionsSetsPreventEnlistInTransaction()
+        {
+            // Arrange
+            _configurationSectionMock.Setup(s => s["preventEnlistInTransaction"]).Returns("false");
+            var sut = new MicrosoftExtensionsSinkOptionsProvider();
+
+            // Act
+            var result = sut.ConfigureSinkOptions(new MSSqlServerSinkOptions(), _configurationSectionMock.Object);
+
+            // Assert
+            Assert.False(result.PreventEnlistInTransaction);
+        }
+
         [Fact]
         public void ConfigureSinkOptionsSetsBatchPostingLimit()
         {

test/Serilog.Sinks.MSSqlServer.Tests/Configuration/Implementations/System.Configuration/SystemConfigurationSinkOptionsProviderTests.cs

@@ -23,5 +23,21 @@ public void ConfigureSinkOptionsReadsBatchSettings()
             // Assert
             Assert.True(sinkOptions.EagerlyEmitFirstEvent);
         }
+
+        [Fact]
+        public void ConfigureSinkOptionsReadsPreventEnlistInTransaction()
+        {
+            // Arrange
+            var configSection = new MSSqlServerConfigurationSection();
+            configSection.PreventEnlistInTransaction.Value = "false";
+            var sinkOptions = new MSSqlServerSinkOptions { PreventEnlistInTransaction = true };
+            var sut = new SystemConfigurationSinkOptionsProvider();
+
+            // Act
+            sut.ConfigureSinkOptions(configSection, sinkOptions);
+
+            // Assert
+            Assert.False(sinkOptions.PreventEnlistInTransaction);
+        }
     }
 }