Hi Roger, Thanks for the comments. Earlier as part of JDK-8062804 we considered scenarios for support of isofields and non isofields (positive case to check isofield works and negative case to check UnsupportedTemporalTypeException is thrown only.
Here is the updated webrev, I have added data check for field.rangeRefinedBy method. http://cr.openjdk.java.net/~bgopularam/JDK-8156718/webrev.02 Please review the changes. Thanks, Bhanu -----Original Message----- From: Roger Riggs Sent: Thursday, May 19, 2016 6:49 PM To: core-libs-dev@openjdk.java.net Subject: Re: RFR 8156718: Need tests for IsoFields getFrom for unsupported non-Iso Temporal fields Hi Bhanu, Can you remind me why the value from line: 459 field.rangeRefinedBy(LocalDate.now()); is not checked also? Is looks odd to see a value passed to a test and not have it verified. It would need to use a fixed date (the same), but that is fine for that test. Thanks, Roger On 5/19/2016 7:16 AM, Stephen Colebourne wrote: > Fine by me > Stephen > > On 19 May 2016 at 11:34, Bhanu Gopularam > <bhanu.prakash.gopula...@oracle.com> wrote: >> Thank you Nadeesh and Stephen. >> >> Here is the updated webrev link: >> http://cr.openjdk.java.net/~bgopularam/JDK-8156718/webrev.01 >> >> Please review. >> >> Bhanu >> >> -----Original Message----- >> From: Stephen Colebourne [mailto:scolebou...@joda.org] >> Sent: Tuesday, May 17, 2016 5:11 PM >> To: core-libs-dev >> Subject: Re: RFR 8156718: Need tests for IsoFields getFrom for >> unsupported non-Iso Temporal fields >> >> I would also like to see the test case methods be named "getFrom" not >> "getfrom". >> >> Stephen >> >> On 17 May 2016 at 05:18, nadeesh tv <nadeesh...@oracle.com> wrote: >>> Hi Bhanu, >>> >>> I think you should add a test case comparing the return value of >>> getFrom() >>> >>> ( Not an official reviewer) >>> >>> Regards, >>> Nadeesh >>> >>> On 5/16/2016 11:46 AM, Bhanu Gopularam wrote: >>>> Hi all, >>>> >>>> Could you please review fix for following issue. >>>> >>>> Bug id: https://bugs.openjdk.java.net/browse/JDK-8156718 >>>> >>>> Solution: Added tck tests for validating getFrom method for >>>> unsupported non-Iso temporal fields >>>> >>>> Webrev: >>>> http://cr.openjdk.java.net/~bgopularam/JDK-8156718/webrev.00/ >>>> >>>> Thanks, >>>> Bhanu >>> >>> -- >>> Thanks and Regards, >>> Nadeesh TV >>>
