JDK-8211289 : dealing with duplications within test libraries
  • Type: Enhancement
  • Component: other-libs
  • Sub-Component: other
  • Affected Version: 12
  • Priority: P4
  • Status: Open
  • Resolution: Unresolved
  • Submitted: 2018-09-28
  • Updated: 2021-10-22
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
Other
tbdUnresolved
Related Reports
Blocks :  
JDK-8243927 - remove temporary workaround added by 8242314 and 8188828
Sub Tasks
JDK-8212033 :  
remove test/lib/jdk/test/lib/util/Compiler.java, use jdk.test.lib.compiler.InMemoryJavaCompiler instead - Open
JDK-8212035 :  
merge jdk.test.lib.util.SimpleHttpServer with jaxp.library.SimpleHttpServer - Resolved
JDK-8212076 :  
remove j.t.u.JarUtils' @deprecated methods - Open
JDK-8212077 :  
use j.t.l.RandomFactory instead of j.t.l.Utils.RANDOM_GENERATOR - Open
JDK-8212078 :  
merge j.t.l.BuildHelper and Platform - Open
JDK-8212079 :  
use j.t.l.u.JarUtils instead of JarBuilder - Open
Description
during work on 8075327, it has been noticed that there are logical duplications within the test libraries. such duplications increase a maintenance burden and also aggravate overall experience w/ the test libraries.

it appears that one of the main causes of this kind of duplication is a poor documentation, thus this RFE should also include writing the documentation and making it easily available for OpenJDK developers.
Comments
a few duplications in other test libraries have been also pointed out by JDK-8211290: - jaxp.library.SimpleHttpServer from test/jaxp/javax/xml/jaxp/libs and jdk.testlibrary.SimpleHttpServer from test/jdk/lib/testlibrary. suggestion: remove jdk.testlibrary.SimpleHttpServer, move jaxp.library.SimpleHttpServer to the top-level test library - test/jdk/lib/testlibrary/java/util/jar/JarBuilder.java and j.t.l.u.JarUtils. suggestion: remove JarBuilder - test/jdk/lib/testlibrary/java/util/jar/Compiler.java and j.t.l.c.InMemoryCompiler. suggestion: remove Compiler
02-10-2018
currently known duplications in /test/lib: - some of j.t.l.u.JarUtils methods. suggestion: remove @deprecated methods - j.t.l.Utils.RANDOM_GENERATOR and j.t.l.RandomFactory. suggestion: get rid of RANDOM_GENERATOR. - j.t.l.BuildHelper and Platform. Although there is no duplication per-se, some of Platform methods/fields should belong to BuildHelper; or BuildHelper should be removed
02-10-2018