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.

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

Exclude tests using jdk.test.whitebox.WhiteBox class #4737

Merged
merged 1 commit into from
Aug 31, 2023

Conversation

fengxue-IS
Copy link
Contributor

Copy link
Contributor

@babsingh babsingh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We also need to temporarily exclude the other tests listed in eclipse-openj9/openj9#18037 (comment) using their corresponding OpenJ9 issues.

openjdk/excludes/ProblemList_openjdk21-openj9.txt Outdated Show resolved Hide resolved
@babsingh
Copy link
Contributor

We also need to temporarily exclude the other tests listed in eclipse-openj9/openj9#18037 (comment) using their corresponding OpenJ9 issues.

I see another PR, #4738, which addresses the above feedback.

Signed-off-by: Jack Lu <Jack.S.Lu@ibm.com>
Copy link
Contributor

@babsingh babsingh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@llxia @LongyuZhang Requesting your review.

Copy link
Contributor

@LongyuZhang LongyuZhang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@llxia llxia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@llxia llxia merged commit 16cfec3 into adoptium:master Aug 31, 2023
babsingh added a commit to babsingh/aqa-tests that referenced this pull request Sep 19, 2023
MethodExitTest is being permanently disabled. See
eclipse-openj9/openj9#17490 for more details on why
MethodExitTest is being permanently disabled.

MovingCompWindow and ToggleNotifyJvmtiTest were permanently disabled
in adoptium#4737. But it didn't correctly point to adoptium#1297. It pointed to
an OpenJ9 issue. This PR also corrects the link to adoptium#1297 for
MovingCompWindow and ToggleNotifyJvmtiTest.

Signed-off-by: Babneet Singh <sbabneet@ca.ibm.com>
babsingh added a commit to babsingh/aqa-tests that referenced this pull request Sep 19, 2023
MethodExitTest is being permanently disabled. See
eclipse-openj9/openj9#17490 for more details on why
MethodExitTest is being permanently disabled.

MovingCompWindow and ToggleNotifyJvmtiTest were permanently disabled
in adoptium#4737. But it didn't correctly point to adoptium#1297. It
pointed to an OpenJ9 issue. This PR also corrects the link to
adoptium#1297 for MovingCompWindow and ToggleNotifyJvmtiTest.

Signed-off-by: Babneet Singh <sbabneet@ca.ibm.com>
llxia pushed a commit that referenced this pull request Sep 20, 2023
MethodExitTest is being permanently disabled. See
eclipse-openj9/openj9#17490 for more details on why
MethodExitTest is being permanently disabled.

MovingCompWindow and ToggleNotifyJvmtiTest were permanently disabled
in #4737. But it didn't correctly point to #1297. It
pointed to an OpenJ9 issue. This PR also corrects the link to
#1297 for MovingCompWindow and ToggleNotifyJvmtiTest.

Signed-off-by: Babneet Singh <sbabneet@ca.ibm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

JTReg test failure due to java.lang.UnsatisfiedLinkError: jdk/test/whitebox/WhiteBox.registerNatives()V
4 participants