Merge pull request #105 from jflo/ghaPRAnnot

Gha pr annot
jflo · Mar 14, 2024 · cece91c · cece91c
2 parents 9bf5427 + fa53803
commit cece91c
Show file tree

Hide file tree

Showing 5 changed files with 31 additions and 33 deletions.
diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md
@@ -1,3 +1,10 @@
+## PR description
+
+## Fixed Issue(s)
+<!-- Please link to fixed issue(s) here using format: fixes #<issue number> -->
+<!-- Example: "fixes #2" -->
+
+
 ### Thanks for sending a pull request! Have you done the following?
 
 - [ ] Checked out our [contribution guidelines](https://github.com/hyperledger/besu/blob/main/CONTRIBUTING.md)?
@@ -12,9 +19,3 @@
 - [ ] locally run all integration tests via: `./gradlew integrationTest`
 - [ ] locally run all reference tests via: `./gradlew ethereum:referenceTests:referenceTests`
 
-
-## PR description
-
-## Fixed Issue(s)
-<!-- Please link to fixed issue(s) here using format: fixes #<issue number> -->
-<!-- Example: "fixes #2" -->
diff --git a/.github/workflows/acceptance-tests.yml b/.github/workflows/acceptance-tests.yml
@@ -1,7 +1,7 @@
 name: acceptance-tests
 on:
   workflow_dispatch:
-  pull_request_target:
+  pull_request:
     branches:
       - main
       - release-*
@@ -70,12 +70,6 @@ jobs:
         with:
           name: acceptance-node-${{matrix.runner_index}}-test-results
           path: 'acceptance-tests/tests/build/test-results/**/TEST-*.xml'
-      - name: Publish Test Report
-        uses: mikepenz/action-junit-report@5f47764eec0e1c1f19f40c8e60a5ba47e47015c5
-        if: (success() || failure()) # always run even if the build step fails
-        with:
-          report_paths: 'acceptance-tests/tests/build/test-results/**/TEST-*.xml'
-          annotate_only: true
   accepttests-passed:
     name: "accepttests-passed"
     runs-on: ubuntu-22.04

diff --git a/.github/workflows/integration-tests.yml b/.github/workflows/integration-tests.yml
@@ -1,7 +1,7 @@
 name: integration-tests
 on:
   workflow_dispatch:
-  pull_request_target:
+  pull_request:
     branches:
       - main
       - release-*
@@ -36,11 +36,5 @@ jobs:
           cache-disabled: true
       - name: run integration tests
         run: ./gradlew integrationTest compileJmh
-      - name: Publish Test Report
-        uses: mikepenz/action-junit-report@5f47764eec0e1c1f19f40c8e60a5ba47e47015c5
-        if: (success() || failure())
-        with:
-          report_paths: '**/build/test-results/integrationTest/TEST-*.xml'
-          annotate_only: true
 
 
diff --git a/.github/workflows/pre-review.yml b/.github/workflows/pre-review.yml
@@ -1,7 +1,7 @@
 name: pre-review
 
 on:
-  pull_request_target:
+  pull_request:
     branches:
       - main
       - release-*
@@ -91,6 +91,22 @@ jobs:
           - "ethereum:api:testRemainder"
           - "ethereum:eth:test"
           - "ethereum:core:test"
+        #includes will need exact strings from gradle args above
+        include:
+          - gradle_args: "test -x besu:test -x consensus:test -x crypto:test -x ethereum:eth:test -x ethereum:api:test -x ethereum:core:test"
+            filename: "everythingElse"
+          - gradle_args: "besu:test consensus:test crypto:test"
+            filename: "consensusCrypto"
+          - gradle_args: "ethereum:api:testBonsai"
+            filename: "apiBonsai"
+          - gradle_args: "ethereum:api:testRemainder"
+            filename: "apiForest"
+          - gradle_args: "ethereum:api:testRemainder"
+            filename: "apiRemainder"
+          - gradle_args: "ethereum:eth:test"
+            filename: "eth"
+          - gradle_args: "ethereum:core:test"
+            filename: "core"
     steps:
       - name: Checkout Repo
         uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11
@@ -108,12 +124,11 @@ jobs:
       - name: run unit tests
         id: unitTest
         run: ./gradlew $GRADLEW_UNIT_TEST_ARGS
-      - name: Publish Test Report
-        uses: mikepenz/action-junit-report@5f47764eec0e1c1f19f40c8e60a5ba47e47015c5
-        if: success() || failure() # always run even if the build step fails
+      - name: Upload Acceptance Test Results
+        uses: actions/upload-artifact@5d5d22a31266ced268874388b861e4b58bb5c2f3
         with:
-          report_paths: '**/test-results/**/TEST-*.xml'
-          annotate_only: true
+          name: unit-${{matrix.filename}}-test-results
+          path: '**/test-results/**/TEST-*.xml'
   unittests-passed:
     name: "unittests-passed"
     runs-on: ubuntu-22.04

diff --git a/.github/workflows/reference-tests.yml b/.github/workflows/reference-tests.yml
@@ -1,7 +1,7 @@
 name: reference-tests
 on:
   workflow_dispatch:
-  pull_request_target:
+  pull_request:
     branches:
       - main
       - release-*
@@ -62,12 +62,6 @@ jobs:
         with:
           name: reference-test-node-${{matrix.runner_index}}-results
           path: '**/build/test-results/referenceTests/TEST-*.xml'
-      - name: Publish Test Report
-        uses: mikepenz/action-junit-report@5f47764eec0e1c1f19f40c8e60a5ba47e47015c5
-        if: success() || failure() # always run even if the build step fails
-        with:
-          report_paths: '**/build/test-results/referenceTests/TEST-*.xml'
-          annotate_only: true
   reftests-passed:
     name: "reftests-passed"
     runs-on: ubuntu-22.04