fix: unit test reporting (#162)

Author: erdos

.github/workflows/pr_flow.yml

@@ -4,6 +4,9 @@ on:
   pull_request:
     branches: [ master ]
 
+env:
+  TEST_REPORT_JUNIT_XML_OUTPUT_DIR: target/surefire-reports
+
 jobs:
   build:
     runs-on: ubuntu-latest
@@ -23,9 +26,11 @@ jobs:
     - name: Run Java Tests
       run: lein pom && mvn test --batch-mode --fail-at-end
     - name: Unit tests in Clojure
-      run: lein junit
-    - name: Publish Test Report
-      if: ${{ always() }}
-      uses: scacap/action-surefire-report@v1
+      run: lein test
+    - name: Test Report
+      uses: dorny/test-reporter@v1
+      if: success() || failure()
       with:
-        report_paths: 'target/surefire-reports/*.xml'
+        name: Unit Tests
+        path: target/surefire-reports/*.xml
+        reporter: java-junit

.github/workflows/pr_visual_flow.yml

@@ -6,6 +6,7 @@ on:
 
 env:
   STENCIL_TEST_VISUAL: true
+  TEST_REPORT_JUNIT_XML_OUTPUT_DIR: target/visual-reports
 
 jobs:
   build:
@@ -29,6 +30,10 @@ jobs:
         name: test-visual-images
         path: |
           /home/runner/work/_temp/**/*.png
-    - name: Publish Test Report
-      if: ${{ always() }}
-      uses: scacap/action-surefire-report@v1
+    - name: Test Report
+      uses: dorny/test-reporter@v1
+      if: success() || failure()
+      with:
+        name: Visual Tests
+        path: target/visual-reports/*.xml
+        reporter: java-junit

project.clj

@@ -12,8 +12,7 @@
   :pom-addition ([:properties ["maven.compiler.source" "8"] ["maven.compiler.target" "8"]])
   :pom-plugins [[org.apache.maven.plugins/maven-surefire-plugin "2.20"]]
   :main io.github.erdos.stencil.Main
-  :aliases      {"junit" ["with-profile" "+test" "test-out" "junit" "target/surefire-reports/clojure.xml"]
-                 "coverage" ["with-profile" "+ci" "cloverage" "--codecov"
+  :aliases      {"coverage" ["with-profile" "+ci" "cloverage" "--codecov"
                              "--exclude-call" "clojure.core/assert"
                              "--exclude-call" "stencil.util/trace"
                              "--exclude-call" "stencil.util/fail"
@@ -38,7 +37,7 @@
                     :dependencies [[junit/junit "4.13.2"]
                                    [org.xmlunit/xmlunit-core "2.5.1"]
                                    [hiccup "1.0.5"]]
-                    :plugins      [[lein-test-out "0.3.1"]]
+                    :plugins      [[lein-test-report-junit-xml "0.2.0"]]
                     :resource-paths    ["test-resources"]
                     :test-paths ["java-test"]
                     :injections [(require 'stencil.spec)