Fix up test to run on all platforms and sub return false

The test should run on all platforms to make sure they don't fail.
The sub for initialize analytics should default to returning false.
If it really not load in that case.

Author: AustinBenoit

analytics/integration_test/src/integration_test.cc

@@ -274,6 +274,7 @@ TEST_F(FirebaseAnalyticsTest, TestSetLogCallback) {
   EXPECT_TRUE(log_callback_called);
   firebase::analytics::SetLogCallback(nullptr);
 }
+#endif  // defined(_WIN32)
 
 TEST_F(FirebaseAnalyticsTest, TestDesktopInitialization) {
   // The analytics dll does not work on 32 bit systems.
@@ -283,24 +284,20 @@ TEST_F(FirebaseAnalyticsTest, TestDesktopInitialization) {
   bool initialized = firebase::analytics::IsDesktopInitialized();
 
 #if defined(_WIN32)
-  // On Windows, since we initialized in SetUpTestSuite, it should be true.
+  // On Windows, it should be true.
   EXPECT_TRUE(initialized);
 #else
-  // On other platforms, we made it return false.
+  // On other platforms, it should return false.
   EXPECT_FALSE(initialized);
 #endif
 }
 
 TEST_F(FirebaseAnalyticsTest, TestDesktopDebugMode) {
-  // The analytics dll does not work on 32 bit systems.
-  SKIP_TEST_ON_WINDOWS_32BIT;
-
   // SetDebugMode doesn't return anything, so we just call it to ensure
   // it doesn't crash.
   firebase::analytics::SetDesktopDebugMode(true);
   firebase::analytics::SetDesktopDebugMode(false);
 }
-#endif  // defined(_WIN32)
 
 TEST_F(FirebaseAnalyticsTest, TestLogEvents) {
   // Log an event with no parameters.

analytics/src/analytics_desktop_dynamic.c

@@ -132,7 +132,7 @@ static void Stub_GoogleAnalytics_EventParameters_Destroy(GoogleAnalytics_EventPa
 
 // Stub for GoogleAnalytics_Initialize
 static bool Stub_GoogleAnalytics_Initialize(GoogleAnalytics_Options* options) {
-    return 1;
+    return 0;
 }
 
 // Stub for GoogleAnalytics_LogEvent