Merge pull request #1928 from triplem/firebase_events_test

disable firebase for tests

.travis.yml

@@ -19,7 +19,7 @@ before_install:
 
 script:
   # Unit Test
-  - ./gradlew -Pcoverage testFullDebugUnitTest jacocoTestFullDebugUnitTestReport
+  - ./gradlew -Pcoverage -PfirebaseDisable testFullDebugUnitTest jacocoTestFullDebugUnitTestReport
 
 after_success:
   - bash <(curl -s https://codecov.io/bash)

app/build.gradle

@@ -143,6 +143,9 @@ android {
         debug {
             testCoverageEnabled(project.hasProperty('coverage'))
         }
+        firebaseDisable {
+            System.setProperty("disableFirebase", "true")
+        }
     }
     productFlavors {
         flavorDimensions "standard"

app/src/main/java/info/nightscout/androidaps/MainApp.java

@@ -148,6 +148,7 @@ public class MainApp extends Application {
         }
 
         mFirebaseAnalytics = FirebaseAnalytics.getInstance(this);
+        mFirebaseAnalytics.setAnalyticsCollectionEnabled(!Boolean.getBoolean("disableFirebase"));
 
         JodaTimeAndroid.init(this);
 

app/src/test/java/info/AAPSMocker.java

@@ -159,6 +159,7 @@ public class AAPSMocker {
     }
 
     public static MainApp mockMainApp() {
+        System.setProperty("disableFirebase", "true");
         PowerMockito.mockStatic(MainApp.class);
         MainApp mainApp = mock(MainApp.class);
         when(MainApp.instance()).thenReturn(mainApp);