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

StepTest.java run fail by dragonwell8/dragonwell11 #3756

Closed
sendaoYan opened this issue Jun 13, 2022 · 0 comments · Fixed by #3757
Closed

StepTest.java run fail by dragonwell8/dragonwell11 #3756

sendaoYan opened this issue Jun 13, 2022 · 0 comments · Fixed by #3757

Comments

@sendaoYan
Copy link
Contributor

Describe the bug
jdk/test/com/sun/jdi/StepTest.java run fail by dragonwell8

To Reproduce
jtreg jdk/test/com/sun/jdi/StepTest.java

result:

java.lang.IllegalArgumentException: Bad method name/signature: com.alibaba.rcm.internal.AbstractResourceContainer.go:()V
        at TestScaffold.resumeTo(TestScaffold.java:844)
        at StepTest.runTests(StepTest.java:131)
        at TestScaffold.startTests(TestScaffold.java:429)
        at StepTest.main(StepTest.java:105)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:498)
        at com.sun.javatest.regtest.agent.MainWrapper$MainThread.run(MainWrapper.java:127)
        at java.lang.Thread.run(Thread.java:855)

Expected behavior
The reason of this testcase run fail, is dragonwell add class "com.alibaba.rcm.internal.AbstractResourceContainer", which has a method signature: "com.alibaba.rcm.internal.AbstractResourceContainer.go:()V", conflict with testcase's go() method.
image

If we modify testcase, then this test will run success(add "com.alibaba.*" as exclude in doStep method):

image

So, can we add this testcase to alibaba problemlist?

@sendaoYan sendaoYan changed the title StepTest.java run fail by dragonwell8 StepTest.java run fail by dragonwell8/dragonwell11 Jun 13, 2022
@llxia llxia closed this as completed in 13dcb35 Jun 14, 2022
llxia pushed a commit that referenced this issue Jun 14, 2022
* Exclude jdk_jdi testcase StepTest.java

Exclude jdk_jdi testcase com/sun/jdi/StepTest.java only vendor is alibaba

Fixes: #3756
Signed-off-by: sendaoYan <[email protected]>

* Exclude jdk_jdi testcase StepTest.java

Exclude jdk_jdi testcase com/sun/jdi/StepTest.java only vendor is alibaba

Fixes: #3756
Signed-off-by: sendaoYan <[email protected]>
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 a pull request may close this issue.

1 participant