[12] RFR of JDK-8210908: Refactor java/util/prefs/PrefsSpi.sh to plain java test

Brent Christian brent.christian at oracle.com
Tue Oct 23 20:39:13 UTC 2018


Hi, Amy

I think this looks quite good as it is.  Just a couple very minor 
comments, that you can take or leave:

* I don't think a List<String> is needed for the 'expected' argument, it 
can just be a String.  The @DataProvider should be able to return 
Object[]s of a List<String> and a String.

* Really, the argument to PrefsSpi is a regex pattern, rather than an 
expected result.

So overall testProvider() could look like:

     public void testProvider(List<String> opts, String pattern) throws 
Throwable

* Also, maybe update the comment (+ copyright year, I suppose) in 
PrefsSpi.java:

/*
  * main class used by regtest PrefsSpi.sh
  */

Thanks,
-Brent

On 10/22/18 8:44 PM, Amy Lu wrote:
> java/util/prefs/PrefsSpi.sh
> 
> Please review this patch to refactor above shell script test to java.
> 
> bug: https://bugs.openjdk.java.net/browse/JDK-8210908
> webrev: http://cr.openjdk.java.net/~amlu/8210908/webrev.00/
> 
> Thanks,
> Amy


More information about the core-libs-dev mailing list