FirebaseApp: send Core Diagnostics data on appDidBecomeActive

Example/Core/Tests/FIRAppTest.m

@@ -15,11 +15,9 @@
 #import "FIRTestCase.h"
 #import "FIRTestComponents.h"
 
-#import <FirebaseCoreDiagnosticsInterop/FIRCoreDiagnosticsData.h>
-#import <FirebaseCoreDiagnosticsInterop/FIRCoreDiagnosticsInterop.h>
-
 #import <FirebaseCore/FIRAnalyticsConfiguration.h>
 #import <FirebaseCore/FIRAppInternal.h>
+#import <FirebaseCore/FIRCoreDiagnosticsConnector.h>
 #import <FirebaseCore/FIROptionsInternal.h>
 
 NSString *const kFIRTestAppName1 = @"test_app_name_1";
@@ -53,6 +51,7 @@ @interface FIRAppTest : FIRTestCase
 
 @property(nonatomic) id appClassMock;
 @property(nonatomic) id observerMock;
+@property(nonatomic) id mockCoreDiagnosticsConnector;
 @property(nonatomic) NSNotificationCenter *notificationCenter;
 
 @end
@@ -65,6 +64,7 @@ - (void)setUp {
   [FIRApp resetApps];
   _appClassMock = OCMClassMock([FIRApp class]);
   _observerMock = OCMObserverMock();
+  _mockCoreDiagnosticsConnector = OCMClassMock([FIRCoreDiagnosticsConnector class]);
 
   // TODO: Remove all usages of defaultCenter in Core, then we can instantiate an instance here to
   //       inject instead of using defaultCenter.
@@ -76,6 +76,7 @@ - (void)tearDown {
   [_notificationCenter removeObserver:_observerMock];
   _observerMock = nil;
   _notificationCenter = nil;
+  _mockCoreDiagnosticsConnector = nil;
 
   [super tearDown];
 }
@@ -730,6 +731,24 @@ - (void)testRegisteringNonConformingLibrary {
   XCTAssertFalse([[FIRApp firebaseUserAgent] containsString:@"InvalidLibrary`/1.0.0"]);
 }
 
+#pragma mark - Core Diagnostics
+
+- (void)testCoreDiagnosticsLoggedWhenFIRAppIsConfigured {
+  [self expectCoreDiagnosticsDataLogWithOptions:[self appOptions]];
+  [self createConfiguredAppWithName:NSStringFromSelector(_cmd)];
+  OCMVerifyAll(self.mockCoreDiagnosticsConnector);
+}
+
+- (void)testCoreDiagnosticsLoggedWhenAppDidBecomeActive {
+  FIRApp *app = [self createConfiguredAppWithName:NSStringFromSelector(_cmd)];
+  [self expectCoreDiagnosticsDataLogWithOptions:app.options];
+
+  [self.notificationCenter postNotificationName:[self appDidBecomeActiveNotificationName]
+                                         object:nil];
+
+  OCMVerifyAll(self.mockCoreDiagnosticsConnector);
+}
+
 #pragma mark - private
 
 - (void)expectNotificationForObserver:(id)observer
@@ -749,4 +768,35 @@ - (void)expectNotificationForObserver:(id)observer
   };
 }
 
+- (void)expectCoreDiagnosticsDataLogWithOptions:(nullable FIROptions *)expectedOptions {
+  OCMExpect(ClassMethod([self.mockCoreDiagnosticsConnector
+      logConfigureCoreWithOptions:[OCMArg checkWithBlock:^BOOL(FIROptions *options) {
+        if (!expectedOptions) {
+          return YES;
+        }
+        return [options.googleAppID isEqualToString:expectedOptions.googleAppID] &&
+               [options.GCMSenderID isEqualToString:expectedOptions.GCMSenderID];
+      }]]));
+}
+
+- (NSNotificationName)appDidBecomeActiveNotificationName {
+#if TARGET_OS_IOS || TARGET_OS_TV
+  return UIApplicationDidBecomeActiveNotification;
+#endif
+
+#if TARGET_OS_OSX
+  return NSApplicationDidBecomeActiveNotification;
+#endif
+}
+
+- (FIRApp *)createConfiguredAppWithName:(NSString *)name {
+  FIROptions *options = [self appOptions];
+  [FIRApp configureWithName:name options:options];
+  return [FIRApp appNamed:name];
+}
+
+- (FIROptions *)appOptions {
+  return [[FIROptions alloc] initWithGoogleAppID:kGoogleAppID GCMSenderID:kGCMSenderID];
+}
+
 @end

Firebase/Core/FIRApp.m

@@ -14,9 +14,15 @@
 
 #include <sys/utsname.h>
 
-#import "FIRApp.h"
+#if __has_include(<UIKit/UIKit.h>)
+#import <UIKit/UIKit.h>
+#endif
 
-#import <FirebaseCoreDiagnosticsInterop/FIRCoreDiagnosticsData.h>
+#if __has_include(<AppKit/AppKit.h>)
+#import <AppKit/AppKit.h>
+#endif
+
+#import "FIRApp.h"
 
 #import "Private/FIRAnalyticsConfiguration.h"
 #import "Private/FIRAppInternal.h"
@@ -283,15 +289,17 @@ - (instancetype)initInstanceWithName:(NSString *)name options:(FIROptions *)opti
   return self;
 }
 
+- (void)dealloc {
+  [[NSNotificationCenter defaultCenter] removeObserver:self];
+}
+
 - (BOOL)configureCore {
   [self checkExpectedBundleID];
   if (![self isAppIDValid]) {
     return NO;
   }
 
-  if ([self isDataCollectionDefaultEnabled]) {
-    [FIRCoreDiagnosticsConnector logConfigureCoreWithOptions:_options];
-  }
+  [self logDiagnostics];
 
 #if TARGET_OS_IOS
   // Initialize the Analytics once there is a valid options under default app. Analytics should
@@ -316,6 +324,8 @@ - (BOOL)configureCore {
   }
 #endif
 
+  [self subscribeForAppDidBecomeActiveNotifications];
+
   return YES;
 }
 
@@ -796,4 +806,31 @@ - (void)sendLogsWithServiceName:(NSString *)serviceName
 }
 #pragma clang diagnostic pop
 
+#pragma mark - App Life Cycle
+
+- (void)subscribeForAppDidBecomeActiveNotifications {
+#if TARGET_OS_IOS || TARGET_OS_TV
+  NSNotificationName notificationName = UIApplicationDidBecomeActiveNotification;
+#endif
+
+#if TARGET_OS_OSX
+  NSNotificationName notificationName = NSApplicationDidBecomeActiveNotification;
+#endif
+
+  [[NSNotificationCenter defaultCenter] addObserver:self
+                                           selector:@selector(appDidBecomeActive:)
+                                               name:notificationName
+                                             object:nil];
+}
+
+- (void)appDidBecomeActive:(NSNotification *)notification {
+  [self logDiagnostics];
+}
+
+- (void)logDiagnostics {
+  if ([self isDataCollectionDefaultEnabled]) {
+    [FIRCoreDiagnosticsConnector logConfigureCoreWithOptions:_options];
+  }
+}
+
 @end