Refactor: Replace direct EventCallbackFactory usage with EventCallback.Factory in debouncer tests

Author: AnnaSasDev

tests/Tests.CodeOfChaos.Extensions.AspNetCore/EventCallbacks/EventCallbackDebouncerGenericTests.cs

@@ -8,13 +8,12 @@ namespace Tests.CodeOfChaos.Extensions.AspNetCore.EventCallbacks;
 // Code
 // ---------------------------------------------------------------------------------------------------------------------
 public class EventCallbackDebouncerGenericTests {
-    private readonly EventCallbackFactory _factory = new();
 
     [Test]
     public async Task GenericDebouncer_ShouldPassValueToCallback() {
         // Arrange
         string? receivedValue = null;
-        EventCallback<string> callback = _factory.Create<string>(this, callback: value => {
+        EventCallback<string> callback = EventCallback.Factory.Create<string>(this, callback: value => {
             receivedValue = value;
             return Task.CompletedTask;
         });
@@ -32,7 +31,7 @@ public async Task GenericDebouncer_ShouldPassValueToCallback() {
     public async Task GenericDebouncer_DefaultValue_ShouldWork() {
         // Arrange
         string receivedValue = "initial";
-        EventCallback<string> callback = _factory.Create<string>(this, callback: value => {
+        EventCallback<string> callback = EventCallback.Factory.Create<string>(this, callback: value => {
             receivedValue = value;
             return Task.CompletedTask;
         });
@@ -50,7 +49,7 @@ public async Task GenericDebouncer_DefaultValue_ShouldWork() {
     public async Task GenericDebouncer_MultipleValues_ShouldUseLastValue() {
         // Arrange
         string? receivedValue = null;
-        EventCallback<string> callback = _factory.Create<string>(this, callback: value => {
+        EventCallback<string> callback = EventCallback.Factory.Create<string>(this, callback: value => {
             receivedValue = value;
             return Task.CompletedTask;
         });
@@ -70,7 +69,7 @@ public async Task GenericDebouncer_MultipleValues_ShouldUseLastValue() {
     public async Task GenericDebouncer_ConcurrentValues_ShouldBeThreadSafe() {
         // Arrange
         var receivedValues = new List<string>();
-        EventCallback<string> callback = _factory.Create<string>(this, callback: value => {
+        EventCallback<string> callback = EventCallback.Factory.Create<string>(this, callback: value => {
             receivedValues.Add(value);
             return Task.CompletedTask;
         });
@@ -91,21 +90,22 @@ public async Task GenericDebouncer_ConcurrentValues_ShouldBeThreadSafe() {
     [Test]
     public async Task GenericDebouncer_AfterDispose_ShouldThrowObjectDisposedException() {
         // Arrange
-        EventCallback<string> callback = _factory.Create<string>(this, callback: _ => Task.CompletedTask);
+        EventCallback<string> callback = EventCallback.Factory.Create<string>(this, callback: _ => Task.CompletedTask);
         var debouncer = new EventCallbackDebouncer<string>(callback);
 
         // Act
         await debouncer.DisposeAsync();
 
         // Assert
         await Assert.ThrowsAsync<ObjectDisposedException>(async () =>
-            await debouncer.InvokeDebouncedAsync("test"));
+            await debouncer.InvokeDebouncedAsync("test")
+        );
     }
 
     [Test]
     public async Task GenericDebouncer_MultipleDispose_ShouldBeIdempotent() {
         // Arrange
-        EventCallback<string> callback = _factory.Create<string>(this, callback: _ => Task.CompletedTask);
+        EventCallback<string> callback = EventCallback.Factory.Create<string>(this, callback: _ => Task.CompletedTask);
         var debouncer = new EventCallbackDebouncer<string>(callback);
 
         // Act & Assert

tests/Tests.CodeOfChaos.Extensions.AspNetCore/EventCallbacks/EventCallbackDebouncerTests.cs

@@ -8,13 +8,12 @@ namespace Tests.CodeOfChaos.Extensions.AspNetCore.EventCallbacks;
 // Code
 // ---------------------------------------------------------------------------------------------------------------------
 public class EventCallbackDebouncerTests {
-    private readonly EventCallbackFactory _factory = new();
 
     [Test]
     public async Task DefaultDebounceMs_ShouldBe100() {
         // Arrange
         int callCount = 0;
-        EventCallback callback = _factory.Create(this, callback: () => {
+        EventCallback callback = EventCallback.Factory.Create(this, callback: () => {
             callCount++;
             return Task.CompletedTask;
         });
@@ -32,7 +31,7 @@ public async Task DefaultDebounceMs_ShouldBe100() {
     public async Task CustomDebounceMs_ShouldRespectSpecifiedTime() {
         // Arrange
         int callCount = 0;
-        EventCallback callback = _factory.Create(this, callback: () => {
+        EventCallback callback = EventCallback.Factory.Create(this, callback: () => {
             callCount++;
             return Task.CompletedTask;
         });
@@ -56,7 +55,7 @@ public async Task CustomDebounceMs_ShouldRespectSpecifiedTime() {
     public async Task MultipleInvocations_ShouldDebounce() {
         // Arrange
         int callCount = 0;
-        EventCallback callback = _factory.Create(this, callback: () => {
+        EventCallback callback = EventCallback.Factory.Create(this, callback: () => {
             callCount++;
             return Task.CompletedTask;
         });
@@ -76,7 +75,7 @@ public async Task MultipleInvocations_ShouldDebounce() {
     public async Task ConcurrentInvocations_ShouldBeThreadSafe() {
         // Arrange
         int callCount = 0;
-        EventCallback callback = _factory.Create(this, callback: () => {
+        EventCallback callback = EventCallback.Factory.Create(this, callback: () => {
             callCount++;
             return Task.CompletedTask;
         });
@@ -97,7 +96,7 @@ public async Task ConcurrentInvocations_ShouldBeThreadSafe() {
     [Test]
     public async Task AfterDispose_ShouldThrowObjectDisposedException() {
         // Arrange
-        EventCallback callback = _factory.Create(this, callback: () => Task.CompletedTask);
+        EventCallback callback = EventCallback.Factory.Create(this, callback: () => Task.CompletedTask);
         var debouncer = new EventCallbackDebouncer(callback);
 
         // Act
@@ -111,7 +110,7 @@ await Assert.ThrowsAsync<ObjectDisposedException>(async () =>
     [Test]
     public async Task MultipleDispose_ShouldBeIdempotent() {
         // Arrange
-        EventCallback callback = _factory.Create(this, callback: () => Task.CompletedTask);
+        EventCallback callback = EventCallback.Factory.Create(this, callback: () => Task.CompletedTask);
         var debouncer = new EventCallbackDebouncer(callback);
 
         // Act & Assert
@@ -123,15 +122,15 @@ public async Task MultipleDispose_ShouldBeIdempotent() {
     public async Task InvocationDuringDebounce_ShouldCancelPrevious() {
         // Arrange
         var executionTimes = new List<DateTime>();
-        EventCallback callback = _factory.Create(this, callback: () => {
+        EventCallback callback = EventCallback.Factory.Create(this, callback: () => {
             executionTimes.Add(DateTime.UtcNow);
             return Task.CompletedTask;
         });
 
         // Act
         await using var debouncer = new EventCallbackDebouncer(callback);
         await debouncer.InvokeDebouncedAsync();
-        await Task.Delay(50); // Wait half the debounced time
+        await Task.Delay(50);// Wait half the debounced time
         await debouncer.InvokeDebouncedAsync();
         await Task.Delay(150);