Added integration tests

Added integration test and removed the ability to set logging callback for now.  Will add in the future.

Author: john-mikhail

analytics/integration_test/src/integration_test.cc

@@ -259,6 +259,13 @@ TEST_F(FirebaseAnalyticsTest, TestLogEvents) {
                                 "spoon_welders");
 }
 
+TEST_F(FirebaseAnalyticsTest, TestNotifyAppLifecycleChange) {
+  // Can't confirm that these do anything but just run them all to ensure the
+  // app doesn't crash.
+  firebase::analytics::NotifyAppLifecycleChange(firebase::analytics::kUnknown);
+  firebase::analytics::NotifyAppLifecycleChange(firebase::analytics::kTermination);
+}
+
 TEST_F(FirebaseAnalyticsTest, TestLogEventWithMultipleParameters) {
   const firebase::analytics::Parameter kLevelUpParameters[] = {
       firebase::analytics::Parameter(firebase::analytics::kParameterLevel, 5),

analytics/src/analytics_desktop.cc

@@ -392,39 +392,6 @@ void NotifyAppLifecycleChange(AppLifecycleState state) {
       static_cast<GoogleAnalytics_AppLifecycleState>(state));
 }
 
-static void (*firebaseLogCallback)(LogLevel log_level, const char* log_message,
-                           void* callback_data) = nullptr;
-
-void SetLogCallback(LogCallback callback) {
-  FIREBASE_ASSERT_RETURN_VOID(internal::IsInitialized());
-
-  firebaseLogCallback = callback;
-
-  GoogleAnalytics_SetLogCallback(
-      [](GoogleAnalytics_LogLevel log_level, const char* message) {
-        if (firebaseLogCallback) {
-          LogLevel fbLogLevel;
-          switch (log_level) {
-            case GoogleAnalytics_LogLevel::kDebug:
-              fbLogLevel = LogLevel::kLogLevelDebug;
-              break;
-            case GoogleAnalytics_LogLevel::kInfo:
-              fbLogLevel = LogLevel::kLogLevelInfo;
-              break;
-            case GoogleAnalytics_LogLevel::kWarning:
-              fbLogLevel = LogLevel::kLogLevelWarning;
-              break;
-            case GoogleAnalytics_LogLevel::kError:
-              fbLogLevel = LogLevel::kLogLevelError;
-              break;
-            default:
-              fbLogLevel = LogLevel::kLogLevelInfo;
-          }
-          firebaseLogCallback(fbLogLevel, message, nullptr);
-        }
-    });
-}
-
 // Overloaded versions of LogEvent for convenience.
 
 void LogEvent(const char* name) {

analytics/src/include/firebase/analytics.h

@@ -23,8 +23,6 @@
 #include <string>
 #include <vector>
 
-#include "app/src/log.h"
-
 #include "firebase/app.h"
 #include "firebase/future.h"
 #include "firebase/internal/common.h"
@@ -585,13 +583,6 @@ enum AppLifecycleState {
 
 void NotifyAppLifecycleChange(AppLifecycleState state);
 
-/// @brief Allows the passing of a callback to be used when the SDK logs any
-/// *messages regarding its behavior.The callback must be thread -
-/// safe.
-/// 
-/// @param[in] callback The callback to use. Must be thread-safe.
-void SetLogCallback(LogCallback callback);
-
 /// Get the instance ID from the analytics service.
 ///
 /// @note This is *not* the same ID as the ID returned by