Skip to content

Commit

Permalink
Skip Ignite test on Java 21
Browse files Browse the repository at this point in the history
  • Loading branch information
lukas-krecan committed Sep 23, 2023
1 parent 74f2d26 commit ccdec58
Show file tree
Hide file tree
Showing 2 changed files with 23 additions and 12 deletions.
13 changes: 3 additions & 10 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,14 +16,6 @@ jobs:
strategy:
matrix:
java: [ '17', '21']
# Kotlin needs access to java.util
include:
- java: '17'
maven-opts: --add-opens java.base/java.util=ALL-UNNAMED
maven-params: ''
- java: '21'
maven-opts: --add-opens java.base/java.util=ALL-UNNAMED
maven-params: ''
steps:
- uses: actions/checkout@v4
- name: Set up JDK ${{ matrix.java }}
Expand All @@ -40,8 +32,9 @@ jobs:
maven-
- name: Build with Maven
env:
MAVEN_OPTS: ${{ matrix.maven-opts }}
run: mvn ${{ matrix.maven-params }} test javadoc:javadoc
# Kotlin needs access to java.util
MAVEN_OPTS: --add-opens java.base/java.util=ALL-UNNAMED
run: mvn test javadoc:javadoc
- name: Publish Test Report
uses: mikepenz/action-junit-report@v4
if: always() # always run even if the previous step fails
Expand Down
22 changes: 20 additions & 2 deletions providers/ignite/shedlock-provider-ignite/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -55,9 +55,9 @@
</build>
<profiles>
<profile>
<id>surefire-java16</id>
<id>surefire-java17</id>
<activation>
<jdk>[16,)</jdk>
<jdk>[17,)</jdk>
</activation>
<build>
<plugins>
Expand All @@ -71,5 +71,23 @@
</plugins>
</build>
</profile>
<profile>
<!-- https://github.com/apache/ignite/issues/10720 -->
<id>skip-tests-in-java21</id>
<activation>
<jdk>21</jdk>
</activation>
<build>
<plugins>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-surefire-plugin</artifactId>
<configuration>
<skipTests>true</skipTests>
</configuration>
</plugin>
</plugins>
</build>
</profile>
</profiles>
</project>

0 comments on commit ccdec58

Please sign in to comment.