Hi Joe, > On Mar 11, 2019, at 2:41 PM, Joe Wang <huizhe.w...@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 :-) > 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.w...@oracle.com >>> <mailto:huizhe.w...@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.ander...@oracle.com <mailto:lance.ander...@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.ander...@oracle.com <mailto:lance.ander...@oracle.com>