Improve test execution error handling in workflows (#312)

+semver:patch

Author: hazzik

.github/workflows/dotnet-linux.yml

@@ -54,9 +54,12 @@ jobs:
 
       - name: Run tests
         run: |
-          dotnet test --no-build -c Release -f net8.0 src/DelegateDecompiler.Tests
-          dotnet test --no-build -c Release -f net8.0 src/DelegateDecompiler.Tests.VB
-          dotnet test --no-build -c Release -f net8.0 src/DelegateDecompiler.EntityFramework.Tests
-          dotnet test --no-build -c Release -f net8.0 src/DelegateDecompiler.EntityFrameworkCore8.Tests
-          dotnet test --no-build -c Release -f net9.0 src/DelegateDecompiler.EntityFrameworkCore9.Tests
-          dotnet test --no-build -c Release -f net10.0 src\DelegateDecompiler.EntityFrameworkCore10.Tests
+          $failed = $false
+          function Fail { $script:failed = $true }
+          dotnet test --no-build -c Release -f net8.0 src/DelegateDecompiler.Tests || Fail
+          dotnet test --no-build -c Release -f net8.0 src/DelegateDecompiler.Tests.VB || Fail
+          dotnet test --no-build -c Release -f net8.0 src/DelegateDecompiler.EntityFramework.Tests || Fail
+          dotnet test --no-build -c Release -f net8.0 src/DelegateDecompiler.EntityFrameworkCore8.Tests || Fail
+          dotnet test --no-build -c Release -f net9.0 src/DelegateDecompiler.EntityFrameworkCore9.Tests || Fail
+          dotnet test --no-build -c Release -f net10.0 src/DelegateDecompiler.EntityFrameworkCore10.Tests || Fail
+          if ($failed) { exit 1 }

.github/workflows/dotnet-windows.yml

@@ -48,9 +48,12 @@ jobs:
 
       - name: Run tests
         run: |
-          dotnet test --no-build -c Release -f net8.0 src/DelegateDecompiler.Tests
-          dotnet test --no-build -c Release -f net8.0 src/DelegateDecompiler.Tests.VB
-          dotnet test --no-build -c Release -f net8.0 src/DelegateDecompiler.EntityFramework.Tests
-          dotnet test --no-build -c Release -f net8.0 src/DelegateDecompiler.EntityFrameworkCore8.Tests
-          dotnet test --no-build -c Release -f net9.0 src/DelegateDecompiler.EntityFrameworkCore9.Tests
-          dotnet test --no-build -c Release -f net10.0 src\DelegateDecompiler.EntityFrameworkCore10.Tests
+          $failed = $false
+          function Fail { $script:failed = $true }
+          dotnet test --no-build -c Release -f net8.0 src/DelegateDecompiler.Tests || Fail
+          dotnet test --no-build -c Release -f net8.0 src/DelegateDecompiler.Tests.VB || Fail
+          dotnet test --no-build -c Release -f net8.0 src/DelegateDecompiler.EntityFramework.Tests || Fail
+          dotnet test --no-build -c Release -f net8.0 src/DelegateDecompiler.EntityFrameworkCore8.Tests || Fail
+          dotnet test --no-build -c Release -f net9.0 src/DelegateDecompiler.EntityFrameworkCore9.Tests || Fail
+          dotnet test --no-build -c Release -f net10.0 src/DelegateDecompiler.EntityFrameworkCore10.Tests || Fail
+          if ($failed) { exit 1 }

src/DelegateDecompiler.Tests/StackOverflowTest.cs

@@ -56,7 +56,7 @@ public void StackOverflowTestOnThreadWithLargeStack()
             thread.Join();
         }
 
-        [Test]
+        [Test, Explicit]
         public void StackOverflowTestOnThreadWithSmallStack()
         {
             Thread thread = new Thread(() =>