Fixed issue where orchestrations that never complete cause E2E tests to run indefinitely

Author: Michael Peng

test/E2E/Azure.Functions.PowerShellWorker.E2E/Azure.Functions.PowerShellWorker.E2E/DurableEndToEndTests.cs

@@ -140,6 +140,11 @@ public async Task CurrentUtcDateTimeClientReplaysCurrentUtcDateTime()
                     {
                         case HttpStatusCode.Accepted:
                         {
+                            if (DateTime.UtcNow > startTime + orchestrationCompletionTimeout)
+                            {
+                                Assert.True(false, $"The orchestration has not completed after {orchestrationCompletionTimeout}");
+                            }
+
                             await Task.Delay(TimeSpan.FromSeconds(2));
                             break;
                         }
@@ -221,6 +226,11 @@ private async Task DurableTimerClientStopsOrchestratorAndUpdatesCurrentUtcDateTi
                     {
                         case HttpStatusCode.Accepted:
                         {
+                            if (DateTime.UtcNow > startTime + orchestrationCompletionTimeout)
+                            {
+                                Assert.True(false, $"The orchestration has not completed after {orchestrationCompletionTimeout}");
+                            }
+                            
                             await Task.Delay(TimeSpan.FromSeconds(2));
                             break;
                         }
@@ -274,6 +284,11 @@ private async Task ExternalEventClientSendsExternalEvents() {
                     {
                         case HttpStatusCode.Accepted:
                         {
+                            if (DateTime.UtcNow > startTime + orchestrationCompletionTimeout)
+                            {
+                                Assert.True(false, $"The orchestration has not completed after {orchestrationCompletionTimeout}");
+                            }
+                            
                             await Task.Delay(TimeSpan.FromSeconds(2));
                             break;
                         }

test/E2E/TestFunctionApp/ExternalEventOrchestrator/run.ps1

@@ -12,8 +12,8 @@ $firstDuration = New-TimeSpan -Seconds $Context.Input.FirstDuration
 $secondDuration = New-TimeSpan -Seconds $Context.Input.SecondDuration
 
 $firstTimeout = Start-DurableTimer -Duration $firstDuration -NoWait
-$firstExternalEvent = Start-DurableExternalEventListener -EventName "FirstExternalEvent"
-$firstCompleted = Wait-DurableTask -Task @($firstTimeout, $firstExternalEvent)
+$firstExternalEvent = Start-DurableExternalEventListener -EventName "FirstExternalEvent" -NoWait
+$firstCompleted = Wait-DurableTask -Task @($firstTimeout, $firstExternalEvent) -Any
 
 if ($firstCompleted -eq $firstTimeout) {
     $output += "FirstTimeout"
@@ -24,8 +24,8 @@ else {
 }
 
 $secondTimeout = Start-DurableTimer -Duration $secondDuration -NoWait
-$secondExternalEvent = Start-DurableExternalEventListener -EventName "SecondExternalEvent"
-$secondCompleted = Wait-DurableTask -Task @($secondTimeout, $secondExternalEvent)
+$secondExternalEvent = Start-DurableExternalEventListener -EventName "SecondExternalEvent" -NoWait
+$secondCompleted = Wait-DurableTask -Task @($secondTimeout, $secondExternalEvent) -Any
 
 if ($secondCompleted -eq $secondTimeout) {
     $output += "SecondTimeout"