Add tests for #142 fixed in previous commit

Author: tsimbalar

test/Serilog.Settings.Configuration.Tests/ConfigurationSettingsTests.cs

@@ -425,7 +425,7 @@ public void SinkWithIConfigurationArguments()
                 ""Serilog"": {            
                     ""Using"": [""TestDummies""],
                     ""WriteTo"": [{
-                        ""Name"": ""DummyRollingFile"",
+                        ""Name"": ""DummyWithConfiguration"",
                         ""Args"": {""pathFormat"" : ""C:\\"",
                                    ""configurationSection"" : { ""foo"" : ""bar"" } }
                     }]        
@@ -435,14 +435,17 @@ public void SinkWithIConfigurationArguments()
             // IConfiguration and IConfigurationSection arguments do not have
             // default values so they will throw if they are not populated
 
+
+            DummyConfigurationSink.Reset();
             var log = ConfigFromJson(json)
                 .CreateLogger();
 
-            DummyRollingFileSink.Reset();
 
             log.Write(Some.InformationEvent());
 
-            Assert.Equal(1, DummyRollingFileSink.Emitted.Count);
+            Assert.NotNull(DummyConfigurationSink.Configuration);
+            Assert.NotNull(DummyConfigurationSink.ConfigSection);
+            Assert.Equal("bar", DummyConfigurationSink.ConfigSection["foo"]);
         }
 
         [Fact]

test/Serilog.Settings.Configuration.Tests/LoggerConfigurationExtensionsTests.cs

@@ -56,7 +56,7 @@ public void ReadFromConfigurationSectionThrowsWhenTryingToCallConfigurationMetho
                 ""NotSerilog"": {            
                     ""Using"": [""TestDummies""],
                     ""WriteTo"": [{
-                        ""Name"": ""DummyRollingFile"",
+                        ""Name"": ""DummyWithConfiguration"",
                         ""Args"": {""pathFormat"" : ""C:\\"",
                                    ""configurationSection"" : { ""foo"" : ""bar"" } }
                     }]        
@@ -74,7 +74,7 @@ public void ReadFromConfigurationSectionThrowsWhenTryingToCallConfigurationMetho
 
             Assert.Equal("Trying to invoke a configuration method accepting a `IConfiguration` argument. " +
                          "This is not supported when only a `IConfigSection` has been provided. " +
-                         "(method 'Serilog.LoggerConfiguration DummyRollingFile(Serilog.Configuration.LoggerSinkConfiguration, Microsoft.Extensions.Configuration.IConfiguration, Microsoft.Extensions.Configuration.IConfigurationSection, System.String, Serilog.Events.LogEventLevel)')",
+                         "(method 'Serilog.LoggerConfiguration DummyWithConfiguration(Serilog.Configuration.LoggerSinkConfiguration, Microsoft.Extensions.Configuration.IConfiguration, Microsoft.Extensions.Configuration.IConfigurationSection, System.String, Serilog.Events.LogEventLevel)')",
                 exception.Message);
 
         }

test/TestDummies/DummyConfigurationSink.cs

@@ -0,0 +1,46 @@
+using System;
+using System.Collections.Generic;
+using Microsoft.Extensions.Configuration;
+using Serilog.Core;
+using Serilog.Events;
+
+namespace TestDummies
+{
+    public class DummyConfigurationSink : ILogEventSink
+    {
+        [ThreadStatic]
+        static List<LogEvent> _emitted;
+
+        [ThreadStatic]
+        static IConfiguration _configuration;
+
+        [ThreadStatic]
+        static IConfigurationSection _configSection;
+
+        public static List<LogEvent> Emitted => _emitted ?? (_emitted = new List<LogEvent>());
+
+        public static IConfiguration Configuration => _configuration;
+
+        public static IConfigurationSection ConfigSection => _configSection;
+
+
+        public DummyConfigurationSink(IConfiguration configuration, IConfigurationSection configSection)
+        {
+            _configuration = configuration;
+            _configSection = configSection;
+        }
+
+        public void Emit(LogEvent logEvent)
+        {
+            Emitted.Add(logEvent);
+        }
+
+        public static void Reset()
+        {
+            _emitted = null;
+            _configuration = null;
+            _configSection = null;
+        }
+
+    }
+}

test/TestDummies/DummyLoggerConfigurationExtensions.cs

@@ -37,14 +37,14 @@ public static LoggerConfiguration DummyRollingFile(
             return loggerSinkConfiguration.Sink(new DummyRollingFileSink(), restrictedToMinimumLevel);
         }
 
-        public static LoggerConfiguration DummyRollingFile(
+        public static LoggerConfiguration DummyWithConfiguration(
             this LoggerSinkConfiguration loggerSinkConfiguration,
             IConfiguration appConfiguration,
             IConfigurationSection configurationSection,
             string pathFormat,
             LogEventLevel restrictedToMinimumLevel = LevelAlias.Minimum)
         {
-            return loggerSinkConfiguration.Sink(new DummyRollingFileSink(), restrictedToMinimumLevel);
+            return loggerSinkConfiguration.Sink(new DummyConfigurationSink(appConfiguration, configurationSection), restrictedToMinimumLevel);
         }
 
         public static LoggerConfiguration DummyRollingFile(