fix tests for new Enabled validation

Author: hjgraca

libraries/src/AWS.Lambda.Powertools.Logging/PowertoolsLoggerConfiguration.cs

@@ -226,12 +226,14 @@ public JsonSerializerOptions JsonOptions
         }
     }
 
+    private LogBufferingOptions _logBuffering = new LogBufferingOptions();
+
     /// <summary>
-    /// Configuration options for log buffering. Logs below the specified level will be buffered
-    /// until the buffer is flushed or an error occurs.
-    /// Buffer logs at the WARNING, INFO, and DEBUG levels and reduce CloudWatch costs by decreasing the number of emitted log messages.
-    /// <para></para>
-    /// Log buffering is disabled by default. Set <see cref="LogBufferingOptions.Enabled"/> to true to enable it.
+    ///     Gets or sets the configuration options for log buffering.
+    ///     When enabled, logs below the specified level are buffered until explicitly flushed or an error occurs.
+    ///     This can reduce CloudWatch costs by decreasing the number of emitted log messages.
+    ///     <para>Log buffering is disabled by default. Set <see cref="LogBufferingOptions.Enabled"/> to true to enable it.</para>
+    ///     <para>Setting this property to null will reset it to a default <see cref="LogBufferingOptions"/> instance with buffering disabled.</para>
     /// </summary>
     /// <example>
     ///     <code>
@@ -248,8 +250,6 @@ public JsonSerializerOptions JsonOptions
     ///     options.LogBuffering.BufferAtLogLevel = LogLevel.Warning;
     ///     </code>
     /// </example>
-    private LogBufferingOptions _logBuffering = new LogBufferingOptions();
-
     public LogBufferingOptions LogBuffering
     {
         get => _logBuffering;

libraries/tests/AWS.Lambda.Powertools.Logging.Tests/Attributes/LoggerAspectTests.cs

@@ -546,17 +546,17 @@ public void OnEntry_WhenBufferingDisabled_ShouldSetBufferingEnabledFlagToFalse()
     }
 
     [Fact]
-    public void OnEntry_WhenLogBufferingIsNull_ShouldSetBufferingEnabledFlagToFalse()
+    public void OnEntry_WhenLogBufferingIsSetToNull_ShouldDefaultToDisabledBuffering()
     {
-        // Arrange - This test covers the null path: _bufferingEnabled = _currentConfig.LogBuffering?.Enabled == true;
+        // Arrange - This test covers the null assignment path: LogBuffering setter defaults to new LogBufferingOptions() when null
         var consoleOut = Substitute.For<IConsoleWrapper>();
 
         var config = new PowertoolsLoggerConfiguration
         {
             Service = "TestService",
             MinimumLogLevel = LogLevel.Information,
             LogOutput = consoleOut,
-            LogBuffering = null
+            LogBuffering = null // This will be converted to a default LogBufferingOptions with Enabled = false
         };
 
         PowertoolsLoggingBuilderExtensions.UpdateConfiguration(config);
@@ -581,9 +581,10 @@ public void OnEntry_WhenLogBufferingIsNull_ShouldSetBufferingEnabledFlagToFalse(
         var loggingAspect = new LoggingAspect(logger);
         loggingAspect.OnEntry(aspectArgs);
 
-        // Assert
+        // Assert - LogBuffering should never be null, but Enabled should be false by default
         var updatedConfig = PowertoolsLoggingBuilderExtensions.GetCurrentConfiguration();
-        Assert.Null(updatedConfig.LogBuffering);
+        Assert.NotNull(updatedConfig.LogBuffering);
+        Assert.False(updatedConfig.LogBuffering.Enabled);
     }
 
     public void Dispose()