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

Alexander Stepanov alexander.v.stepanov at oracle.com
Wed Aug 5 15:40:24 UTC 2015


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