Throw InvalidOperationException when IConfiguration is not available

but user tries to invoke a config method accepting a `IConfiguration`so that it is obvious that something is wrong.

Probably accidentally fixes another issue

Author: tsimbalar

src/Serilog.Settings.Configuration/Settings/Configuration/ConfigurationReader.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Generic;
 using System.Linq;
 using System.Reflection;
@@ -331,7 +331,15 @@ static void CallConfigurationMethods(ILookup<string, Dictionary<string, IConfigu
                                 select directive.Key == null ? p.DefaultValue : directive.Value.ConvertTo(p.ParameterType, declaredLevelSwitches)).ToList();
 
                     var parm = methodInfo.GetParameters().FirstOrDefault(i => i.ParameterType == typeof(IConfiguration));
-                    if (parm != null) call[parm.Position - 1] = _configuration;
+                    if (parm != null)
+                    {
+                        if (_configuration is null)
+                        {
+                            throw new InvalidOperationException("Trying to invoke a configuration method accepting a `IConfiguration` argument. " +
+                                                                $"This is not supported when only a `IConfigSection` has been provided. (method '{methodInfo}')");
+                        }
+                        call[parm.Position - 1] = _configuration;
+                    }
 
                     call.Insert(0, receiver);
 
@@ -348,7 +356,11 @@ internal static MethodInfo SelectConfigurationMethod(IEnumerable<MethodInfo> can
             return candidateMethods
                 .Where(m => m.Name == name &&
                             m.GetParameters().Skip(1)
-                            .All(p => p.HasDefaultValue || suppliedArgumentValues.Any(s => s.Key.Equals(p.Name, StringComparison.OrdinalIgnoreCase))))
+                            .All(p => p.HasDefaultValue
+                                      || suppliedArgumentValues.Any(s => s.Key.Equals(p.Name, StringComparison.OrdinalIgnoreCase))
+                                      // parameters of type IConfiguration are implicitly populated with provided Configuration
+                                      || p.ParameterType == typeof(IConfiguration)
+                                      ))
                 .OrderByDescending(m =>
                 {
                     var matchingArgs = m.GetParameters().Where(p => suppliedArgumentValues.Any(s => s.Key.Equals(p.Name, StringComparison.OrdinalIgnoreCase))).ToList();

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

@@ -45,5 +45,38 @@ public void ReadFromConfigurationSectionReadsFromAnArbitrarySection()
             Assert.NotNull(evt);
             Assert.Equal("Test", evt.Properties["App"].LiteralValue());
         }
+
+
+
+        [Fact]
+        [Trait("BugFix", "https://github.com/serilog/serilog-settings-configuration/issues/143")]
+        public void ReadFromConfigurationSectionThrowsWhenTryingToCallConfigurationMethodWithIConfigurationParam()
+        {
+            var json = @"{
+                ""NotSerilog"": {            
+                    ""Using"": [""TestDummies""],
+                    ""WriteTo"": [{
+                        ""Name"": ""DummyRollingFile"",
+                        ""Args"": {""pathFormat"" : ""C:\\"",
+                                   ""configurationSection"" : { ""foo"" : ""bar"" } }
+                    }]        
+                }
+            }";
+
+            var config = new ConfigurationBuilder()
+                .AddJsonString(json)
+                .Build();
+
+            var exception = Assert.Throws<InvalidOperationException>(() =>
+               new LoggerConfiguration()
+                   .ReadFrom.ConfigurationSection(config.GetSection("NotSerilog"))
+                   .CreateLogger());
+
+            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)')",
+                exception.Message);
+
+        }
     }
 }