Fix for when an async block throws

Author: PandaMagnus

IntelliTect.TestTools.TestFramework.Tests/TestCaseTests/TestFailureTests.cs

@@ -1,5 +1,6 @@
 using IntelliTect.TestTools.TestFramework.Tests.TestData.Dependencies;
 using IntelliTect.TestTools.TestFramework.Tests.TestData.TestBlocks;
+using Microsoft.Playwright;
 using System;
 using System.Threading.Tasks;
 using Xunit;
@@ -41,5 +42,19 @@ public async Task DependencyWithMissingDependencyThrowsOriginalError()
             Assert.IsType<InvalidOperationException>(ex.InnerException);
             Assert.Contains("oops", ex.InnerException!.Message, StringComparison.InvariantCultureIgnoreCase);
         }
+
+        [Fact]
+        public async Task CgiIssue()
+        {
+            TestCase tc = new TestBuilder()
+                .AddAsyncTestBlock<PlaywrightError>()
+                .Build();
+
+            var ex = await Assert.ThrowsAsync<TestCaseException>(() => tc.ExecuteAsync());
+            Assert.False(tc.Passed);
+            Assert.NotNull(ex.InnerException);
+            Assert.IsType<InvalidOperationException>(ex.InnerException);
+            Assert.Contains("oops", ex.InnerException!.Message, StringComparison.InvariantCultureIgnoreCase);
+        }
     }
 }

IntelliTect.TestTools.TestFramework.Tests/TestData/TestBlocks/SingleDependencyBlocks.cs

@@ -1,5 +1,7 @@
 using IntelliTect.TestTools.TestFramework.Tests.TestData.Dependencies;
+using Microsoft.Playwright;
 using System;
+using System.Threading.Tasks;
 using Xunit;
 
 namespace IntelliTect.TestTools.TestFramework.Tests.TestData.TestBlocks
@@ -70,4 +72,17 @@ public bool Execute(SomeDependency dep)
             return true;
         }
     }
+
+    public class PlaywrightError : TestBlock
+    {
+        public async Task Execute()
+        {
+            await Task.Delay(1);
+            throw new InvalidOperationException("Oops");
+            //using IPlaywright pw = await Playwright.CreateAsync();
+            //IAPIRequestContext context = await pw.APIRequest.NewContextAsync();
+            //await context.GetAsync("http://bad.url.com");
+            // Playwright exception thrown for ENOTFOUND
+        }
+    }
 }

IntelliTect.TestTools.TestFramework/TestCase.cs

@@ -422,6 +422,14 @@ private async Task<bool> TryRunBlock(Block block, object blockInstance, List<obj
                     () => FinallyBlockExceptions.Add(ex)
                 );
             }
+            catch (Exception ex)
+            {
+                HandleFinallyBlock(
+                    block,
+                    () => TestBlockException = ex,
+                    () => FinallyBlockExceptions.Add(ex)
+                );
+            }
 
             if (result) Log?.Debug($"Test block completed successfully.");
             return result;