chore: use dorny/test-reporter for reporting gh action test results (#2036)

* chore: use dorny/test-reporter for reporting gh action test results

---------

Co-authored-by: Philip Jenvey <pjenvey@underboss.org>

Author: chenba

.cargo/audit.toml

@@ -3,4 +3,5 @@ ignore = [
        "RUSTSEC-2024-0365", # Bound by diesel 1.4 (4GB limit n/a to tokenserver)
        "RUSTSEC-2024-0421", # Bound by diesel 1.4, `idna`  < 0.1.5, Upgrade to >=1.0.0
        "RUSTSEC-2024-0437", # Bound by grpcio 0.13,
+       "RUSTSEC-2026-0002", # Bound by mysql_async, via diesel-async
 ]

.config/nextest.toml

@@ -17,7 +17,8 @@ fail-fast = false
 
 [profile.ci.junit]
 path = "junit.xml"
-
 report-name = "syncstorage-unit-tests"
 store-success-output = false
 store-failure-output = true
+store-success-timing = true
+store-failure-timing = true

.github/actions/setup-rust/action.yml

@@ -28,7 +28,7 @@ runs:
 
     - name: Create workspace directory
       shell: bash
-      run: mkdir -p ${{ inputs.workspace-path }}
+      run: mkdir -p ${{ inputs.workspace-path }}/test-results
 
     - name: Display Version Info
       shell: bash

.github/workflows/mysql.yml

@@ -80,6 +80,15 @@ jobs:
         env:
           SYNC_SYNCSTORAGE__ENFORCE_QUOTA: 1
 
+      - name: Publish Test Report
+        uses: dorny/test-reporter@a810f9bf83f2344124a920a7a0a85a6716e791f0
+        if: always()
+        with:
+          name: MySQL Unit Tests
+          path: workflow/test-results/*.xml
+          reporter: java-junit
+          fail-on-error: false
+
       - name: Upload test results
         if: always()
         uses: actions/upload-artifact@v6
@@ -184,6 +193,15 @@ jobs:
         env:
           SYNCSTORAGE_RS_IMAGE: app:build
 
+      - name: Publish E2E Test Report
+        uses: dorny/test-reporter@a810f9bf83f2344124a920a7a0a85a6716e791f0
+        if: always()
+        with:
+          name: MySQL E2E Tests
+          path: workflow/test-results/*.xml
+          reporter: java-junit
+          fail-on-error: false
+
       - name: Upload e2e test results
         if: always()
         uses: actions/upload-artifact@v6

.github/workflows/postgres.yml

@@ -86,6 +86,15 @@ jobs:
         env:
           SYNC_SYNCSTORAGE__DATABASE_URL: postgresql://test:test@127.0.0.1/syncstorage
 
+      - name: Publish Test Report
+        uses: dorny/test-reporter@a810f9bf83f2344124a920a7a0a85a6716e791f0
+        if: always()
+        with:
+          name: Postgres Unit Tests
+          path: workflow/test-results/*.xml
+          reporter: java-junit
+          fail-on-error: false
+
       - name: Upload test results
         if: always()
         uses: actions/upload-artifact@v6
@@ -190,6 +199,15 @@ jobs:
         env:
           SYNCSTORAGE_RS_IMAGE: app:build
 
+      - name: Publish E2E Test Report
+        uses: dorny/test-reporter@a810f9bf83f2344124a920a7a0a85a6716e791f0
+        if: always()
+        with:
+          name: Postgres E2E Tests
+          path: workflow/test-results/*.xml
+          reporter: java-junit
+          fail-on-error: false
+
       - name: Upload e2e test results
         if: always()
         uses: actions/upload-artifact@v6

.github/workflows/spanner.yml

@@ -93,6 +93,15 @@ jobs:
           RUST_TEST_THREADS: 1
         run: make spanner_test_with_coverage
 
+      - name: Publish Test Report
+        uses: dorny/test-reporter@a810f9bf83f2344124a920a7a0a85a6716e791f0
+        if: always()
+        with:
+          name: Spanner Unit Tests
+          path: workflow/test-results/*.xml
+          reporter: java-junit
+          fail-on-error: false
+
       - name: Upload test results
         if: always()
         uses: actions/upload-artifact@v6
@@ -197,6 +206,15 @@ jobs:
         env:
           SYNCSTORAGE_RS_IMAGE: app:build
 
+      - name: Publish E2E Test Report
+        uses: dorny/test-reporter@a810f9bf83f2344124a920a7a0a85a6716e791f0
+        if: always()
+        with:
+          name: Spanner E2E Tests
+          path: workflow/test-results/*.xml
+          reporter: java-junit
+          fail-on-error: false
+
       - name: Upload e2e test results
         if: always()
         uses: actions/upload-artifact@v6

Makefile

@@ -29,7 +29,7 @@ RUST_LOG ?= debug
 WORKFLOW := build-deploy
 EPOCH_TIME := $(shell date +"%s")
 TEST_RESULTS_DIR ?= workflow/test-results
-TEST_PROFILE := $(if $(CIRCLECI),ci,default)
+TEST_PROFILE := $(if $(or $(CIRCLECI),$(GITHUB_ACTIONS)),ci,default)
 TEST_FILE_PREFIX := $(if $(CIRCLECI),$(CIRCLE_BUILD_NUM)__$(EPOCH_TIME)__$(CIRCLE_PROJECT_REPONAME)__$(WORKFLOW)__)
 UNIT_JUNIT_XML := $(TEST_RESULTS_DIR)/$(TEST_FILE_PREFIX)unit__results.xml
 MYSQL_UNIT_JUNIT_XML := $(TEST_RESULTS_DIR)/$(TEST_FILE_PREFIX)mysql_unit__results.xml