Skip to content

Commit

Permalink
Fix GitHub Actions tests: invoke the correct task!
Browse files Browse the repository at this point in the history
The JDK 21 / Gradle 8.5 job was also missing a project property.
  • Loading branch information
tbroyer committed Dec 10, 2023
1 parent 096ebdd commit 4cca0c4
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions .github/workflows/gradle.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -75,14 +75,14 @@ jobs:
id: gradle-build
uses: gradle/gradle-build-action@v2
with:
arguments: --scan test "-Ptest.gradle-version=${{ matrix.gradle }}"
arguments: --scan integrationTest "-Ptest.gradle-version=${{ matrix.gradle }}"

- name: Store reports
if: always() && (steps.gradle-build.outcome == 'success' || steps.gradle-build.outcome == 'failure')
uses: actions/upload-artifact@v3
with:
name: gradle-${{ matrix.gradle }}-tests
path: build/reports/tests/test/
path: build/reports/tests/integrationTest/

test-jdk:
needs: linux
Expand All @@ -109,14 +109,14 @@ jobs:
id: gradle-build
uses: gradle/gradle-build-action@v2
with:
arguments: --scan test "-Ptest.java-toolchain=${{ matrix.java }}"
arguments: --scan integrationTest "-Ptest.java-toolchain=${{ matrix.java }}"

- name: Store reports
if: always() && (steps.gradle-build.outcome == 'success' || steps.gradle-build.outcome == 'failure')
uses: actions/upload-artifact@v3
with:
name: jdk-${{ matrix.java }}-tests
path: build/reports/tests/test/
path: build/reports/tests/integrationTest/

test-jdk21-gradle85:
needs: linux
Expand All @@ -140,11 +140,11 @@ jobs:
id: gradle-build
uses: gradle/gradle-build-action@v2
with:
arguments: --scan test "-Ptest.java-toolchain=21"
arguments: --scan integrationTest "-Ptest.java-toolchain=21" "-Ptest.gradle-version=8.5"

- name: Store reports
if: always() && (steps.gradle-build.outcome == 'success' || steps.gradle-build.outcome == 'failure')
uses: actions/upload-artifact@v3
with:
name: jdk-21-tests
path: build/reports/tests/test/
path: build/reports/tests/integrationTest/

0 comments on commit 4cca0c4

Please sign in to comment.