RFR: JDK-8077371: Binary files in JAXP test should be removed [v4]
Lance Andersen
lancea at openjdk.org
Sat Apr 29 16:32:22 UTC 2023
On Thu, 27 Apr 2023 17:35:12 GMT, Mahendra Chhipa <mchhipa at openjdk.org> wrote:
>> Test is updated to create the binary files during test execution.
>
> Mahendra Chhipa has updated the pull request incrementally with one additional commit since the last revision:
>
> Implemented the review comments.
test/jdk/javax/xml/jaxp/datatype/8033980/SerializationTest.java line 85:
> 83:
> 84: // Generates the Java Pseudo code for base64 encoded string that can be cut & pasted into the test
> 85: final StringBuilder sb = new StringBuilder();
Can you please explain the purpose of the following code in the `setup` method?. This appears to be what you used to generate the _JDK**X**GregorianCalendarAndDurationSerData.java_ files. I don't see why this would be needed in a setup method.
Providing a method, along with comments of how to use it, to create byte arrays make sense, just not in a setup method.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13537#discussion_r1181098153
More information about the core-libs-dev
mailing list