fix: 修正单元测试

Author: stratosblue

test/Cuture.AspNetCore.ActionAuditing.Test/AsyncAuditDataStorageTest.cs

@@ -17,25 +17,56 @@ public class AsyncAuditDataStorageTest
     /// </summary>
     public class TestAsyncAuditDataStorage : AsyncAuditDataStorage<string>
     {
+        #region Public 构造函数
+
         public TestAsyncAuditDataStorage(ILogger<AsyncAuditDataStorage<string>> logger) : base(logger)
         {
         }
 
-        protected override ValueTask<string> CreateDataAsync(ActionAuditingExecutingContext context, CancellationToken cancellationToken)
-        {
-            return ValueTask.FromResult("testData");
-        }
+        #endregion Public 构造函数
+
+        #region Protected Internal 方法
 
         protected internal override Task SaveDataAsync(string data, CancellationToken cancellationToken)
         {
             return Task.CompletedTask;
         }
+
+        #endregion Protected Internal 方法
+
+        #region Protected 方法
+
+        protected override ValueTask<string> CreateDataAsync(ActionAuditingExecutingContext context, CancellationToken cancellationToken)
+        {
+            return ValueTask.FromResult("testData");
+        }
+
+        #endregion Protected 方法
     }
 
-    #endregion
+    #endregion 测试辅助类
 
     #region 测试方法
 
+    /// <summary>
+    /// 测试AddAsync方法是否正确写入数据到通道
+    /// </summary>
+    [TestMethod]
+    public async Task AddAsync_ShouldWriteToChannel()
+    {
+        // Arrange
+        var mockLogger = new Mock<ILogger<AsyncAuditDataStorage<string>>>();
+        var storage = new TestAsyncAuditDataStorage(mockLogger.Object);
+        var mockContext = new Mock<ActionAuditingExecutingContext>(null, null, null, null);
+
+        // Act
+        await storage.AddAsync(mockContext.Object, CancellationToken.None);
+
+        // Assert
+        var result = await storage.DataChannel.Reader.ReadAsync();
+        Assert.AreEqual("testData", result);
+    }
+
     /// <summary>
     /// 测试构造函数是否正确初始化
     /// </summary>
@@ -54,36 +85,35 @@ public void Constructor_ShouldInitializeCorrectly()
     }
 
     /// <summary>
-    /// 测试AddAsync方法是否正确写入数据到通道
+    /// 测试Dispose方法是否取消运行令牌
     /// </summary>
     [TestMethod]
-    public async Task AddAsync_ShouldWriteToChannel()
+    public void Dispose_ShouldCancelRunningToken()
     {
         // Arrange
         var mockLogger = new Mock<ILogger<AsyncAuditDataStorage<string>>>();
         var storage = new TestAsyncAuditDataStorage(mockLogger.Object);
-        var mockContext = new Mock<ActionAuditingExecutingContext>(null, null, null, null);
 
         // Act
-        await storage.AddAsync(mockContext.Object, CancellationToken.None);
+        storage.Dispose();
 
         // Assert
-        var result = await storage.DataChannel.Reader.ReadAsync();
-        Assert.AreEqual("testData", result);
+        Assert.IsTrue(storage.RunningCancellationToken.IsCancellationRequested);
     }
 
     /// <summary>
-    /// 测试Dispose方法是否取消运行令牌
+    /// 测试Dispose方法是否处理多次调用
     /// </summary>
     [TestMethod]
-    public void Dispose_ShouldCancelRunningToken()
+    public void Dispose_ShouldHandleMultipleCalls()
     {
         // Arrange
         var mockLogger = new Mock<ILogger<AsyncAuditDataStorage<string>>>();
         var storage = new TestAsyncAuditDataStorage(mockLogger.Object);
 
         // Act
         storage.Dispose();
+        storage.Dispose(); // 第二次调用
 
         // Assert
         Assert.IsTrue(storage.RunningCancellationToken.IsCancellationRequested);
@@ -97,6 +127,9 @@ public async Task RunSaveLoopAsync_ShouldLogError_WhenSaveFails()
     {
         // Arrange
         var mockLogger = new Mock<ILogger<AsyncAuditDataStorage<string>>>();
+        mockLogger.Setup(x => x.IsEnabled(LogLevel.Warning))
+                  .Returns(true);
+
         var storage = new Mock<TestAsyncAuditDataStorage>(mockLogger.Object) { CallBase = true };
         storage.Setup(x => x.SaveDataAsync(It.IsAny<string>(), It.IsAny<CancellationToken>()))
                .ThrowsAsync(new Exception("Test exception"));
@@ -141,23 +174,5 @@ public async Task RunSaveLoopAsync_ShouldLogWarning_WhenCancelled()
             Times.AtLeastOnce);
     }
 
-    /// <summary>
-    /// 测试Dispose方法是否处理多次调用
-    /// </summary>
-    [TestMethod]
-    public void Dispose_ShouldHandleMultipleCalls()
-    {
-        // Arrange
-        var mockLogger = new Mock<ILogger<AsyncAuditDataStorage<string>>>();
-        var storage = new TestAsyncAuditDataStorage(mockLogger.Object);
-
-        // Act
-        storage.Dispose();
-        storage.Dispose(); // 第二次调用
-
-        // Assert
-        Assert.IsTrue(storage.RunningCancellationToken.IsCancellationRequested);
-    }
-
-    #endregion
+    #endregion 测试方法
 }

test/Cuture.AspNetCore.ActionAuditing.Test/DefaultActionAuditingFilterTest.cs

@@ -64,6 +64,13 @@ public async Task Should_Return_True_When_HasPermissionProvider()
         Assert.IsTrue(result);
     }
 
+    [TestInitialize]
+    public void TestInitialize()
+    {
+        _loggerMock.Setup(x => x.IsEnabled(LogLevel.Warning))
+                   .Returns(true);
+    }
+
     #endregion Public 方法
 
     #region Private 方法