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

Erik Joelsson erik.joelsson at oracle.com
Mon May 2 12:04:19 UTC 2016


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