<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