Skip to content

Commit

Permalink
[FFI_test] Enable the excluded tests on z/OS in JDK21
Browse files Browse the repository at this point in the history
The change enables the excluded tests specific to the default
library loading and struct given the related issues have been
resolved via
ibmruntimes/openj9-openjdk-jdk21#159
and eclipse-openj9#19468.

Related: [Internal
441](https://github.ibm.com/runtimes/javanext/issues/441)

Signed-off-by: ChengJin01 <jincheng@ca.ibm.com>
  • Loading branch information
ChengJin01 authored and AswathySK committed Jun 4, 2024
1 parent ef712f6 commit 5a0e6e1
Show file tree
Hide file tree
Showing 4 changed files with 3 additions and 15 deletions.
12 changes: 0 additions & 12 deletions test/TestConfig/resources/excludes/latest_exclude_21.txt
Original file line number Diff line number Diff line change
Expand Up @@ -31,18 +31,6 @@ org.openj9.test.nestmates.NestAttributeTest:testGetNestMembersAPINestMemberWrong
org.openj9.test.nestmates.NestAttributeTest:testGetNestMembersAPINestMemberWrongNestHost NA generic-all
org.openj9.test.nestmates.NestAttributeTest:testGetNestMembersAPINestMemberWrongNestHost2 NA generic-all

org.openj9.test.jep442.downcall.InvalidDownCallTests:test_heapSegmentForStructArgument javanext/issues/441 zos_390-64
org.openj9.test.jep442.downcall.InvalidDownCallTests:test_invalidMemoryLayoutForMemAddr javanext/issues/441 zos_390-64
org.openj9.test.jep442.downcall.InvalidDownCallTests:test_invalidMemoryLayoutForReturnType javanext/issues/441 zos_390-64
org.openj9.test.jep442.downcall.PrimitiveTypeTests1:test_memoryAllocFreeFromDefaultLib_1 javanext/issues/441 zos_390-64
org.openj9.test.jep442.downcall.PrimitiveTypeTests1:test_printfFromDefaultLibWithMemAddr_1 javanext/issues/441 zos_390-64
org.openj9.test.jep442.downcall.PrimitiveTypeTests1:test_printfFromDefaultLibWithMemAddr_LinkerOption_1 javanext/issues/441 zos_390-64
org.openj9.test.jep442.downcall.PrimitiveTypeTests1:test_strlenFromDefaultLibWithMemAddr_1 javanext/issues/441 zos_390-64
org.openj9.test.jep442.downcall.PrimitiveTypeTests2:test_memoryAllocFreeFromDefaultLib_2 javanext/issues/441 zos_390-64
org.openj9.test.jep442.downcall.PrimitiveTypeTests2:test_printfFromDefaultLibWithMemAddr_2 javanext/issues/441 zos_390-64
org.openj9.test.jep442.downcall.PrimitiveTypeTests2:test_printfFromDefaultLibWithMemAddr_LinkerOption_2 javanext/issues/441 zos_390-64
org.openj9.test.jep442.downcall.PrimitiveTypeTests2:test_strlenFromDefaultLibWithMemAddr_2 javanext/issues/441 zos_390-64

# Exclude Java 19 Thread related failures

org.openj9.test.java.lang.Test_ThreadGroup:test_activeCount NA generic-all
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@
* verifies the downcalls with the shared downcall handlder (cached as soft reference in OpenJDK)
* in multithreading.
*/
@Test(groups = { "level.sanity", "disabled.os.zos" })
@Test(groups = { "level.sanity" })
public class MultiThreadingTests2 implements Thread.UncaughtExceptionHandler {
private volatile Throwable initException;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@
* [2] the test suite is mainly intended for the following Clinker API:
* MethodHandle downcallHandle(MemorySegment symbol, FunctionDescriptor function)
*/
@Test(groups = { "level.sanity", "disabled.os.zos" })
@Test(groups = { "level.sanity" })
public class StructTests1 {
private static boolean isAixOS = System.getProperty("os.name").toLowerCase().contains("aix");
private static Linker linker = Linker.nativeLinker();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@
* [2] the test suite is mainly intended for the following Clinker API:
* MethodHandle downcallHandle(FunctionDescriptor function)
*/
@Test(groups = { "level.sanity", "disabled.os.zos" })
@Test(groups = { "level.sanity" })
public class StructTests2 {
private static boolean isAixOS = System.getProperty("os.name").toLowerCase().contains("aix");
private static Linker linker = Linker.nativeLinker();
Expand Down

0 comments on commit 5a0e6e1

Please sign in to comment.