Skip to content

Commit

Permalink
Remove special casing of Java 21 test
Browse files Browse the repository at this point in the history
  • Loading branch information
Juuxel committed Jul 1, 2024
1 parent 9e4a70f commit 4944d0f
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 55 deletions.
50 changes: 0 additions & 50 deletions .github/workflows/test-push.yml
Original file line number Diff line number Diff line change
Expand Up @@ -148,53 +148,3 @@ jobs:
with:
name: Reproducible Build ${{ matrix.os }} (${{ matrix.java }}) Results
path: build/reports/

# Special case this test to run on Java 21
neoForge1206Test:
needs: build

strategy:
fail-fast: false
matrix:
java: [ 21 ]
os: [ ubuntu-22.04 ]

runs-on: ${{ matrix.os }}
steps:
- uses: actions/checkout@v4
- uses: actions/setup-java@v4
with:
java-version: ${{ matrix.java }}
distribution: 'temurin'

- run: ./gradlew test --tests *NeoForge1206Test --stacktrace --warning-mode fail

- uses: actions/upload-artifact@v4
if: ${{ failure() }}
with:
name: NeoForge 1.20.6 Build Results
# Special case this test to run on Java 21
forge1206Test:
needs: build

strategy:
fail-fast: false
matrix:
java: [ 21 ]
os: [ ubuntu-22.04 ]

runs-on: ${{ matrix.os }}
steps:
- uses: actions/checkout@v4
- uses: actions/setup-java@v4
with:
java-version: ${{ matrix.java }}
distribution: 'temurin'

- run: ./gradlew test --tests *Forge1206Test --stacktrace --warning-mode fail

- uses: actions/upload-artifact@v4
if: ${{ failure() }}
with:
name: Forge 1.20.6 Build Results
path: build/reports/
5 changes: 0 additions & 5 deletions build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -424,11 +424,6 @@ tasks.register('writeActionsTestMatrix') {
return
}

if (it.name.endsWith("NeoForge1206Test.groovy") || it.name.endsWith("Forge1206Test.groovy")) {
// Arch: The 1.20.6 tests require Java 21
return
}

def className = it.path.toString().replace(".groovy", "")
className = className.substring(className.lastIndexOf("integration/") + "integration/".length()).replace('/', '.')

Expand Down

0 comments on commit 4944d0f

Please sign in to comment.