#66 : Always initialize DaprOptions to ensure valid fluent builder in…

…italization
man-group · Oct 27, 2024 · e4f7886 · e4f7886
1 parent 97b3920
commit e4f7886
Show file tree

Hide file tree

Showing 5 changed files with 8 additions and 8 deletions.
diff --git a/src/Man.Dapr.Sidekick/Options/DaprOptions.cs b/src/Man.Dapr.Sidekick/Options/DaprOptions.cs
@@ -11,13 +11,13 @@ public DaprOptions()
             LogLevel = Process.DaprProcessLogger.DebugLevel;
         }
 
-        public DaprSidecarOptions Sidecar { get; set; }
+        public DaprSidecarOptions Sidecar { get; set; } = new();
 
-        public DaprPlacementOptions Placement { get; set; }
+        public DaprPlacementOptions Placement { get; set; } = new();
 
-        public DaprSchedulerOptions Scheduler { get; set; }
+        public DaprSchedulerOptions Scheduler { get; set; } = new();
 
-        public DaprSentryOptions Sentry { get; set; }
+        public DaprSentryOptions Sentry { get; set; } = new();
 
         /// <summary>
         /// Creates a deep clone of this instance.

diff --git a/tests/Man.Dapr.Sidekick.Tests/Process/DaprPlacementProcessTests.cs b/tests/Man.Dapr.Sidekick.Tests/Process/DaprPlacementProcessTests.cs
@@ -16,7 +16,7 @@ public void Should_create_placement_section_if_null()
                 var p = new MockDaprPlacementProcess();
                 var options = new DaprOptions();
 
-                Assert.That(options.Placement, Is.Null);
+                Assert.That(options.Placement, Is.Not.Null);
 
                 var newOptions = p.GetProcessOptions(options);
                 Assert.That(newOptions, Is.Not.Null);

diff --git a/tests/Man.Dapr.Sidekick.Tests/Process/DaprSchedulerProcessTests.cs b/tests/Man.Dapr.Sidekick.Tests/Process/DaprSchedulerProcessTests.cs
@@ -15,7 +15,7 @@ public void Should_create_scheduler_section_if_null()
                 var p = new MockDaprSchedulerProcess();
                 var options = new DaprOptions();
 
-                Assert.That(options.Scheduler, Is.Null);
+                Assert.That(options.Scheduler, Is.Not.Null);
 
                 var newOptions = p.GetProcessOptions(options);
                 Assert.That(newOptions, Is.Not.Null);

diff --git a/tests/Man.Dapr.Sidekick.Tests/Process/DaprSentryProcessTests.cs b/tests/Man.Dapr.Sidekick.Tests/Process/DaprSentryProcessTests.cs
@@ -15,7 +15,7 @@ public void Should_create_sentry_section_if_null()
                 var p = new MockDaprSentryProcess();
                 var options = new DaprOptions();
 
-                Assert.That(options.Sentry, Is.Null);
+                Assert.That(options.Sentry, Is.Not.Null);
 
                 var newOptions = p.GetProcessOptions(options);
                 Assert.That(newOptions, Is.Not.Null);

diff --git a/tests/Man.Dapr.Sidekick.Tests/Process/DaprSidecarProcessTests.cs b/tests/Man.Dapr.Sidekick.Tests/Process/DaprSidecarProcessTests.cs
@@ -17,7 +17,7 @@ public void Should_assign_defaults()
                 var p = new MockDaprSidecarProcess();
                 var options = new DaprOptions();
 
-                Assert.That(options.Sidecar, Is.Null);
+                Assert.That(options.Sidecar, Is.Not.Null);
 
                 var newOptions = p.GetProcessOptions(options);
                 Assert.That(newOptions, Is.Not.Null);