Review request for JDK-8051709: Convert JAXP function tests: javax.xml.datatype to jtreg (testng) tests

[Lance Andersen]

Hi Frank,

On Jan 27, 2015, at 4:40 AM, Frank Yuan <frank.yuan at oracle.com> wrote:

> Thank you, Lance!
>  
> I applied some experience from your previous comments:)

I a glad they were useful :-)
>  
> I have a question for your comment, could you check it below in line?

See below
>  
> Best Regards
> Frank
>  
> From: Lance Andersen [mailto:lance.andersen at oracle.com] 
> Sent: Tuesday, January 27, 2015 3:24 AM
> To: Frank Yuan
> Cc: 'huizhe wang'; 'Core-Libs-Dev'; 'Gustavo Galimberti'
> Subject: Re: Review request for JDK-8051709: Convert JAXP function tests: javax.xml.datatype to jtreg (testng) tests
>  
> Hi Frank,
>  
> I think this looks good. 
>  
>  
> Not sure if you are going to add more tests in the future, but would be good to have tests such as
>  
> new Duration(x.toString()).equals(x)
>  
>  
> Perhaps a few more checks on expected toString() output….
>  
> //Frank: I will do it
>  
>  
> For XMLGregorianCalendarTest.java,  I would consider at some point adding more permutations of some of the tests that are validating a bugs(now that you are adding this as a new test suite to openjdk)
>  
> //Frank: I am not sure what you mean, which bug do you want me to add test for?

For example checkIsValid()

I would add a DataProvider and add more permutations to test so that you can reduce other potential errors..

Again, a nice to have for a next update.

The problem I always have when we add one off tests, it becomes very hard to manage your test suite and really understand the quality of your coverage.  Better to enhance existing tests to fill in weaknesses as this helps keep your test suite from getting out of control…



>  
> Best
> Lance
>  
> On Jan 26, 2015, at 1:42 AM, Frank Yuan <frank.yuan at oracle.com> wrote:
> 
> 
> Hi, Joe and All
> 
> We are working on moving internal jaxp functional tests to open jdk repo.
> This is the datatype suite. Would you please review these test?  Any comment
> will be appreciated.
> 
> bug: https://bugs.openjdk.java.net/browse/JDK-8051709
> webrev: http://cr.openjdk.java.net/~fyuan/8051709/webrev.00/
> 
> 
> Thanks,
> 
> Frank
> 
>  
> <image001.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
>  
> 
> 



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