RFR(jdk 13/java.xml) 8219705: Wrong media-type for a given serialization method
Lance Andersen
lance.andersen at oracle.com
Mon Mar 11 20:11:50 UTC 2019
> On Mar 11, 2019, at 3:15 PM, Joe Wang <huizhe.wang at oracle.com> wrote:
>
>
>
> 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?
works for me
>
> -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 <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>
>>>>
>>>>
>>>>
>>
>> <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