From 6fe2e19546f40fe0a46f3669edfa175808e40c22 Mon Sep 17 00:00:00 2001
From: shartte <shartte@users.noreply.github.com>
Date: Wed, 8 Nov 2023 23:02:14 +0100
Subject: [PATCH] Added a GHA Pull-Request Workflow (#35)

There is also a separate test-reporting workflow to allow the reporting of JUnit test results as a Github check, even if the build is running without write-permission (i.e. the PR is made from a fork).
---
 .github/workflows/pr.yml          | 47 +++++++++++++++++++++++++++++++
 .github/workflows/test-report.yml | 23 +++++++++++++++
 2 files changed, 70 insertions(+)
 create mode 100644 .github/workflows/pr.yml
 create mode 100644 .github/workflows/test-report.yml

diff --git a/.github/workflows/pr.yml b/.github/workflows/pr.yml
new file mode 100644
index 000000000..0896f29c7
--- /dev/null
+++ b/.github/workflows/pr.yml
@@ -0,0 +1,47 @@
+name: 'Build Pull Request'
+run-name: Build for PR ${{ github.event.pull_request.number }}
+
+on:
+  pull_request:
+    types:
+      - synchronize
+      - opened
+      - ready_for_review
+      - reopened
+
+jobs:
+  build:
+    name: Build and Test
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4
+        with:
+          fetch-depth: 1000
+          fetch-tags: true
+
+      - name: Setup JDK
+        uses: actions/setup-java@v3
+        with:
+          java-version: '17'
+          distribution: 'microsoft'
+
+      - name: Setup Gradle
+        uses: gradle/gradle-build-action@v2
+
+      - name: Build
+        run: ./gradlew --info -s -x check build
+
+      - name: Test
+        run: ./gradlew --info -s check
+
+      # Always upload test results
+      - name: Merge Test Reports
+        if: success() || failure()
+        run: npx junit-report-merger junit.xml "**/TEST-*.xml"
+
+      - uses: actions/upload-artifact@v3
+        if: success() || failure()
+        with:
+          name: test-results
+          path: junit.xml
diff --git a/.github/workflows/test-report.yml b/.github/workflows/test-report.yml
new file mode 100644
index 000000000..a8abebcde
--- /dev/null
+++ b/.github/workflows/test-report.yml
@@ -0,0 +1,23 @@
+name: 'Test Report'
+on:
+  workflow_run:
+    workflows: ['Build Pull Request']
+    types:
+      - completed
+permissions:
+  contents: read
+  actions: read
+  checks: write
+jobs:
+  report:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: neoforged/action-test-reporter@v1
+        with:
+          artifact: test-results
+          name: Test Report
+          path: '**/*.xml'
+          reporter: java-junit
+        # This should not affect the result of the check created
+        # for the origin PR. This just affects this post-processing job itself.
+        continue-on-error: true