Added asserts.

Author: CodeBlanch

src/Shared/Configuration/OpenTelemetryConfigurationExtensions.cs

@@ -3,6 +3,7 @@
 
 #nullable enable
 
+using System.Diagnostics;
 #if !NETFRAMEWORK && !NETSTANDARD2_0
 using System.Diagnostics.CodeAnalysis;
 #endif
@@ -27,7 +28,9 @@ public static bool TryGetStringValue(
 #endif
         out string? value)
     {
-        value = configuration[key];
+        Debug.Assert(configuration != null, "configuration was null");
+
+        value = configuration![key];
 
         return !string.IsNullOrWhiteSpace(value);
     }
@@ -41,6 +44,8 @@ public static bool TryGetUriValue(
 #endif
         out Uri? value)
     {
+        Debug.Assert(logger != null, "logger was null");
+
         if (!configuration.TryGetStringValue(key, out var stringValue))
         {
             value = null;
@@ -49,7 +54,7 @@ public static bool TryGetUriValue(
 
         if (!Uri.TryCreate(stringValue, UriKind.Absolute, out value))
         {
-            logger.LogInvalidConfigurationValue(key, stringValue!);
+            logger!.LogInvalidConfigurationValue(key, stringValue!);
             return false;
         }
 
@@ -62,6 +67,8 @@ public static bool TryGetIntValue(
         string key,
         out int value)
     {
+        Debug.Assert(logger != null, "logger was null");
+
         if (!configuration.TryGetStringValue(key, out var stringValue))
         {
             value = default;
@@ -70,7 +77,7 @@ public static bool TryGetIntValue(
 
         if (!int.TryParse(stringValue, NumberStyles.None, CultureInfo.InvariantCulture, out value))
         {
-            logger.LogInvalidConfigurationValue(key, stringValue!);
+            logger!.LogInvalidConfigurationValue(key, stringValue!);
             return false;
         }
 
@@ -83,6 +90,8 @@ public static bool TryGetBoolValue(
         string key,
         out bool value)
     {
+        Debug.Assert(logger != null, "logger was null");
+
         if (!configuration.TryGetStringValue(key, out var stringValue))
         {
             value = default;
@@ -91,7 +100,7 @@ public static bool TryGetBoolValue(
 
         if (!bool.TryParse(stringValue, out value))
         {
-            logger.LogInvalidConfigurationValue(key, stringValue!);
+            logger!.LogInvalidConfigurationValue(key, stringValue!);
             return false;
         }
 
@@ -108,6 +117,8 @@ public static bool TryGetValue<T>(
 #endif
         out T? value)
     {
+        Debug.Assert(logger != null, "logger was null");
+
         if (!configuration.TryGetStringValue(key, out var stringValue))
         {
             value = default;
@@ -116,7 +127,7 @@ public static bool TryGetValue<T>(
 
         if (!tryParseFunc(stringValue!, out value))
         {
-            logger.LogInvalidConfigurationValue(key, stringValue!);
+            logger!.LogInvalidConfigurationValue(key, stringValue!);
             return false;
         }