Refactor tests: Standardize assertion syntax and remove unused variables

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

@@ -88,7 +88,7 @@ public async Task GenericDebouncer_ConcurrentValues_ShouldBeThreadSafe() {
         await debouncer.FlushAsync();
 
         // Assert
-        await Assert.That(receivedValues).HasCount().EqualTo(1);
+        await Assert.That(receivedValues).Count().IsEqualTo(1);
         await debouncer.DisposeAsync();
     }
 

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

@@ -160,6 +160,6 @@ public async Task InvocationDuringDebounce_ShouldCancelPrevious_EventCallback()
         await debouncer.FlushAsync();
 
         // Assert
-        await Assert.That(executionTimes).HasCount().EqualTo(1);
+        await Assert.That(executionTimes).Count().IsEqualTo(1);
     }
 }

tests/Tests.CodeOfChaos.Extensions.FluentValidation/ValidatorExtensionTests.cs

@@ -54,7 +54,7 @@ public async Task TryValidate_ValidInstance_ReturnsTrueAndNoFailures() {
 
     [Test]
     public async Task TryValidate_InvalidInstance_ReturnsFalseAndFailures() {
-        bool result = _mockedValidator.TryValidate(InvalidInstance, out List<ValidationFailure>? failures);
+        bool result = _mockedValidator.TryValidate(InvalidInstance, out List<ValidationFailure> failures);
 
         await Assert.That(result).IsFalse();
         await Assert.That(failures).IsEquivalentTo(ValidationFailures);

tests/Tests.CodeOfChaos.Extensions/Debouncers/Regular/ActionDebouncerGenericTests.cs

@@ -66,7 +66,7 @@ public async Task GenericDebouncer_ConcurrentValues_ShouldBeThreadSafe() {
         await debouncer.FlushAsync();
 
         // Assert
-        await Assert.That(receivedValues).HasCount().EqualTo(1);
+        await Assert.That(receivedValues).Count().IsEqualTo(1);
         await debouncer.DisposeAsync();
     }
 

tests/Tests.CodeOfChaos.Extensions/Debouncers/Regular/ActionDebouncerTests.cs

@@ -145,6 +145,6 @@ public async Task InvocationDuringDebounce_ShouldCancelPrevious_Action() {
         await debouncer.FlushAsync();
 
         // Assert
-        await Assert.That(executionTimes).HasCount().EqualTo(1);
+        await Assert.That(executionTimes).Count().IsEqualTo(1);
     }
 }

tests/Tests.CodeOfChaos.Extensions/Debouncers/Regular/FuncDebouncerGenericTests.cs

@@ -69,7 +69,7 @@ public async Task GenericDebouncer_ConcurrentValues_ShouldBeThreadSafe() {
         await debouncer.FlushAsync();
 
         // Assert
-        await Assert.That(receivedValues).HasCount().EqualTo(1);
+        await Assert.That(receivedValues).Count().IsEqualTo(1);
         await debouncer.DisposeAsync();
     }
 

tests/Tests.CodeOfChaos.Extensions/Debouncers/Regular/FuncDebouncerTests.cs

@@ -144,7 +144,7 @@ public async Task InvocationDuringDebounce_ShouldCancelPrevious_Func() {
         await debouncer.FlushAsync();
 
         // Assert
-        await Assert.That(executionTimes).HasCount().EqualTo(1);
+        await Assert.That(executionTimes).Count().IsEqualTo(1);
     }
 
     [Test]

tests/Tests.CodeOfChaos.Extensions/Debouncers/Throttled/ThrottledDebouncerGenericTests.cs

@@ -84,7 +84,7 @@ public async Task ThrottledDebouncer_ThrottleBehavior_ShouldExecuteImmediatelyAf
         await debouncer.FlushAsync(); // Ensure second execution has completed
 
         // Assert
-        await Assert.That(receivedValues).HasCount().GreaterThanOrEqualTo(2);
+        await Assert.That(receivedValues).Count().IsGreaterThanOrEqualTo(2);
         await Assert.That(receivedValues[0].Value).IsEqualTo("first");
         await Assert.That(receivedValues[^1].Value).IsEqualTo("fourth");
 
@@ -123,7 +123,7 @@ public async Task ThrottledDebouncer_ContinuousRequests_ShouldRespectThrottleInt
 
         // Assert
         await Assert.That(executionCount).IsGreaterThan(1);
-        await Assert.That(receivedValues).HasCount().EqualTo(executionCount);
+        await Assert.That(receivedValues).Count().IsEqualTo(executionCount);
     }
 
     [Test]
@@ -171,7 +171,7 @@ await Task.WhenAll(
         // Assert
         await Assert.That(executionCount).IsGreaterThanOrEqualTo(2);
         await Assert.That(executionCount).IsLessThanOrEqualTo(3);
-        await Assert.That(receivedValues).HasCount().EqualTo(executionCount);
+        await Assert.That(receivedValues).Count().IsEqualTo(executionCount);
 
     }
 
@@ -246,7 +246,7 @@ public async Task ThrottledDebouncer_DefaultValues_ShouldUseDefaultDebounceAndTh
         await debouncer.FlushAsync();
 
         // Assert
-        await Assert.That(executionTimes).HasCount().EqualTo(1);
+        await Assert.That(executionTimes).Count().IsEqualTo(1);
         double executionDelay = (executionTimes[0] - startTime).TotalMilliseconds;
         await Assert.That(executionDelay).IsGreaterThanOrEqualTo(95); // Account for timing variations
     }

tests/Tests.CodeOfChaos.Extensions/ReflectionHelperTests.cs

@@ -1,6 +1,7 @@
 // ---------------------------------------------------------------------------------------------------------------------
 // Imports
 // ---------------------------------------------------------------------------------------------------------------------
+using System.Diagnostics.CodeAnalysis;
 using System.Reflection;
 
 namespace Tests.CodeOfChaos.Extensions;
@@ -9,7 +10,10 @@ namespace Tests.CodeOfChaos.Extensions;
 // ---------------------------------------------------------------------------------------------------------------------
 public class ReflectionHelperTests {
     private class TestClass {
-        public TestClass(string nonNullable, string? nullable, int valueType, int? nullableValueType) {}// ReSharper disable UnusedParameter.Local
+        [SuppressMessage("ReSharper", "UnusedParameter.Local")] 
+        public TestClass(string nonNullable, string? nullable, int valueType, int? nullableValueType) {}
+
+        [SuppressMessage("ReSharper", "UnusedParameter.Local")] 
         public static void MethodWithNullability(
             string nonNullable,
             string? nullable,

tests/Tests.CodeOfChaos.Extensions/SpanLINQ/SpanLINQAllTests.cs

@@ -21,7 +21,7 @@ public async Task All() {
         bool result = span.All(static x => x % 2 == 0);
 
         // Assert
-        await Assert.That(result).IsTrue();;
+        await Assert.That(result).IsTrue();
     }
 
     [Test]

tests/Tests.CodeOfChaos.Extensions/TaskHelperTests.cs

@@ -27,7 +27,6 @@ public async Task FromTaskOrDefault_ShouldReturnDefault_int(int? input, int? exp
     [Arguments(false, false)]
     public async Task FromTaskOrDefault_ShouldReturnExpected_ReferenceType(bool setAsDefault, bool expectedIsNull) {
         // Arrange
-        var expected = new TestClass();
         Task<TestClass?> task = Task.FromResult(setAsDefault ? null : new TestClass());
 
         // Act