RFR(M): JDK-8165805 - [TESTBUG] CDS needs to support JVMTI CFLH - test development

[Mikhailo Seledtsov]

Thank you,

Misha

On 9/14/16, 6:26 PM, Ioi Lam wrote:
> Hi Misha,
>
> I've reviewed the second version. It looks good. No more comments.
>
> Thanks
> - Ioi
>
> On 9/13/16 10:08 PM, Mikhailo Seledtsov wrote:
>> Thank you again for review. I have implemented all the review 
>> recommendations.
>> The update also contains new test case from Jiangli to test super/sub 
>> class pair, where super and sub classes belong to different packages.
>> Thank you Jiangli for the test case.
>>
>> Here is an updated webrev: 
>> http://cr.openjdk.java.net/~mseledtsov/8165805.02/
>>
>> Thank you,
>> Misha
>>
>> On 9/9/16, 7:31 PM, Mikhailo Seledtsov wrote:
>>>     Please review the tests that accompany the changes for
>>>     "JDK-8078644 - CDS needs to support JVMTI CFLH"
>>>
>>>     The tests include several test cases that are produced by 
>>> combination
>>>     of transforming related classes: super/sub class and 
>>> interface/implementor.
>>>     The classes reside in CDS archive. Tests check to ensure that 
>>> correct
>>>     transformation is performed, classes are loaded from the archive 
>>> only when appropriate,
>>>     and that test process using the archive completes w/o 
>>> errors/crashes.
>>>
>>>
>>>     Details:
>>>
>>>     The main class is TransformRelatedClasses.java; start review 
>>> with this class.
>>>     Other classes are:
>>>       - TransformSuperAndSubClasses.java, 
>>> TransformInterfaceAndImplementor.java
>>>         the "actual tests", but really just test headers calling 
>>> into TransformRelatedClasses.java
>>>
>>>       - TransformTestCommon.java - common methods for transformation 
>>> tests
>>>
>>>       - TestEntry.java - an entry into combinatorial test table 
>>> representing a single test case
>>>
>>>       - CDSTestUtils.java, TransformUtil.java - utility classes that 
>>> are common  CDS
>>>         tests
>>>
>>>       - TransformerAgent.java - agent that performs transformation 
>>> (comes with .mf file)
>>>
>>>
>>>     JBS: https://bugs.openjdk.java.net/browse/JDK-8165805
>>>     Webrev: http://cr.openjdk.java.net/~mseledtsov/8165805.00/
>>>     Testing:
>>>         Ran these tests on a standard set of platforms thru 
>>> automated test system.
>>>
>>> Thank you,
>>> Misha
>>>
>