RFR 8156718: Need tests for IsoFields getFrom for unsupported non-Iso Temporal fields
Bhanu Gopularam
bhanu.prakash.gopularam at oracle.com
Fri May 20 09:22:27 UTC 2016
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 at 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.gopularam at 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:scolebourne at 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.tv at 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
>>>
More information about the core-libs-dev
mailing list