Updated BindableSentryAspNetCoreOptions to pass tests

Author: jamescrosswell

src/Sentry.AspNetCore/BindableSentryAspNetCoreOptions.cs

@@ -19,6 +19,7 @@ internal class BindableSentryAspNetCoreOptions : BindableSentryLoggingOptions
     public bool? FlushOnCompletedRequest { get; set; }
     public bool? FlushBeforeRequestCompleted { get; set; }
     public bool? AdjustStandardEnvironmentNameCasing { get; set; }
+    public bool? AutoRegisterTracing { get; set; }
 
     public void ApplyTo(SentryAspNetCoreOptions options)
     {
@@ -28,5 +29,6 @@ public void ApplyTo(SentryAspNetCoreOptions options)
         options.FlushOnCompletedRequest = FlushOnCompletedRequest ?? options.FlushOnCompletedRequest;
         options.FlushBeforeRequestCompleted = FlushBeforeRequestCompleted ?? options.FlushBeforeRequestCompleted;
         options.AdjustStandardEnvironmentNameCasing = AdjustStandardEnvironmentNameCasing ?? options.AdjustStandardEnvironmentNameCasing;
+        options.AutoRegisterTracing = AutoRegisterTracing ?? options.AutoRegisterTracing;
     }
 }

test/Sentry.AspNetCore.Tests/SentryTracingBuilderTests.cs

@@ -45,7 +45,11 @@ public void UseRouting_AutoRegisterTracingDisabled_SentryTracingNotRegistered()
         // Arrange
         _fixture.ConfigureServices = services => services.AddRouting();
         _fixture.Configure = applicationBuilder => applicationBuilder.UseRouting();
-        _fixture.ConfigureOptions = options => options.AutoRegisterTracing = false;
+        _fixture.ConfigureOptions = options =>
+        {
+            options.Dsn = Sentry.Constants.DisableSdkDsnValue;
+            options.AutoRegisterTracing = false;
+        };
 
         // Act - implicit
         var (_, builder) = _fixture.GetSut();
@@ -60,7 +64,11 @@ public void UseRouting_OtelInstrumentation_SentryTracingNotRegistered()
         // Arrange
         _fixture.ConfigureServices = services => services.AddRouting();
         _fixture.Configure = applicationBuilder => applicationBuilder.UseRouting();
-        _fixture.ConfigureOptions = options => options.Instrumenter = Instrumenter.OpenTelemetry;
+        _fixture.ConfigureOptions = options =>
+        {
+            options.Dsn = Sentry.Constants.DisableSdkDsnValue;
+            options.Instrumenter = Instrumenter.OpenTelemetry;
+        };
 
         // Act - implicit
         var (_, builder) = _fixture.GetSut();
@@ -82,6 +90,10 @@ public void UseRouting_SentryTracingRegisteredWithoutWarning()
             services.AddRouting();
         };
         _fixture.Configure = applicationBuilder => applicationBuilder.UseRouting();
+        _fixture.ConfigureOptions = options =>
+        {
+            options.Dsn = Sentry.Constants.DisableSdkDsnValue;
+        };
 
         // Act
         var (_, builder) = _fixture.GetSut();
@@ -107,6 +119,10 @@ public void UseSentryTracing_AutoRegisterTracing_Warning()
             applicationBuilder.UseRouting();
             applicationBuilder.UseSentryTracing();
         };
+        _fixture.ConfigureOptions = options =>
+        {
+            options.Dsn = Sentry.Constants.DisableSdkDsnValue;
+        };
 
         // Act
         var _ = _fixture.GetSut();