Merge pull request #1 from ORCID/feat/test

feat/test

.github/workflows/build_debug.yml

@@ -69,8 +69,8 @@ jobs:
 
 ##############################################################################
 
-  unit_tests_publish:
-    uses: ./.github/workflows/maven_test_publish.yml
+  mvn_test:
+    uses: ./.github/workflows/mvn_test.yml
     with:
       version_tag: ${{ inputs.version_tag }}
       bump: ${{ inputs.bump }}
@@ -86,7 +86,7 @@ jobs:
     secrets: inherit # pass all secrets for uploading assets
     needs:
       - seed_maven_cache
-      - unit_tests_publish
+      - mvn_test
     permissions:
       checks: write
       contents: read

.github/workflows/build_test_release.yml

@@ -70,8 +70,8 @@ jobs:
     needs:
       - seed_maven_cache
 
-  unit_tests_publish:
-    uses: ./.github/workflows/unit_tests_publish.yml
+  mvn_test:
+    uses: ./.github/workflows/mvn_test.yml
     with:
       version_tag: ${{ inputs.version_tag }}
       bump: ${{ inputs.bump }}
@@ -88,7 +88,7 @@ jobs:
     secrets: inherit # pass all secrets for uploading assets
     needs:
       - lint
-      - unit_tests_publish
+      - mvn_test
       - install_maven_dependencies
     permissions:
       checks: write

.github/workflows/build_test_release_tag.yml

@@ -54,8 +54,8 @@ jobs:
   lint:
     uses: ./.github/workflows/lint.yml
 
-  unit_tests_publish:
-    uses: ./.github/workflows/unit_tests_publish.yml
+  mvn_test:
+    uses: ./.github/workflows/mvn_test.yml
     with:
       version_tag: ${{ inputs.version_tag }}
       bump: ${{ inputs.bump }}
@@ -70,7 +70,7 @@ jobs:
     secrets: inherit # pass all secrets for uploading assets
     needs:
       - lint
-      - unit_tests_publish
+      - mvn_test
     permissions:
       checks: write
       contents: read

.github/workflows/maven_test_publish.yml → .github/workflows/mvn_test.yml

@@ -1,7 +1,6 @@
-name: maven_test_publish
+name: mvn_test
 
 # run maven tests and publish results in one step
-# run from privileged branch
 
 permissions:
   checks: write
@@ -78,7 +77,13 @@ on:
 
 
 jobs:
-  maven_test:
+  mvn_test:
+
+    strategy:
+      matrix:
+        include:
+          - project: hello_world
+
     runs-on: ubuntu-latest
     steps:
       - name: git-checkout-ref-action
@@ -109,19 +114,25 @@ jobs:
           java-version: '11'
 
 ########################################################################################
-      - name: "Test ${{ inputs.project }}"
+      - name: "Test ${{ matrix.project }}"
         run: |
-          mvn -T 1C --batch-mode -Dmaven.test.failure.ignore=true test \
+          mvn -T 1C --batch-mode test \
               -Dorg.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=warn \
-              --projects=${{ inputs.project }}
+              --projects=${{ matrix.project }} --fail-at-end
 
-      # NOTE: if the above test fail then this step will report that failure and stop the run
-      - name: Publish Unit Test Results
+      - name: Publish unit test report for non forked repo
+        if: ${{ failure() && (github.event.pull_request.head.repo.full_name == github.repository) }} # only report if a test has failed and we are a non forked repo
         uses: dorny/test-reporter@e9fa2f582c0ebbe2e263fd18fad744d52e0b0203
-        if: always()
         with:
-          name: "maven_test_publish_${{ inputs.project }}"
-          path: ${{ inputs.project }}/target/surefire-reports/*.xml
+          name: "Unit test report for ${{ matrix.project }}"
+          path: ${{ matrix.project }}/target/surefire-reports/*.xml
           reporter: java-junit
           fail-on-error: true
 
+      - name: Annotate only test report for forked repo
+        uses: mikepenz/action-junit-report@ec3a351c13e080dc4fa94c49ab7ad5bf778a9668 # v5
+        if: ${{ failure() &&  (github.event.pull_request.head.repo.full_name != github.repository) }} # only report if a test has failed and we are a forked repo
+        with:
+          report_paths: ${{ matrix.project }}/target/surefire-reports/*.xml
+          annotate_only: true # forked repo cannot write to checks so just do annotations
+

.github/workflows/pr.yml

@@ -17,8 +17,8 @@ concurrency:
   cancel-in-progress: true
 
 jobs:
-  unit_tests_artifact:
-    uses: ./.github/workflows/unit_tests_artifact.yml
+  mvn_test:
+    uses: ./.github/workflows/mvn_test.yml
     with:
       seed_maven_cache: true
       install_maven_dependencies: true