RFR(L): 8199255: [TESTBUG] Open source VM testbase default methods tests

[mikhailo]

Hi Calvin,

   Thank you for review. I will fix the issue with vmTestbase_nsk_stress 
(merge issue) and will remove the blank line at line 1273 prior to check in.


Misha


On 05/23/2018 03:15 PM, Calvin Cheung wrote:
> Hi Misha,
>
> I've compared the file.list from your closed webrev with the one from 
> this open webrev and didn't see any missing files.
> Also spot checked a few copyright headers and they look good.
>
> Regarding TEST.groups, why was the following removed?
> 1160 vmTestbase_nsk_stress = \
> 1161   vmTestbase/nsk/stress
>
> Could you also remove the extra blank line added at line 1273?
>
> thanks,
> Calvin
>
> On 5/21/18, 11:34 AM, Mikhailo Seledtsov wrote:
>> Please review this change that will open source VM default method tests.
>> These tests have been used internally for a while, and are now being 
>> open sourced. Since this is not an creation of new tests, we would 
>> like to keep the changes during this review to a minimum required for 
>> open sourcing these tests, such as major issues and integration 
>> blockers. If you have other feedback regarding improvements to these 
>> tests, please file RFE(s) that will be addressed later in order of 
>> priority.
>>
>> Here is what was done for this change:
>>   1. Moved the tests to OpenJDK repository to the specified directory 
>> location and structure.
>>   3. Updated Copyright statements accordingly.
>>   4. Updated "@library" statements accordingly.
>>   5. Updated TEST.groups and a HotSpot test make file
>>
>>   JBS:    https://bugs.openjdk.java.net/browse/JDK-8199255
>>   Webrev: http://cr.openjdk.java.net/~mseledtsov/8199255.01/
>>
>>   Testing:
>>       1. Ran the following tests on open-only repository and build, 
>> using "make run-test" (Linux-x64)
>>          vmTestbase_vm_defmeth
>>          All PASS
>>
>>       2. Automated multip-platform test system (usual 4 platforms):
>>          - vmTestbase_vm_defmeth
>>          - hs-tier{1,2}
>>          In progress
>>
>>
>> Thank you,
>> Misha
>>