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

Igor Ignatyev igor.ignatyev at oracle.com
Mon May 2 11:21:09 UTC 2016


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