[threeten-dev] Please help to review new test cases for java.time.Year and YearMonth

Stephen Colebourne scolebourne at joda.org
Mon Apr 29 04:52:06 PDT 2013


Looks good to me
Stephen

On 28 April 2013 04:48, Patrick Zhang <patrick.zhang at oracle.com> wrote:
> Hi Team,
>
> Please help to review new added test case for java.time.Year and
> java.time.YearMonth.
> The webrev has been reviewed about before 1 month. But I forget to request
> it for pushing because of repo building problem on past.
> Now it has been modified according to latest test code and run again. :)
>
> Included apis:
>
> plus(long, TemporalUnit)
> minus(long, TemporalUnit)
> plus(TemporalAmount)
> minus(TemporalAmount)
> with(TemporalField, long)
> with(TemporalAdjuster)
>
> webrev:
> http://cr.openjdk.java.net/~pzhang/JSR310/java/time/Year/webrev/
>
> Test result:
> http://cr.openjdk.java.net/~pzhang/JSR310/java/time/Year/TCKYear.jtr
> http://cr.openjdk.java.net/~pzhang/JSR310/java/time/Year/TCKYearMonth.jtr
>


More information about the threeten-dev mailing list