Migrate from googletest 1.8 to googletest 1.10 (#67)

Summary:
X-link: facebookincubator/hsthrift#67

Updating `googletest` from `1.8.0` to `1.10.0`

Reviewed By: mzlee, igorsugak, luciang, meyering, r-barnes

Differential Revision: D34351084

fbshipit-source-id: 939b3985ab63a06b6d511ec8711c2d5863bdfea8

Author: dimitribouche

build/fbcode_builder/manifests/mvfst

@@ -25,7 +25,7 @@ folly
 fizz
 
 [dependencies.all(test=on, not(os=windows))]
-googletest_1_8
+googletest
 
 [shipit.pathmap]
 fbcode/quic/public_root = .

build/fbcode_builder/manifests/proxygen

@@ -32,7 +32,7 @@ wangle
 mvfst
 
 [dependencies.test=on]
-googletest_1_8
+googletest
 
 [shipit.pathmap]
 fbcode/proxygen/public_tld = .

cmake/ProxygenTest.cmake

@@ -7,8 +7,8 @@
 option(BUILD_TESTS  "Enable tests" OFF)
 include(CTest)
 if(BUILD_TESTS)
-  find_package(GMock 1.8.0 MODULE REQUIRED)
-  find_package(GTest 1.8.0 MODULE REQUIRED)
+  find_package(GMock 1.10.0 MODULE REQUIRED)
+  find_package(GTest 1.10.0 MODULE REQUIRED)
 endif()
 
 function(proxygen_add_test)
@@ -29,7 +29,7 @@ function(proxygen_add_test)
       set(PROXYGEN_TEST_SOURCES "${PROXYGEN_TEST_TARGET}.cpp")
     endif()
 
-    add_executable(${PROXYGEN_TEST_TARGET} 
+    add_executable(${PROXYGEN_TEST_TARGET}
       "${PROXYGEN_TEST_SOURCES}"
     )
 

proxygen/build.sh

@@ -159,7 +159,7 @@ function setup_googletest() {
   fi
   cd "$GTEST_DIR"
   git fetch --tags
-  git checkout release-1.8.0
+  git checkout release-1.10.0
   echo -e "${COLOR_GREEN}Building googletest ${COLOR_OFF}"
   mkdir -p "$GTEST_BUILD_DIR"
   cd "$GTEST_BUILD_DIR" || exit

proxygen/httpserver/Mocks.h

@@ -29,8 +29,6 @@ class MockResponseHandler : public ResponseHandler {
 #endif
 #endif
 
-// googletest switched APIs between multiple versions we target
-#if defined(MOCK_METHOD)
   MOCK_METHOD((void), pauseIngress, (), (noexcept));
   MOCK_METHOD((void), refreshTimeout, (), (noexcept));
   MOCK_METHOD((void), resumeIngress, (), (noexcept));
@@ -45,26 +43,8 @@ class MockResponseHandler : public ResponseHandler {
               newPushedResponse,
               (PushHandler*),
               (noexcept));
-#else
-  GMOCK_METHOD0_(, noexcept, , pauseIngress, void());
-  GMOCK_METHOD0_(, noexcept, , refreshTimeout, void());
-  GMOCK_METHOD0_(, noexcept, , resumeIngress, void());
-  GMOCK_METHOD0_(, noexcept, , sendAbort, void());
-  GMOCK_METHOD0_(, noexcept, , sendChunkTerminator, void());
-  GMOCK_METHOD0_(, noexcept, , sendEOM, void());
-  GMOCK_METHOD1_(, noexcept, , sendBody, void(std::shared_ptr<folly::IOBuf>));
-  GMOCK_METHOD1_(, noexcept, , sendChunkHeader, void(size_t));
-  GMOCK_METHOD1_(, noexcept, , sendHeaders, void(HTTPMessage&));
-  GMOCK_METHOD1_(, noexcept, , sendTrailers, void(const HTTPHeaders&));
-  GMOCK_METHOD1_(
-      ,
-      noexcept,
-      ,
-      newPushedResponse,
-      folly::Expected<ResponseHandler*, ProxygenError>(PushHandler*));
-#endif
+  MOCK_METHOD(void, getCurrentTransportInfo, (wangle::TransportInfo*), (const));
 
-  MOCK_CONST_METHOD1(getCurrentTransportInfo, void(wangle::TransportInfo*));
 #ifdef __clang__
 #pragma clang diagnostic pop
 #endif
@@ -92,7 +72,6 @@ class MockRequestHandler : public RequestHandler {
 #pragma clang diagnostic ignored "-Winconsistent-missing-override"
 #endif
 #endif
-#if defined(MOCK_METHOD)
   MOCK_METHOD((bool), canHandleExpect, (), (noexcept));
   MOCK_METHOD((void), onBody, (std::shared_ptr<folly::IOBuf>), (noexcept));
   MOCK_METHOD((void), onEOM, (), (noexcept));
@@ -104,19 +83,7 @@ class MockRequestHandler : public RequestHandler {
   MOCK_METHOD((void), onUpgrade, (UpgradeProtocol), (noexcept));
   MOCK_METHOD((void), requestComplete, (), (noexcept));
   MOCK_METHOD((void), setResponseHandler, (ResponseHandler*), (noexcept));
-#else
-  GMOCK_METHOD0_(, noexcept, , onEOM, void());
-  GMOCK_METHOD0_(, noexcept, , onEgressPaused, void());
-  GMOCK_METHOD0_(, noexcept, , requestComplete, void());
-  GMOCK_METHOD1_(, noexcept, , onBody, void(std::shared_ptr<folly::IOBuf>));
-  GMOCK_METHOD1_(, noexcept, , onError, void(ProxygenError));
-  GMOCK_METHOD1_(, noexcept, , onGoaway, void(ErrorCode));
-  GMOCK_METHOD1_(, noexcept, , onRequest, void(std::shared_ptr<HTTPMessage>));
-  GMOCK_METHOD1_(, noexcept, , onUpgrade, void(UpgradeProtocol));
-  GMOCK_METHOD1_(, noexcept, , setResponseHandler, void(ResponseHandler*));
-  GMOCK_METHOD0_(, noexcept, , onEgressResumed, void());
-  GMOCK_METHOD0_(, noexcept, , canHandleExpect, bool());
-#endif
+
 #ifdef __clang__
 #pragma clang diagnostic pop
 #endif

proxygen/httpserver/filters/tests/CompressionFilterTest.cpp

@@ -261,7 +261,7 @@ class CompressionFilterTest : public Test {
 
 typedef ::testing::Types<ZlibTest, ZstdTest> CompressionCodecs;
 
-TYPED_TEST_CASE(CompressionFilterTest, CompressionCodecs);
+TYPED_TEST_SUITE(CompressionFilterTest, CompressionCodecs);
 
 // Basic smoke test
 TYPED_TEST(CompressionFilterTest, NonchunkedCompression) {

proxygen/httpserver/samples/echo/test/EchoHandlerTest.cpp

@@ -19,8 +19,8 @@ using namespace testing;
 
 class MockEchoStats : public EchoStats {
  public:
-  MOCK_METHOD0(recordRequest, void());
-  MOCK_METHOD0(getRequestCount, uint64_t());
+  MOCK_METHOD(void, recordRequest, ());
+  MOCK_METHOD(uint64_t, getRequestCount, ());
 };
 
 class EchoHandlerFixture : public testing::Test {

proxygen/httpserver/tests/HTTPServerTest.cpp

@@ -193,20 +193,12 @@ TEST(HttpServerStartStop, TestUseExistingIoExecutor) {
 
 class MockRequestHandlerFactory : public RequestHandlerFactory {
  public:
-// googletest switched APIs between multiple versions we target
-#if defined(MOCK_METHOD)
   MOCK_METHOD((void), onServerStart, (folly::EventBase*), (noexcept));
   MOCK_METHOD((void), onServerStop, (), (noexcept));
   MOCK_METHOD((RequestHandler*),
               onRequest,
               (RequestHandler*, HTTPMessage*),
               (noexcept));
-#else
-  GMOCK_METHOD1_(, noexcept, , onServerStart, void(folly::EventBase* evb));
-  GMOCK_METHOD0_(, noexcept, , onServerStop, void());
-  GMOCK_METHOD2_(
-      , noexcept, , onRequest, RequestHandler*(RequestHandler*, HTTPMessage*));
-#endif
 };
 
 TEST(HttpServerStartStop, TestZeroThreadsMeansNumCPUs) {

proxygen/lib/http/codec/compress/experimental/simulator/HPACKQueueTests.cpp

@@ -150,4 +150,4 @@ TEST_P(HPACKQueueTests, QueueDeleted) {
   }
 }
 
-INSTANTIATE_TEST_CASE_P(Queue, HPACKQueueTests, ::testing::Values(0, 1, 2, 3));
+INSTANTIATE_TEST_SUITE_P(Queue, HPACKQueueTests, ::testing::Values(0, 1, 2, 3));

proxygen/lib/http/codec/compress/test/HPACKContextTests.cpp

@@ -320,6 +320,6 @@ TEST_P(HPACKContextTests, ContextUpdate) {
   }
 }
 
-INSTANTIATE_TEST_CASE_P(Context,
-                        HPACKContextTests,
-                        ::testing::Values(true, false));
+INSTANTIATE_TEST_SUITE_P(Context,
+                         HPACKContextTests,
+                         ::testing::Values(true, false));