Change the open source logic to use firebase_cpp_cc_test instead of cc_test

The Firestore iOS SDK defines cc_test already, which is causing conflicts. Rename the C++ testing infrastructure to use firebase_cpp_cc_test, which is less likely to have issues.

PiperOrigin-RevId: 298426910

Author: a-maurice

app/rest/tests/CMakeLists.txt

@@ -1,4 +1,4 @@
-# Copyright 2019 Google
+# Copyright 2019 Google LLC
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -42,22 +42,22 @@ target_include_directories(sample_resource_lib
     ${FLATBUFFERS_SOURCE_DIR}/include
 )
 
-cc_test(firebase_app_rest_request_test
+firebase_cpp_cc_test(firebase_app_rest_request_test
   SOURCES
     request_test.h
     request_test.cc
   DEPENDS
     firebase_rest_lib
 )
 
-cc_test(firebase_app_rest_request_binary_test
+firebase_cpp_cc_test(firebase_app_rest_request_binary_test
   SOURCES
     request_binary_test.cc
   DEPENDS
     firebase_rest_lib
 )
 
-cc_test(firebase_app_rest_request_json_test
+firebase_cpp_cc_test(firebase_app_rest_request_json_test
   SOURCES
     ../request_json.h
     request_json_test.cc
@@ -67,43 +67,43 @@ cc_test(firebase_app_rest_request_json_test
     sample_resource_lib
 )
 
-cc_test(firebase_app_rest_response_test
+firebase_cpp_cc_test(firebase_app_rest_response_test
   SOURCES
     response_test.cc
   DEPENDS
     firebase_rest_lib
 )
 
-cc_test(firebase_app_rest_response_binary_test
+firebase_cpp_cc_test(firebase_app_rest_response_binary_test
   SOURCES
     response_binary_test.cc
   DEPENDS
     firebase_rest_lib
 )
 
-cc_test(firebase_app_rest_response_json_test
+firebase_cpp_cc_test(firebase_app_rest_response_json_test
   SOURCES
     response_json_test.cc
   DEPENDS
     firebase_rest_lib
     sample_resource_lib
 )
 
-cc_test(firebase_app_rest_util_test
+firebase_cpp_cc_test(firebase_app_rest_util_test
   SOURCES
     util_test.cc
   DEPENDS
     firebase_rest_lib
 )
 
-cc_test(firebase_app_rest_www_form_url_encoded_test
+firebase_cpp_cc_test(firebase_app_rest_www_form_url_encoded_test
   SOURCES
     www_form_url_encoded_test.cc
   DEPENDS
     firebase_rest_lib
 )
 
-cc_test(firebase_app_rest_transport_mock_test
+firebase_cpp_cc_test(firebase_app_rest_transport_mock_test
   SOURCES
     transport_mock_test.cc
     ../transport_mock.h
@@ -119,31 +119,31 @@ cc_test(firebase_app_rest_transport_mock_test
 #[[
 
 # google3 Dependency: FLAGS_test_tmpdir, CHECK(), CHECK_EQ
-cc_test(firebase_app_rest_request_file_test
+firebase_cpp_cc_test(firebase_app_rest_request_file_test
   SOURCES
     request_file_test.cc
   DEPENDS
     firebase_rest_lib
 )
 
 # google3 Dependency: absl/strings/escaping (absl::CEscape)
-cc_test(firebase_app_rest_gzipheader_unittest
+firebase_cpp_cc_test(firebase_app_rest_gzipheader_unittest
   SOURCES
     gzipheader_unittest.cc
   DEPENDS
     firebase_rest_lib
 )
 
 # google3 Dependency: absl/strings/escaping (absl::CEscape)
-cc_test(firebase_app_rest_zlibwrapper_unittest
+firebase_cpp_cc_test(firebase_app_rest_zlibwrapper_unittest
   SOURCES
     zlibwrapper_unittest.cc
   DEPENDS
     firebase_rest_lib
 )
 
 # google3 Dependency: net/.../http2server, net/util/ports.h (net_util::PickUnusedPort())
-cc_test(firebase_app_rest_transport_curl_test
+firebase_cpp_cc_test(firebase_app_rest_transport_curl_test
   SOURCES
     transport_curl_test.cc
   DEPENDS

cmake/test_rules.cmake

@@ -1,4 +1,4 @@
-# Copyright 2019 Google
+# Copyright 2019 Google LLC
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -14,10 +14,12 @@
 
 include(CMakeParseArguments)
 
-# cc_test(
+# firebase_cpp_cc_test(
 #   target
 #   SOURCES sources...
 #   DEPENDS libraries...
+#   INCLUDES include directories...
+#   DEFINES definitions...
 # )
 #
 # Defines a new test executable target with the given target name, sources, and
@@ -28,7 +30,7 @@ function(cc_test name)
   endif()
 
   set(multi DEPENDS SOURCES INCLUDES DEFINES)
-  # Parse the arguments into cc_test_SOURCES and cc_test_DEPENDS.
+  # Parse the arguments into cc_test_SOURCES, ..._DEPENDS, etc.
   cmake_parse_arguments(cc_test "" "" "${multi}" ${ARGN})
 
   list(APPEND cc_test_DEPENDS gmock gtest gtest_main)
@@ -96,7 +98,7 @@ function(ios_test_add_frameworks name)
                          GoogleDataTransport GoogleDataTransportCCTSupport
                          GoogleUtilities nanopb )
 
-  foreach(FRAMEWORK IN LISTS DEFAULT_FRAMEWORKS 
+  foreach(FRAMEWORK IN LISTS DEFAULT_FRAMEWORKS
           ios_test_add_frameworks_CUSTOM_FRAMEWORKS)
     LIST(APPEND INCLUDES "-framework ${FRAMEWORK}")
   endforeach()
@@ -105,7 +107,7 @@ function(ios_test_add_frameworks name)
   set(FRAMEWORK_INCLUDES ${INCLUDES} PARENT_SCOPE)
 endfunction()
 
-# cc_test_on_ios(
+# firebase_cpp_cc_test_on_ios(
 #   target
 #   HOST host
 #   SOURCES sources...
@@ -121,7 +123,7 @@ endfunction()
 
 # baseline FirebaseAnalytics.  DEFINES will be added to the
 # target_compile_definitions call.
-function(cc_test_on_ios name)
+function(firebase_cpp_cc_test_on_ios name)
   if (NOT IOS)
     return()
   endif()