Skip to content

Commit

Permalink
JDK16: Disable 2 fault tolerance tests w.r.t. "opens java.lang.invoke"
Browse files Browse the repository at this point in the history
  • Loading branch information
famod committed May 7, 2021
1 parent 5f2ab8d commit d929049
Show file tree
Hide file tree
Showing 2 changed files with 25 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@

import org.junit.jupiter.api.Assertions;
import org.junit.jupiter.api.Disabled;
import org.junit.jupiter.api.Tag;
import org.junit.jupiter.api.Test;
import org.junit.jupiter.api.condition.DisabledOnOs;
import org.junit.jupiter.api.condition.OS;
Expand Down Expand Up @@ -178,6 +179,7 @@ public void testIssue8795() {

@Test
@DisableIfBuiltWithGraalVMOlderThan(GRAALVM_21_0)
@Tag("failsOnJDK16")
public void testFaultTolerance() {
RestAssured.when().get("/client/fault-tolerance").then()
.body(is("Hello fallback!"));
Expand Down
24 changes: 23 additions & 1 deletion tcks/microprofile-fault-tolerance/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@
<dependency>org.eclipse.microprofile.fault-tolerance:microprofile-fault-tolerance-tck</dependency>
</dependenciesToScan>
<reuseForks>false</reuseForks>
<excludes>
<excludes combine.children="append">
<!-- We do not support enablement via beans.xml -->
<exclude>org.eclipse.microprofile.fault.tolerance.tck.interceptor.xmlInterceptorEnabling.FaultToleranceInterceptorEnableByXmlTest</exclude>
</excludes>
Expand Down Expand Up @@ -85,4 +85,26 @@
</dependency>
</dependencies>

<profiles>
<profile>
<id>jdk16-workarounds</id>
<activation>
<jdk>[16,)</jdk>
</activation>
<build>
<plugins>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-surefire-plugin</artifactId>
<configuration>
<excludes>
<!-- fails with " module java.base does not "opens java.lang.invoke" to unnamed module" -->
<exclude>org.eclipse.microprofile.fault.tolerance.tck.fallbackmethod.FallbackMethodDefaultMethodTest</exclude>
</excludes>
</configuration>
</plugin>
</plugins>
</build>
</profile>
</profiles>
</project>

0 comments on commit d929049

Please sign in to comment.