Don't immediatly check the request was collected

Author: LukeButters

source/Halibut/Queue/Redis/NodeHeartBeat/NodeHeartBeatWatcher.cs

@@ -134,6 +134,15 @@ static async Task WaitForRequestToBeCollected(
 
             while (!cancellationToken.IsCancellationRequested)
             {
+                await Try.IgnoringError(async () =>
+                {
+                    await Task.WhenAny(
+                        Task.Delay(timeBetweenCheckingIfRequestWasCollected, cancellationToken),
+                        redisPending.WaitForRequestToBeMarkedAsCollected(cancellationToken));
+                });
+                
+                if(cancellationToken.IsCancellationRequested) break;
+                
                 try
                 {
                     // Has something else determined the request was collected?
@@ -156,13 +165,6 @@ static async Task WaitForRequestToBeCollected(
                 {
                     log.WriteException(EventType.Diagnostic, "Error checking if request {0} is still on queue", ex, request.ActivityId);
                 }
-                
-                await Try.IgnoringError(async () =>
-                {
-                    await Task.WhenAny(
-                        Task.Delay(timeBetweenCheckingIfRequestWasCollected, cancellationToken),
-                        redisPending.WaitForRequestToBeMarkedAsCollected(cancellationToken));
-                });
             }
 
             log.Write(EventType.Diagnostic, "Stopped waiting for request {0} to be collected (cancelled)", request.ActivityId);