commit e74cdb3f8c33212618c090d90075b8d928f942f7
author Bankim Bhavsar <bankim@cloudera.com> Tue Feb 09 18:01:02 2021 -0800
committer Bankim Bhavsar <bankim@cloudera.com> Mon Feb 22 05:34:52 2021 +0000
tree 988e5b1e9d600071fc88856efc0f3e3e639e002a
parent 91f332a6936b8f9efd5e4a53768d4489f6ba79e1

[thirdparty] Upgrade gmock/gtest to 1.10.0

Upgrade gmock/gtest from 1.8.0 to latest stable 1.10.0 release
mainly to allow explicitly skipping tests using GTEST_SKIP()
macro. Includes bunch of other enhancements as well:
https://github.com/google/googletest/releases/tag/release-1.10.0

Earlier we were simply using gtest headers and not linking
gtest library. With the upgrade, kudu build fails with gtest
linker errors and hence building and linking gtest as well.

As per the gtest upgrade notes _TEST_CASE has been deprecated
in favor of _TEST_SUITE and hence replaced the usage accordingly
otherwise deprecation warnings are spewed on Mac OS build.

Note:
- Building the gmock/gtest in Debug mode names the library as
libgmockd/libgtestd instead of the expected libgmock/libgtest.
Though we could possibly update cmake modules to use the
explicitly debug named libraries or simply rename when copying
to installed directories. Using the default cmake build type
for building gmock/gtest instead of the earlier debug mode.

Tests:
- Verified debug build on Ubuntu 18.04, va1022 RHEL7 and
Mac OS 10.15.7

Change-Id: I1f7f5c11bef29a0d428d8d55d70ebdf888c6f645
Reviewed-on: http://gerrit.cloudera.org:8080/17056
Tested-by: Alexey Serbin <aserbin@cloudera.com>
Reviewed-by: Alexey Serbin <aserbin@cloudera.com>