Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

jdk24 OpenJDK java/foreign/passheapsegment/TestPassHeapSegment testNoHeapCaptureCallState should have thrown an exception of type class java.lang.IllegalArgumentException #21019

Closed
pshipton opened this issue Jan 25, 2025 · 3 comments · Fixed by adoptium/aqa-tests#5986
Closed

jdk24 OpenJDK java/foreign/passheapsegment/TestPassHeapSegment testNoHeapCaptureCallState should have thrown an exception of type class java.lang.IllegalArgumentException #21019

pshipton opened this issue Jan 25, 2025 · 3 comments · Fixed by adoptium/aqa-tests#5986
Assignees
@theresa-m
Labels
comp:vm jdk24 test excluded test failure
Milestone
Java 24 (0.50)

Comments

@pshipton
Copy link
Member

https://openj9-jenkins.osuosl.org/job/Test_openjdk24_j9_sanity.openjdk_aarch64_mac_Nightly_testList_1/3
java/foreign/passheapsegment/TestPassHeapSegment.java

20:46:31  test TestPassHeapSegment.testNoHeapCaptureCallState(): failure
20:46:31  org.testng.TestException: 
20:46:31  Method TestPassHeapSegment.testNoHeapCaptureCallState()[pri:0, instance:TestPassHeapSegment@634536a4] should have thrown an exception of type class java.lang.IllegalArgumentException
@pshipton pshipton added this to the Java 24 (0.50) milestone Jan 25, 2025
pshipton added a commit to pshipton/openjdk-tests that referenced this issue Jan 25, 2025
@pshipton
More temp excludes and one perm exclude for jdk24 OpenJ9
0e32ad0 
perm issue eclipse-openj9/openj9#21016

temp issues
eclipse-openj9/openj9#20357
eclipse-openj9/openj9#21017
eclipse-openj9/openj9#21018
eclipse-openj9/openj9#21019

Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
@pshipton pshipton mentioned this issue Jan 25, 2025
Merged
@pshipton
Copy link
Member Author

Exclude it via adoptium/aqa-tests#5912

llxia pushed a commit to adoptium/aqa-tests that referenced this issue Jan 26, 2025
@pshipton @llxia
More temp excludes and one perm exclude for jdk24 OpenJ9
6d65b47 
perm issue eclipse-openj9/openj9#21016

temp issues
eclipse-openj9/openj9#20357
eclipse-openj9/openj9#21017
eclipse-openj9/openj9#21018
eclipse-openj9/openj9#21019

Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
@babsingh
Copy link
Contributor

babsingh commented Feb 4, 2025

@theresa-m Can you also please take a look at this issue?

This might be related to #21018 (comment). The same extension repo commit also modifies TestPassHeapSegment testNoHeapCaptureCallState: 8336768: Allow captureCallState and critical linker options to be combined. There might be API changes that we need to reflect in our implementation. To begin the investigation, we should run the test with RI's latest JDK24 early-access build to see if it passes the test.

This was referenced Feb 26, 2025
Merged
Merged
@pshipton
Copy link
Member Author

pshipton commented Mar 4, 2025 •
edited
Loading

@theresa-m any updates? nm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@theresa-m theresa-m

Labels
comp:vm jdk24 test excluded test failure
Projects
None yet
Milestone
Java 24 (0.50)
Development

Successfully merging a pull request may close this issue.

Enable Java 24 ffi tests theresa-m/openjdk-tests
3 participants
@babsingh @theresa-m @pshipton