perf: eliminate closure and Func creation

Author: TimothyMakkison

TUnit.Core/ContextProvider.cs

@@ -61,17 +61,20 @@ public AssemblyHookContext GetOrCreateAssemblyContext(Assembly assembly)
     /// <summary>
     /// Gets or creates a class context
     /// </summary>
+    [UnconditionalSuppressMessage("Trimming", "IL2111",
+        Justification = "Type parameter is annotated at the method boundary.")]
     public ClassHookContext GetOrCreateClassContext(
         [DynamicallyAccessedMembers(DynamicallyAccessedMemberTypes.PublicConstructors | DynamicallyAccessedMemberTypes.PublicProperties | DynamicallyAccessedMemberTypes.PublicMethods)]
         Type classType)
     {
-        return _classContexts.GetOrAdd(classType, type =>
+        return _classContexts.GetOrAdd(classType, static ([DynamicallyAccessedMembers(DynamicallyAccessedMemberTypes.PublicConstructors | DynamicallyAccessedMemberTypes.PublicProperties | DynamicallyAccessedMemberTypes.PublicMethods)]
+            type, state) =>
         {
-            return new ClassHookContext(GetOrCreateAssemblyContext(classType.Assembly))
+            return new ClassHookContext(state.GetOrCreateAssemblyContext(type.Assembly))
             {
-                ClassType = classType
+                ClassType = type
             };
-        });
+        }, this);
     }
 
     /// <summary>

TUnit.Engine/Discovery/ReflectionTestDataCollector.cs

@@ -170,15 +170,15 @@ public async IAsyncEnumerable<TestMetadata> CollectTestsStreamingAsync(
             // Stream tests from this assembly
             await foreach (var test in DiscoverTestsInAssemblyStreamingAsync(assembly, cancellationToken))
             {
-                ImmutableInterlocked.Update(ref _discoveredTests, list => list.Add(test));
+                ImmutableInterlocked.Update(ref _discoveredTests, static (list, test) => list.Add(test), test);
                 yield return test;
             }
         }
 
         // Stream dynamic tests
         await foreach (var dynamicTest in DiscoverDynamicTestsStreamingAsync(testSessionId, cancellationToken))
         {
-            ImmutableInterlocked.Update(ref _discoveredTests, list => list.Add(dynamicTest));
+            ImmutableInterlocked.Update(ref _discoveredTests, static (list, dynamicTest) => list.Add(dynamicTest), dynamicTest);
             yield return dynamicTest;
         }
     }

TUnit.PublicAPI/Tests.Core_Library_Has_No_API_Changes.DotNet10_0.verified.txt

@@ -419,6 +419,7 @@ namespace
         public .TestSessionContext TestSessionContext { get; }
         public .TestContext CreateTestContext(string testName, [.(..None | ..PublicParameterlessConstructor | ..PublicConstructors | ..PublicMethods | ..PublicProperties)]  classType, .TestBuilderContext testBuilderContext, .CancellationToken cancellationToken) { }
         public .AssemblyHookContext GetOrCreateAssemblyContext(.Assembly assembly) { }
+        [.("Trimming", "IL2111", Justification="Type parameter is annotated at the method boundary.")]
         public .ClassHookContext GetOrCreateClassContext([.(..None | ..PublicParameterlessConstructor | ..PublicConstructors | ..PublicMethods | ..PublicProperties)]  classType) { }
     }
     public class CultureExecutor : .DedicatedThreadExecutor

TUnit.PublicAPI/Tests.Core_Library_Has_No_API_Changes.DotNet8_0.verified.txt

@@ -419,6 +419,7 @@ namespace
         public .TestSessionContext TestSessionContext { get; }
         public .TestContext CreateTestContext(string testName, [.(..None | ..PublicParameterlessConstructor | ..PublicConstructors | ..PublicMethods | ..PublicProperties)]  classType, .TestBuilderContext testBuilderContext, .CancellationToken cancellationToken) { }
         public .AssemblyHookContext GetOrCreateAssemblyContext(.Assembly assembly) { }
+        [.("Trimming", "IL2111", Justification="Type parameter is annotated at the method boundary.")]
         public .ClassHookContext GetOrCreateClassContext([.(..None | ..PublicParameterlessConstructor | ..PublicConstructors | ..PublicMethods | ..PublicProperties)]  classType) { }
     }
     public class CultureExecutor : .DedicatedThreadExecutor

TUnit.PublicAPI/Tests.Core_Library_Has_No_API_Changes.DotNet9_0.verified.txt

@@ -419,6 +419,7 @@ namespace
         public .TestSessionContext TestSessionContext { get; }
         public .TestContext CreateTestContext(string testName, [.(..None | ..PublicParameterlessConstructor | ..PublicConstructors | ..PublicMethods | ..PublicProperties)]  classType, .TestBuilderContext testBuilderContext, .CancellationToken cancellationToken) { }
         public .AssemblyHookContext GetOrCreateAssemblyContext(.Assembly assembly) { }
+        [.("Trimming", "IL2111", Justification="Type parameter is annotated at the method boundary.")]
         public .ClassHookContext GetOrCreateClassContext([.(..None | ..PublicParameterlessConstructor | ..PublicConstructors | ..PublicMethods | ..PublicProperties)]  classType) { }
     }
     public class CultureExecutor : .DedicatedThreadExecutor