changes from review

Author: tonyredondo

tracer/src/Datadog.Trace/ClrProfiler/AutoInstrumentation/Testing/MsTestV2/TestAssemblyInfoRunAssemblyInitializeIntegration.cs

@@ -54,16 +54,18 @@ internal static CallTargetState OnMethodBegin<TTarget, TContext>(TTarget instanc
             instance.AssemblyCleanupMethod ??= EmptyCleanUpMethodInfo;
         }
 
-        string? assemblyName;
+        string? assemblyName = null;
         if (testContext.TryDuckCast<TestContextStruct>(out var context) && context.TestMethod is { AssemblyName: { } contextAssemblyName })
         {
             assemblyName = contextAssemblyName;
         }
-        else if (instance.Instance?.TryDuckCast<ITestAssemblyInfoWithAssembly>(out var instanceWithAssembly) == true)
+
+        if (string.IsNullOrEmpty(assemblyName) && instance.Instance?.TryDuckCast<ITestAssemblyInfoWithAssembly>(out var instanceWithAssembly) == true)
         {
             assemblyName = instanceWithAssembly.Assembly.GetName().Name;
         }
-        else
+
+        if (string.IsNullOrEmpty(assemblyName))
         {
             Common.Log.Warning("Module name cannot be extracted!");
             return CallTargetState.GetDefault();
@@ -95,7 +97,7 @@ private static void EmptyCleanUpMethod()
     }
 
 #pragma warning disable SA1201
-    internal interface ITestAssemblyInfoWithAssembly : ITestAssemblyInfo
+    internal interface ITestAssemblyInfoWithAssembly
 #pragma warning restore SA1201
     {
         Assembly Assembly { get; }

tracer/src/Datadog.Trace/ClrProfiler/AutoInstrumentation/Testing/MsTestV2/TestMethodRunnerExecuteIntegration3_8.cs

@@ -46,7 +46,7 @@ public static class TestMethodRunnerExecuteIntegration3_8
             return new CallTargetReturn<TReturn?>(returnValue);
         }
 
-        if (returnValue is IList { Count: > 0 } lstResults)
+        if (returnValue is ICollection { Count: > 0 } lstResults)
         {
             foreach (var testResultObject in lstResults)
             {

tracer/src/Datadog.Trace/ClrProfiler/AutoInstrumentation/Testing/MsTestV2/UnitTestRunnerRunSingleTestAsyncIntegration3_8.cs

@@ -55,9 +55,9 @@ internal static CallTargetState OnMethodBegin<TTarget, TTestMethod, TMessageLogg
         }
 
         MethodInfoCacheItem? methodInfoCacheItem;
-        if (state.State is StrongBox<MethodInfoCacheItem?> box)
+        if (state.State is StrongBox<MethodInfoCacheItem?> { Value: { } value })
         {
-            methodInfoCacheItem = box.Value ?? MsTestIntegration.IsTestMethodRunnableThreadLocal.Value;
+            methodInfoCacheItem = value;
         }
         else
         {