Some cleanup and fixes to deploy.yml

Author: PandaMagnus

.github/workflows/deploy.yml

@@ -41,21 +41,21 @@ jobs:
       run: dotnet test --filter FullyQualifiedName!~ExampleTests --no-build --configuration Release --verbosity normal
     - name: Pack
       if: startsWith(github.ref, 'refs/tags/v')
-      run: dotnet pack -p:PackageVersion=${{ env.nugetVersion }} --configuration Release -o ${{github.workspace}}/IntelliTect.SelenatePack --no-build 
+      run: dotnet pack -p:PackageVersion=${{ env.nugetVersion }} --configuration Release -o ${{github.workspace}}/IntelliTect.TestToolsPack --no-build 
     - name: Upload Artifacts
       if: startsWith(github.ref, 'refs/tags/v')
       uses: actions/upload-artifact@v4
       with:
         name: NuGet
-        path: ${{github.workspace}}/IntelliTect.SelenatePack
+        path: ${{github.workspace}}/IntelliTect.TestToolsPack
 
   deploy:
     if: startsWith(github.ref, 'refs/tags/v')
     runs-on: ubuntu-latest
     needs: build-and-test
     environment:
       name: 'Production'
-      url: 'https://www.nuget.org/packages/IntelliTect.TestTools.Selenate'
+      url: 'https://www.nuget.org/packages/IntelliTect.TestTools.TestFramework'
     steps:
       - name: Download artifact from build job
         uses: actions/download-artifact@v4
@@ -65,11 +65,11 @@ jobs:
         run: |
           $tagVersion = "${{ github.ref }}".substring(11)
           echo "::set-output name=TAG_VERSION::$tagVersion"
-          dotnet nuget push IntelliTect.Selenate.$tagVersion.nupkg --source https://api.nuget.org/v3/index.json -k ${{ secrets.NUGET_API_KEY }} --skip-duplicate
+          dotnet nuget push IntelliTect.TestTools.TestFramework.$tagVersion.nupkg --source https://api.nuget.org/v3/index.json -k ${{ secrets.NUGET_API_KEY }} --skip-duplicate
         id: tag-version
       - name: Upload nupkg to Releases
         uses: softprops/action-gh-release@v2
         with:
           fail_on_unmatched_files: true
           generate_release_notes: true
-          files: IntelliTect.Selenate.${{ steps.tag-version.outputs.TAG_VERSION }}.nupkg
+          files: IntelliTect.TestTools.TestFramework.${{ steps.tag-version.outputs.TAG_VERSION }}.nupkg

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

@@ -46,7 +46,7 @@ public async Task DependencyWithMissingDependencyThrowsOriginalError()
         public async Task CgiIssue()
         {
             TestCase tc = new TestBuilder()
-                .AddAsyncTestBlock<PlaywrightError>()
+                .AddAsyncTestBlock<AsyncError>()
                 .Build();
 
             var ex = await Assert.ThrowsAsync<TestCaseException>(() => tc.ExecuteAsync());

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

@@ -72,16 +72,12 @@ public bool Execute(SomeDependency dep)
         }
     }
 
-    public class PlaywrightError : TestBlock
+    public class AsyncError : 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

@@ -94,12 +94,9 @@ public async Task ExecuteAsync()
                     bool runSuccess = await TryRunBlock(tb, testBlockInstance, executeArgs);
                     if (!runSuccess)
                     {
-                        if (TestBlockException is null)
-                        {
-                            TestBlockException = new(
+                        TestBlockException ??= new(
                                 $"Unknown error occurred while running test block {tb}. " +
                                 "Please file an issue: https://github.com/IntelliTect/TestTools.TestFramework/issues");
-                        }
                         break;
                     }
                 }
@@ -122,6 +119,10 @@ public async Task ExecuteAsync()
 
                 if (TestBlockException is null)
                 {
+                    // Note: This likely needs to be moved up above the finally blocks.
+                    // If a test case "passes" i.e. finishes all of its test blocks, we probably need to know that
+                    //  in the finally blocks.
+                    // Need to think about how to handle "passed" state if a finally block fails.
                     Passed = true;
                     Log?.Info("Test case finished successfully.");
                 }
@@ -405,14 +406,6 @@ private async Task<bool> TryRunBlock(Block block, object blockInstance, List<obj
                     () => FinallyBlockExceptions.Add(ex.InnerException)
                 );
             }
-            catch (ArgumentException ex)
-            {
-                HandleFinallyBlock(
-                    block,
-                    () => TestBlockException = ex,
-                    () => FinallyBlockExceptions.Add(ex)
-                );
-            }
             catch (TargetParameterCountException ex)
             {
                 ex.Data.Add("AdditionalInfo", "Test block failed: Mismatched count between Execute method arguments and supplied dependencies.");