dimas-b commented on code in PR #1897:
URL: https://github.com/apache/polaris/pull/1897#discussion_r2153005510
##########
.github/workflows/gradle.yml:
##########
@@ -33,65 +33,99 @@ on:
branches: [ "main" ]
jobs:
+
+ # TODO remove this job in a future PR
build:
runs-on: ubuntu-latest
permissions:
contents: read
+ steps:
+ - name: Always succeeds
+ run: echo "Success!"
+ unit-tests:
+ name: Unit Tests
+ runs-on: ubuntu-latest
+ permissions:
+ contents: read
steps:
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4
- name: Set up JDK 21
uses: actions/setup-java@c5195efecf7bdfc987ee8bae7a71cb8b11521c00 # v4
with:
java-version: '21'
distribution: 'temurin'
-
- # Configure Gradle for optimal use in GiHub Actions, including caching
of downloaded dependencies.
- # See: https://github.com/gradle/actions/blob/main/setup-gradle/README.md
- name: Setup Gradle
uses:
gradle/actions/setup-gradle@ac638b010cf58a27ee6c972d7336334ccaf61c96 # v4
with:
- # The setup-gradle action fails, if the wrapper is not using the
right version or is not present.
- # Our `gradlew` validates the integrity of the `gradle-wrapper.jar`,
so it's safe to disable this.
validate-wrappers: false
-
- - name: Code style checks and tests
- run: ./gradlew --continue check
-
- - name: Check Maven publication
- run: ./gradlew publishToMavenLocal sourceTarball
-
+ - name: Run unit tests
+ run: |
+ ./gradlew check sourceTarball distTar distZip publishToMavenLocal \
+ -x :polaris-quarkus-service:test \
+ -x :polaris-quarkus-admin:test \
+ -x intTest --continue
- name: Archive test results
uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02
# v4
if: always()
with:
- name: upload-test-artifacts
+ name: upload-unit-test-artifacts
path: |
**/build/test-results/**
- - name: Stop Gradle daemons
- run: ./gradlew --stop
-
- # Ensure that the build works properly when building against the "latest
greatest" Java version
- - name: Set up JDK 23
+ quarkus-tests:
+ name: Quarkus Tests
+ runs-on: ubuntu-latest
+ permissions:
+ contents: read
+ steps:
+ - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4
+ - name: Set up JDK 21
uses: actions/setup-java@c5195efecf7bdfc987ee8bae7a71cb8b11521c00 # v4
with:
- java-version: '23'
+ java-version: '21'
distribution: 'temurin'
- - name: Show Java version
- run: java -version
- - name: Clean
- run: ./gradlew clean
- - name: Build
- run: ./gradlew compileAll
- - name: Run selected tests
- run: ./gradlew :polaris-quarkus-service:intTest
+ - name: Setup Gradle
+ uses:
gradle/actions/setup-gradle@ac638b010cf58a27ee6c972d7336334ccaf61c96 # v4
+ with:
+ validate-wrappers: false
+ - name: Run Quarkus tests
+ run: |
+ ./gradlew \
+ :polaris-quarkus-service:test \
Review Comment:
I'd assume `intTest` takes the longest time in general, so it might be
preferable to break it down if the goal is to reduce CI time :thinking:
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
