<Beans Dev> [9] Review Request: 8132566 [TESTBUG] add regression test for inherited classes with the new bean annotations

Alexander Scherbatiy alexandr.scherbatiy at oracle.com
Wed Aug 5 15:39:13 UTC 2015


   The fix looks good to me.

   Thanks,
   Alexandr.

On 8/5/2015 6:40 PM, Alexander Stepanov wrote:
> Alexander,
>
> Sorry, could you please look at the updated version?
> http://cr.openjdk.java.net/~avstepan/8132566/webrev.01/
>
> A new test was added to cover JDK-8132669 (and now is failing because 
> of it):
> http://cr.openjdk.java.net/~avstepan/8132566/webrev.01/test/java/beans/Introspector/8132566/OverrideUserDefPropertyInfoTest.java.html 
>
>
> It was sent to the mailing list a couple of hours ago...
>
> Thanks,
> Alexander
>
>
> On 8/5/2015 6:27 PM, Alexander Scherbatiy wrote:
>>
>>   The fix looks good to me.
>>
>>   Thanks,
>>   Alexandr.
>>
>> On 8/4/2015 3:49 PM, Alexander Stepanov wrote:
>>> Hello,
>>>
>>> Could you please review the following fix:
>>> http://cr.openjdk.java.net/~avstepan/8132566/webrev.00/
>>> for
>>> https://bugs.openjdk.java.net/browse/JDK-8132566
>>>
>>> At the moment 'OverridePropertyInfoTest.java' will fail because of
>>> https://bugs.openjdk.java.net/browse/JDK-8132565
>>> (and, additionally,  both of them, - because of 
>>> https://bugs.openjdk.java.net/browse/JDK-8131939).
>>>
>>> If something like, e.g., (ClassProperty.isExpert() || 
>>> BaseClassProperty.isExpert()) is indeed should be returned when 
>>> requesting ClassProperty.isExpert(), then please let me know.
>>>
>>> Thanks,
>>> Alexander
>>
>



More information about the beans-dev mailing list