RFR(jdk 13/java.xml) 8219705: Wrong media-type for a given serialization method

Joe Wang huizhe.wang at oracle.com
Mon Mar 11 19:15:26 UTC 2019



On 3/11/19, 11:54 AM, Lance Andersen wrote:
> Hi Joe,
>
>> On Mar 11, 2019, at 2:41 PM, Joe Wang <huizhe.wang at oracle.com 
>> <mailto:huizhe.wang at oracle.com>> wrote:
>>
>> Thanks Lance!
>>
>> The assertEquals message adds the name (prNames[i]) of the property 
>> (e.g. media-type) so that it shows "java.lang.AssertionError: 
>> media-type expected [text/html] but found [text/xml]" when fail.
>
> Yep, I saw that, just my personal taste I sometimes like a bit more 
> that is why I just mentioned it, it is all good either way :-)

I added a msg so that it displays in a bit detail in case of a failure, 
e.g.:
java.lang.AssertionError: The value of the property 'media-type' should 
be 'text/html' when the method is 'html'.
  expected [text/html] but found [text/xml]

How does that sound?

-Joe

>
>> There's also a debug message that displays all of the properties when 
>> the test passes:
>> method: actual: html, expected: html
>> version: actual: 4.0, expected: 4.0
>> indent: actual: yes, expected: yes
>> media-type: actual: text/html, expected: text/html
>>
>> Best,
>> Joe
>>
>> On 3/11/19, 11:31 AM, Lance Andersen wrote:
>>> Hi Joe
>>>
>>> I think this is fine.  I might consider beefing up the test 
>>> assertEquals  message  it is just the name
>>>
>>> Assert.assertEquals(value, prValues[i], prNames[i]);
>>>
>>>
>>> HTH
>>>
>>> Best
>>> Lance
>>>> On Mar 11, 2019, at 2:22 PM, Joe Wang <huizhe.wang at oracle.com 
>>>> <mailto:huizhe.wang at oracle.com>> wrote:
>>>>
>>>> Please review a fix for a JCK failure. All XML-related JCK and SQE 
>>>> tests passed after the patch.
>>>>
>>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8219705
>>>> webrevs: http://cr.openjdk.java.net/~joehw/jdk13/8219705/webrev/ 
>>>> <http://cr.openjdk.java.net/%7Ejoehw/jdk13/8219705/webrev/>
>>>>
>>>> Thanks,
>>>> Joe
>>>>
>>>
>>> <Mail Attachment.gif> 
>>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
>>> <http://oracle.com/us/design/oracle-email-sig-198324.gif><http://oracle.com/us/design/oracle-email-sig-198324.gif>
>>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>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 <mailto:Lance.Andersen at oracle.com>
>>>
>>>
>>>
>
> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
> <http://oracle.com/us/design/oracle-email-sig-198324.gif><http://oracle.com/us/design/oracle-email-sig-198324.gif>
> <http://oracle.com/us/design/oracle-email-sig-198324.gif>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 <mailto:Lance.Andersen at oracle.com>
>
>
>


More information about the core-libs-dev mailing list