RFR(M) : 8148244 : Finalize and integrate GTest implementation

Igor Ignatyev igor.ignatyev at oracle.com
Thu May 5 14:58:55 UTC 2016


Hi,

I’ve rebased the changes to the latest jdk9/hs and removed the comment from JvmMapfile.gmk. Also I’ve guarded building gtest w/ if TEST_IN_BUILD != false (http://cr.openjdk.java.net/~iignatyev/8148244-hotspot/webrev.02/make/lib/CompileGtest.gmk.html), so we won’t build it when we compile build jdk in case of cross-compliation.

new webrevs:
http://cr.openjdk.java.net/~iignatyev/8148244-root/webrev.02
http://cr.openjdk.java.net/~iignatyev/8148244-hotspot/webrev.02/

Thanks,
— Igor


> On May 2, 2016, at 3:55 PM, Jesper Wilhelmsson <jesper.wilhelmsson at oracle.com> wrote:
> 
> Even though the comment on line 155 in JvmMapfile.gmk is accurate I agree with removing it.
> 
> Besides that the change looks good!
> /Jesper
> 
> 
> Den 2/5/16 kl. 14:04, skrev Erik Joelsson:
>> Hello,
>> 
>> Build changes look good to me, but I also wrote most of them, except for
>> makefiles/lib/JvmMapfile.gmk line 155. I could not have written that. Please
>> remove.
>> 
>> The webrevs seem to be generated against jdk9/hs without the removal of the old
>> build system. I would recommend making that merge and generate new webrevs since
>> makefiles moved around quite a bit.
>> 
>> /Erik
>> 
>> On 2016-05-02 13:21, Igor Ignatyev wrote:
>>> http://cr.openjdk.java.net/~iignatyev/8148244-hotspot/webrev.00
>>>> 615 lines changed: 605 ins; 2 del; 8 mod;
>>> http://cr.openjdk.java.net/~iignatyev/8148244-root-wo-gtest/webrev.00/
>>>> 218 lines changed: 209 ins; 0 del; 9 mod;
>>> 
>>> http://cr.openjdk.java.net/~iignatyev/8148244-root/webrev.00/
>>>> 32029 lines changed: 32018 ins; 0 del; 11 mod;
>>> Hi all,
>>> 
>>> could you please review this patch which delivers core of JEP 281: HotSpot C++
>>> Unit-Test Framework[1]?
>>> in more details, this patch
>>>  - integrates sources of gtest-1.7.0[2] to /test/fmw/gtest (~32k LOC)
>>>  - introduces TEST macros (/hotspot/test/native/unittest.hpp)
>>>   - TEST — a basic unit test, doesn’t require an inited JVM
>>>   - TEST_VM — a test which requires an inited JVM, but expected to leave JVM
>>> in a valid state
>>>   - TEST_OTHER_VM — a test which requires a clean inited JVM
>>>   - TEST_VM_ASSERT/TEST_VM_ASSERT_MSG - tests to verify assert, “death”[3]
>>> tests which require inited JVM
>>>  - contains two tests as smoke tests for the project and examples. new tests
>>> should be added to /hotspot/test/native/ using the same directories layout as
>>> a corresponding product source files, and have prefix ‘test_’, e.g.
>>> test/native/runtime/test_os.cpp w/ tests for runtime/test_os.cpp
>>>  - updates makefiles to build/run tests : to run tests, one should use 'make
>>> test TEST=hotspot_gtest’
>>>  - adds hotspot_gtest to jprt testset
>>> 
>>> more information on the project can be found in JEP 281[1].
>>> 
>>> please be informed, that existing unit tests (aka internal tests) will be
>>> ported into new framework later, this activity is tracking by JDK-8077965[4].
>>> 
>>> PS This patch relies on JDK-8149591[5] which is under review[6].
>>> 
>>> Thanks,
>>> — Igor
>>> 
>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8148244
>>> webrevs:
>>>  root      : http://cr.openjdk.java.net/~iignatyev/8148244-root/webrev.00/
>>>  w/o gtest :
>>> http://cr.openjdk.java.net/~iignatyev/8148244-root-wo-gtest/webrev.00/
>>>  hotspot   : http://cr.openjdk.java.net/~iignatyev/8148244-hotspot/webrev.00
>>> 
>>> [1] https://bugs.openjdk.java.net/browse/JDK-8047975
>>> [2] https://github.com/google/googletest/releases/tag/release-1.7.0
>>> [3]
>>> https://github.com/google/googletest/blob/master/googletest/docs/V1_7_AdvancedGuide.md#death-tests
>>> 
>>> [4] https://bugs.openjdk.java.net/browse/JDK-8077965
>>> [5] https://bugs.openjdk.java.net/browse/JDK-8149591
>>> [6] http://mail.openjdk.java.net/pipermail/hotspot-dev/2016-April/022937.html
>> 



More information about the hotspot-dev mailing list