Don't enforce global timeout when debugging (Azure#25524)

JoshLove-msft · web-flow · commit 5e83d8708995 · 2021-11-25T02:09:17.000Z
diff --git a/sdk/core/Azure.Core.TestFramework/src/ClientTestBase.cs b/sdk/core/Azure.Core.TestFramework/src/ClientTestBase.cs
@@ -3,6 +3,7 @@
 
 using System;
 using System.Collections.Generic;
+using System.Diagnostics;
 using System.Reflection;
 using Castle.DynamicProxy;
 using NUnit.Framework;
@@ -35,7 +36,7 @@ public virtual void GlobalTimeoutTearDown()
         {
             var executionContext = TestExecutionContext.CurrentContext;
             var duration = DateTime.UtcNow - executionContext.StartTime;
-            if (duration > TimeSpan.FromSeconds(GLOBAL_TEST_TIMEOUT_IN_SECONDS))
+            if (duration > TimeSpan.FromSeconds(GLOBAL_TEST_TIMEOUT_IN_SECONDS) && !Debugger.IsAttached)
             {
                 executionContext.CurrentResult.SetResult(ResultState.Failure, $"Test exceeded global time limit of {GLOBAL_TEST_TIMEOUT_IN_SECONDS} seconds. Duration: {duration}");
             }

Original file line number	Diff line number	Diff line change
`@@ -3,6 +3,7 @@`
`3`	`3`
`4`	`4`	`using System;`
`5`	`5`	`using System.Collections.Generic;`
	`6`	`+using System.Diagnostics;`
`6`	`7`	`using System.Reflection;`
`7`	`8`	`using Castle.DynamicProxy;`
`8`	`9`	`using NUnit.Framework;`
`@@ -35,7 +36,7 @@ public virtual void GlobalTimeoutTearDown()`
`35`	`36`	`{`
`36`	`37`	`var executionContext = TestExecutionContext.CurrentContext;`
`37`	`38`	`var duration = DateTime.UtcNow - executionContext.StartTime;`
`38`		`- if (duration > TimeSpan.FromSeconds(GLOBAL_TEST_TIMEOUT_IN_SECONDS))`
	`39`	`+ if (duration > TimeSpan.FromSeconds(GLOBAL_TEST_TIMEOUT_IN_SECONDS) && !Debugger.IsAttached)`
`39`	`40`	`{`
`40`	`41`	`executionContext.CurrentResult.SetResult(ResultState.Failure, $"Test exceeded global time limit of {GLOBAL_TEST_TIMEOUT_IN_SECONDS} seconds. Duration: {duration}");`
`41`	`42`	`}`