RFR: 8073357: schema1.xsd has wrong content. Sequence of the enum values has been changed

[huizhe wang]

Hi Aleksej,

You may also consider using a golden/expected output file to be compared 
with the output. It makes a test more readable (easily see what's 
expected), easier to maintain with a separated golden file than multiple 
literal strings in the test, and technically simpler logic (e.g. 
runSchemaGen and then compareResult) and without opening the output file 
multiple times.

Regards,
Joe

On 4/20/2015 1:22 PM, Lance Andersen wrote:
> Hi Aleksej,
>
> The updates to the test seem reasonable.
>
> Best
> Lance
> On Apr 20, 2015, at 2:00 PM, Aleksej Efimov <aleksej.efimov at oracle.com> wrote:
>
>> Hello,
>>
>> The JDK9 schemagen tool hadn't preserved order of the enum values [1] and it was fixed in standalone project and was synced to JDK as part of JAXWS integration [2].
>> Can I have a review for the 'test/javax/xml/ws/8046817/GenerateEnumSchema.java' test update [3] to include test case for this functionality. The test and the partial backport of [2] will be backported to JDK8.
>> The modified test shows no failures with JPRT on all platforms.
>>
>> With Best Regards,
>> Aleksej
>>
>> [1] JBS bug: https://bugs.openjdk.java.net/browse/JDK-8073357
>> [2] JAXWS integration: https://bugs.openjdk.java.net/browse/JDK-8076549
>> [3] Webrev: http://cr.openjdk.java.net/~aefimov/8073357/9/00/
>
>
> Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
> Oracle Java Engineering
> 1 Network Drive
> Burlington, MA 01803
> Lance.Andersen at oracle.com
>
>
>