Use .NET 8 SDK's artifacts output and enable test reports in CI (#23)

* Use .NET 8 SDK's artifacts output * Enable TRX reports in build pipeline * Switch from test reporter to basic artifact
rjmurillo · May 30, 2024 · d8635ba · d8635ba
1 parent eaf5d47
commit d8635ba
Show file tree

Hide file tree

Showing 7 changed files with 36 additions and 10 deletions.
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -39,16 +39,13 @@ jobs:
     - name: Test
       run: dotnet test --no-build --configuration Release
 
-    # TODO: Create a TRX report and upload as an artifact. Tracked by #22. 
-    # - name: Publish Test Report
-    #   uses: dorny/test-reporter@v1
-    #   if: success() || failure()
-    #   with:
-    #     name: .NET Test Report (${{ matrix.os }})
-    #     path: "artifacts/TestResults/**/*.trx"
-    #     reporter: dotnet-trx
-    #     fail-on-error: true
-    #     fail-on-empty: true
+    - name: Publish Test Report
+      uses: actions/upload-artifact@v4
+      if: success() || failure()
+      with:
+        name: .NET Test Reports (${{ matrix.os }})
+        path: "artifacts/TestResults/**/*.trx"
+        if-no-files-found: error
 
     - name: Upload binlogs
       uses: actions/upload-artifact@v4

diff --git a/Directory.Build.props b/Directory.Build.props
@@ -0,0 +1,8 @@
+<Project>
+  <PropertyGroup>
+    <RepoRoot>$(MSBuildThisFileDirectory)</RepoRoot>
+  </PropertyGroup>
+
+  <Import Project="build/targets/artifacts/Artifacts.props" />
+  <Import Project="build/targets/tests/Tests.props" />
+</Project>
diff --git a/Directory.Build.targets b/Directory.Build.targets
@@ -0,0 +1,4 @@
+<Project>
+  <Import Project="build/targets/artifacts/Artifacts.targets" />
+  <Import Project="build/targets/tests/Tests.targets" />
+</Project>
diff --git a/build/targets/artifacts/Artifacts.props b/build/targets/artifacts/Artifacts.props
@@ -0,0 +1,6 @@
+<Project>
+  <PropertyGroup>
+    <ArtifactsPath>$(RepoRoot)/artifacts</ArtifactsPath>
+    <ArtifactsTestResultsPath>$(ArtifactsPath)/TestResults</ArtifactsTestResultsPath>
+  </PropertyGroup>
+</Project>
diff --git a/build/targets/artifacts/Artifacts.targets b/build/targets/artifacts/Artifacts.targets
@@ -0,0 +1,2 @@
+<Project>
+</Project>
diff --git a/build/targets/tests/Tests.props b/build/targets/tests/Tests.props
@@ -0,0 +1,2 @@
+<Project>
+</Project>
diff --git a/build/targets/tests/Tests.targets b/build/targets/tests/Tests.targets
@@ -0,0 +1,7 @@
+<Project>
+  <PropertyGroup Condition=" '$(IsTestProject)' == 'true' ">
+    <!-- Enable test logging to TRX files and place them in the artifacts directory -->
+    <VSTestLogger>trx%3bLogFileName=$(MSBuildProjectName).trx</VSTestLogger>
+    <VSTestResultsDirectory Condition="'$(VSTestResultsDirectory)' == ''">$(ArtifactsTestResultsPath)/$(TargetFramework)</VSTestResultsDirectory>
+  </PropertyGroup>
+</Project>