-
-
Notifications
You must be signed in to change notification settings - Fork 68
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
Check validity of modularity service loaders test #137
Comments
Hi @chandrams Since you had originally developed the use case for the modularity serviceloaders tests (if I am not mistaken), could you please shed some light on the following? Two of the 3 Modularity Service loaders tests fail, consistently, both on hotspot and openj9 jdk9 and jdk10 sdks. I am wondering if the tests have become invalid, or if it is pointing to a legitimate issue. Test 2 The first failure happens in Test 2, As you can see in the error message in the description of the issue, it lists only one display service (DisplayServiceImpl4) instead of all 4, as listed in test1. Test 3 I commented out test 2 for testing test 3 , and that too failed at the same validation as T2, but interestingly, this time we get 2 services in the 'actual' list, instead of 1 as seen in test2. https://urv-jenkins.canlab.ibm.com:9095/view/SV_Tests/job/test.personal/256/console
Have these two tests become invalid ? |
The failing tests have been excluded from running in the build : #138 |
Hi @Mesbah-Alam As discussed, these issues were seen when the tests were written for which an internal bug was raised. But this was during initial days of modularity, we will need to validate the test scenario and expected test behavior and raise a bug if required. |
@chandrams , I've dug up the internal RTC defect that you had raised on this issue against the JCL team in August 2016 (forwarded to you on Slack) and noticed that we have indeed had the exact failure since during early days. What was the source (modularity jep, etc) of the original use case, based on which this test was written? Could you point me to that? |
This is the modularity source link - http://openjdk.java.net/projects/jigsaw/spec/sotms/#services |
@chandrams, yes, I could use some help! Since you wrote it originally based on the spec, could you please explain the failure that has been happening? Was the test written correctly? Did it ever pass on any Oracle SDK? |
Modularity service loaders test fails with following error on all platforms on both hotspot and openj9 jdk9 and jdk10 builds. This test was written based on old modularity implementations and likely has become invalid. The check at which the test fails may need to be updated/ removed.
https://ci.adoptopenjdk.net/view/System%20tests/job/openjdk9_hs_systemtest_s390x_linux/27/tapResults/
The text was updated successfully, but these errors were encountered: