Updated WriteLog to always write the log on exceptions (#350)

Updated BuildException usages to always include log data Closes #349
roryprimrose · Aug 19, 2024 · 4aed4fa · 4aed4fa
1 parent 7af0751
commit 4aed4fa
Show file tree

Hide file tree

Showing 7 changed files with 333 additions and 43 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -19,7 +19,7 @@ jobs:
     steps:
 
     - name: Checkout
-      uses: actions/checkout@v4.1.1
+      uses: actions/checkout@v4.1.6
       with:
         fetch-depth: 0
 
@@ -59,7 +59,7 @@ jobs:
 
     - name: Generate coverage report
       # run: reportgenerator -reports:**/coverage.cobertura.xml -targetdir:Report -reporttypes:HtmlInline_AzurePipelines;Cobertura
-      uses: danielpalme/ReportGenerator-GitHub-Action@5.2.3
+      uses: danielpalme/ReportGenerator-GitHub-Action@5.2.4
       with:
         reports: "**/coverage*cobertura.xml"
         targetdir: "Report"
@@ -85,7 +85,7 @@ jobs:
       run: dotnet pack "./${{ env.projectName }}/${{ env.projectName }}.csproj" -c Release --no-build --include-symbols -o $GITHUB_WORKSPACE/staging
 
     - name: Publish build artifacts
-      uses: actions/upload-artifact@v4.3.1
+      uses: actions/upload-artifact@v4.3.3
       with:
         name: packages
         path: staging
@@ -95,7 +95,7 @@ jobs:
       run: |
         echo '${{ steps.gitversion.outputs.NuGetVersionV2 }}' > version.txt
     - name: Upload version
-      uses: actions/upload-artifact@v4.3.1
+      uses: actions/upload-artifact@v4.3.3
       with:
         name: version
         path: version.txt
@@ -108,7 +108,7 @@ jobs:
 
     steps:
     - name: Download packages
-      uses: actions/download-artifact@v4.1.4
+      uses: actions/download-artifact@v4.1.7
       with:
         name: packages
 
@@ -129,12 +129,12 @@ jobs:
     steps:
 
     - name: Download packages
-      uses: actions/download-artifact@v4.1.4
+      uses: actions/download-artifact@v4.1.7
       with:
         name: packages
 
     - name: Download version
-      uses: actions/download-artifact@v4.1.4
+      uses: actions/download-artifact@v4.1.7
       with:
         name: version
 

diff --git a/ModelBuilder.UnitTests/ModelBuilder.UnitTests.csproj b/ModelBuilder.UnitTests/ModelBuilder.UnitTests.csproj
@@ -18,9 +18,9 @@
     </PackageReference>
     <PackageReference Include="NodaTime" Version="3.1.11" />
     <PackageReference Include="nsubstitute" Version="5.1.0" />
-    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.9.0" />
-    <PackageReference Include="xunit" Version="2.7.0" />
-    <PackageReference Include="xunit.runner.visualstudio" Version="2.5.7">
+    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.10.0" />
+    <PackageReference Include="xunit" Version="2.9.0" />
+    <PackageReference Include="xunit.runner.visualstudio" Version="2.8.2">
       <PrivateAssets>all</PrivateAssets>
       <IncludeAssets>runtime; build; native; contentfiles; analyzers</IncludeAssets>
     </PackageReference>

diff --git a/ModelBuilder.UnitTests/Models/PropertySetFailure.cs b/ModelBuilder.UnitTests/Models/PropertySetFailure.cs
@@ -0,0 +1,9 @@
+namespace ModelBuilder.UnitTests.Models
+{
+    using System;
+
+    internal class PropertySetFailure
+    {
+        public string Name { get => "Name"; set => throw new NotImplementedException(); }
+    }
+}