adding test that log context properties are included (#28)

Author: Ivan Gavryliuk

src/Serilog.Sinks.ApplicationInsights.Tests/ApplicationInsightsTest.cs

@@ -41,6 +41,7 @@ protected ApplicationInsightsTest()
             Logger = new LoggerConfiguration()
                 .WriteTo.ApplicationInsightsTraces(tc)
                 .MinimumLevel.Debug()
+                .Enrich.FromLogContext()
                 .CreateLogger();
         }
 

src/Serilog.Sinks.ApplicationInsights.Tests/FormattingTests.cs

@@ -1,4 +1,5 @@
 using Microsoft.ApplicationInsights.Channel;
+using Serilog.Context;
 using Xunit;
 
 namespace Serilog.Sinks.ApplicationInsights.Tests
@@ -20,5 +21,16 @@ public void Message_template_is_not_in_trace_custom_property()
 
             Assert.False(LastSubmittedTraceTelemetry.Properties.ContainsKey("MessageTemplate"));
         }
+
+        [Fact]
+        public void Message_properies_include_log_context()
+        {
+            using (LogContext.PushProperty("custom1", "value1"))
+            {
+                Logger.Information("test context");
+
+                Assert.True(LastSubmittedTraceTelemetry.Properties.TryGetValue("custom1", out string value1) && value1 == "value1");
+            }
+        }
     }
 }

src/Serilog.Sinks.ApplicationInsights/Sinks/ApplicationInsights/ApplicationInsightsSinkBase.cs

@@ -141,7 +141,10 @@ public virtual void Emit(LogEvent logEvent)
                 {
                     foreach (ITelemetry telemetry in telemetries)
                     {
-                        TrackTelemetry(telemetry);
+                        if (telemetry != null)
+                        {
+                            TrackTelemetry(telemetry);
+                        }
                     }
                 }
             }