cleanup

Author: dadhi

src/FastExpressionCompiler/TestTools.cs

@@ -564,6 +564,7 @@ public CallerArgumentExpression(string parameterName) { }
 #endif
 
 /// <summary>Wrapper for the context per test method</summary>
+[UnconditionalSuppressMessage("ReflectionAnalysis", "IL2026:RequiresUnreferencedCode", Justification = "This is used for the testing purposes only.")]
 public struct TestContext
 {
     public readonly TestRun TestRun;
@@ -600,8 +601,6 @@ public bool IsTrue(bool actual,
             Fail(testName, sourceLineNumber, AssertKind.IsTrue,
                 $"Expected `IsTrue({actualName})`, but found false");
 
-    [UnconditionalSuppressMessage("ReflectionAnalysis", "IL2026:RequiresUnreferencedCode",
-        Justification = "The method is used for the testing purposes only.")]
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
     public bool AreEqual<T>(T expected, T actual,
         [CallerArgumentExpression(nameof(expected))] string expectedName = "<expected>",
@@ -610,8 +609,6 @@ public bool AreEqual<T>(T expected, T actual,
         Equals(expected, actual) || Fail(testName, sourceLineNumber, AssertKind.AreEqual,
             $"Expected `AreEqual({expectedName}, {actualName})`, but found `{expected.ToCode()}` is Not equal to `{actual.ToCode()}`");
 
-    [UnconditionalSuppressMessage("ReflectionAnalysis", "IL2026:RequiresUnreferencedCode",
-        Justification = "The method is used for the testing purposes only.")]
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
     public static bool AreSame<T>(T expected, T actual,
         [CallerArgumentExpression(nameof(expected))]
@@ -621,8 +618,6 @@ public static bool AreSame<T>(T expected, T actual,
         ReferenceEquals(expected, actual) ? true : throw new AssertionException(
             $"Expected `AreSame({expectedName}, {actualName})`, but found `{expected.ToCode()}` is Not the same `{actual.ToCode()}`");
 
-    [UnconditionalSuppressMessage("ReflectionAnalysis", "IL2026:RequiresUnreferencedCode",
-        Justification = "The method is used for the testing purposes only.")]
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
     public static bool AreNotSame<T>(T expected, T actual,
         [CallerArgumentExpression(nameof(expected))]
@@ -632,8 +627,6 @@ public static bool AreNotSame<T>(T expected, T actual,
         !ReferenceEquals(expected, actual) ? true : throw new AssertionException(
             $"Expected `AreNotSame({expectedName}, {actualName})`, but found `{expected.ToCode()}` is same as `{actual.ToCode()}`");
 
-    [UnconditionalSuppressMessage("ReflectionAnalysis", "IL2026:RequiresUnreferencedCode",
-        Justification = "The method is used for the testing purposes only.")]
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
     public static bool AreEqual<T>(T expected, T actual,
         [CallerArgumentExpression(nameof(expected))]
@@ -643,8 +636,6 @@ public static bool AreEqual<T>(T expected, T actual,
         Equals(expected, actual) ? true : throw new AssertionException(
             $"Expected `AreEqual({expectedName}, {actualName})`, but found `{expected.ToCode()}` is Not equal to `{actual.ToCode()}`");
 
-    [UnconditionalSuppressMessage("ReflectionAnalysis", "IL2026:RequiresUnreferencedCode",
-        Justification = "The method is used for the testing purposes only.")]
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
     public static bool AreNotEqual<T>(T expected, T actual,
         [CallerArgumentExpression(nameof(expected))] string expectedName = "expected", [CallerArgumentExpression(nameof(actual))]
@@ -655,8 +646,6 @@ public static bool AreNotEqual<T>(T expected, T actual,
     public record struct ItemsCompared<T>(int Index, bool IsEqual, T Expected, T Actual);
 
     /// <summary>Should cover the case with the `expected` to be an array as well.</summary>
-    [UnconditionalSuppressMessage("ReflectionAnalysis", "IL2026:RequiresUnreferencedCode",
-        Justification = "The method is used for the testing purposes only.")]
     public static bool AreEqual<T>(IEnumerable<T> expected, IEnumerable<T> actual,
         [CallerArgumentExpression(nameof(expected))] string expectedName = "expected",
         [CallerArgumentExpression(nameof(actual))] string actualName = "actual")
@@ -805,15 +794,14 @@ public static bool AreEqual<T>(IEnumerable<T> expected, IEnumerable<T> actual,
         return true;
     }
 
+    [MethodImpl(MethodImplOptions.AggressiveInlining)]
     public static bool AreEqual<T>(T[] expected, T[] actual,
         [CallerArgumentExpression(nameof(expected))]
         string expectedName = "expected",
         [CallerArgumentExpression(nameof(actual))]
         string actualName = "actual") =>
         AreEqual((IEnumerable<T>)expected, actual, expectedName, actualName);
 
-    [UnconditionalSuppressMessage("ReflectionAnalysis", "IL2026:RequiresUnreferencedCode",
-        Justification = "The method is used for the testing purposes only.")]
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
     public static bool GreaterOrEqual<T>(T expected, T actual,
         [CallerArgumentExpression(nameof(expected))]
@@ -824,8 +812,6 @@ public static bool GreaterOrEqual<T>(T expected, T actual,
         expected.CompareTo(actual) >= 0 ? true : throw new AssertionException(
             $"Expected `GreaterOrEqual({expectedName}, {actualName})`, but found `{expected.ToCode()} < {actual.ToCode()}`");
 
-    [UnconditionalSuppressMessage("ReflectionAnalysis", "IL2026:RequiresUnreferencedCode",
-        Justification = "The method is used for the testing purposes only.")]
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
     public static bool Less<T>(T expected, T actual,
         [CallerArgumentExpression(nameof(expected))]
@@ -836,8 +822,6 @@ public static bool Less<T>(T expected, T actual,
         expected.CompareTo(actual) < 0 ? true : throw new AssertionException(
             $"Expected `Less({expectedName}, {actualName})`, but found `{expected.ToCode()} >= {actual.ToCode()}`");
 
-    [UnconditionalSuppressMessage("ReflectionAnalysis", "IL2026:RequiresUnreferencedCode",
-        Justification = "The method is used for the testing purposes only.")]
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
     public static bool Greater<T>(T expected, T actual,
         [CallerArgumentExpression(nameof(expected))]
@@ -848,8 +832,6 @@ public static bool Greater<T>(T expected, T actual,
         expected.CompareTo(actual) > 0 ? true : throw new AssertionException(
             $"Expected `Greater({expectedName}, {actualName})`, but found `{expected.ToCode()} <= {actual.ToCode()}`");
 
-    [UnconditionalSuppressMessage("ReflectionAnalysis", "IL2026:RequiresUnreferencedCode",
-        Justification = "The method is used for the testing purposes only.")]
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
     public static bool LessOrEqual<T>(T expected, T actual,
         [CallerArgumentExpression(nameof(expected))]
@@ -860,8 +842,6 @@ public static bool LessOrEqual<T>(T expected, T actual,
         expected.CompareTo(actual) <= 0 ? true : throw new AssertionException(
             $"Expected `LessOrEqual({expectedName}, {actualName})`, but found `{expected.ToCode()} > {actual.ToCode()}`");
 
-    [UnconditionalSuppressMessage("ReflectionAnalysis", "IL2026:RequiresUnreferencedCode",
-        Justification = "The method is used for the testing purposes only.")]
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
     public bool IsNull<T>(T actual,
         [CallerArgumentExpression(nameof(actual))] string actualName = "actual",
@@ -904,17 +884,13 @@ public bool IsFalse(bool actual,
         !actual || Fail(testName, sourceLineNumber, AssertKind.IsFalse,
             $"Expected `IsFalse({actualName})`, but found true");
 
-    [UnconditionalSuppressMessage("ReflectionAnalysis", "IL2026:RequiresUnreferencedCode",
-        Justification = "The method is used for the testing purposes only.")]
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
     public static bool IsInstanceOf<T>(object actual,
         [CallerArgumentExpression(nameof(actual))]
         string actualName = "actual") =>
         actual is T ? true : throw new AssertionException(
             $"Expected `IsInstanceOf<{typeof(T).ToCode()}>({actualName})`, but found `IsInstanceOf<{actual?.GetType().ToCode() ?? "_"}>({actual.ToCode()})`");
 
-    [UnconditionalSuppressMessage("ReflectionAnalysis", "IL2026:RequiresUnreferencedCode",
-        Justification = "The method is used for the testing purposes only.")]
     public static E Throws<E>(Action action,
         [CallerArgumentExpression(nameof(action))]
         string actionName = "<action to throw>")
@@ -936,8 +912,6 @@ public static E Throws<E>(Action action,
         throw new AssertionException($"Expected `Throws<{typeof(E).ToCode()}>({actionName})`, but no exception was thrown");
     }
 
-    [UnconditionalSuppressMessage("ReflectionAnalysis", "IL2026:RequiresUnreferencedCode",
-        Justification = "The method is used for the testing purposes only.")]
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
     public static bool Contains(string expected, string actual,
         [CallerArgumentExpression(nameof(expected))]
@@ -947,8 +921,6 @@ public static bool Contains(string expected, string actual,
         actual.Contains(expected) ? true : throw new AssertionException(
             $"Expected string `Contains({expectedName}, {actualName})`, but found expected `{expected.ToCode()}` is not in `{actual.ToCode()}`");
 
-    [UnconditionalSuppressMessage("ReflectionAnalysis", "IL2026:RequiresUnreferencedCode",
-        Justification = "The method is used for the testing purposes only.")]
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
     public static bool DoesNotContain(string expected, string actual,
         [CallerArgumentExpression(nameof(expected))]
@@ -958,8 +930,6 @@ public static bool DoesNotContain(string expected, string actual,
         !actual.Contains(expected) ? true : throw new AssertionException(
             $"Expected string `DoesNotContain({expectedName}, {actualName})`, but found expected `{expected.ToCode()}` is in `{actual.ToCode()}`");
 
-    [UnconditionalSuppressMessage("ReflectionAnalysis", "IL2026:RequiresUnreferencedCode",
-        Justification = "The method is used for the testing purposes only.")]
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
     public static bool StartsWith(string expected, string actual,
         [CallerArgumentExpression(nameof(expected))]
@@ -1030,4 +1000,4 @@ public void Run<T>(T test, TestTracking tracking = TestTracking.TrackFailedTests
     }
 }
 
-#pragma warning restore CS1591
+#pragma warning restore CS1591