Flaky test investigation

Youssef1313 · Youssef1313 · commit 70f7c3bed9ce · 2025-02-12T14:11:03.000+01:00
diff --git a/src/Adapter/MSTest.TestAdapter/Helpers/FixtureMethodRunner.cs b/src/Adapter/MSTest.TestAdapter/Helpers/FixtureMethodRunner.cs
@@ -174,8 +174,15 @@ internal static class FixtureMethodRunner
                 UnitTestOutcome.Timeout,
                 string.Format(CultureInfo.InvariantCulture, methodCanceledMessageFormat, methodInfo.DeclaringType!.FullName, methodInfo.Name));
         }
-        catch (Exception)
+        catch (Exception ex)
         {
+            if (ex is AggregateException aggregateEx && aggregateEx.InnerExceptions.OfType<TaskCanceledException>().Any(tce => tce.CancellationToken == cancellationTokenSource.Token))
+            {
+                return new(
+                    UnitTestOutcome.Timeout,
+                    "Are we going to hit this??");
+            }
+
             // We throw the real exception to have the original stack trace to elaborate up the chain.
             if (realException is not null)
             {

Original file line number	Diff line number	Diff line change
`@@ -174,8 +174,15 @@ internal static class FixtureMethodRunner`
`174`	`174`	`UnitTestOutcome.Timeout,`
`175`	`175`	`string.Format(CultureInfo.InvariantCulture, methodCanceledMessageFormat, methodInfo.DeclaringType!.FullName, methodInfo.Name));`
`176`	`176`	`}`
`177`		`- catch (Exception)`
	`177`	`+ catch (Exception ex)`
`178`	`178`	`{`
	`179`	`+ if (ex is AggregateException aggregateEx && aggregateEx.InnerExceptions.OfType<TaskCanceledException>().Any(tce => tce.CancellationToken == cancellationTokenSource.Token))`
	`180`	`+ {`
	`181`	`+ return new(`
	`182`	`+ UnitTestOutcome.Timeout,`
	`183`	`+ "Are we going to hit this??");`
	`184`	`+ }`
	`185`	`+`
`179`	`186`	`// We throw the real exception to have the original stack trace to elaborate up the chain.`
`180`	`187`	`if (realException is not null)`
`181`	`188`	`{`