Unit test build failures

Author: tejasd

firebase-perf/src/test/java/com/google/firebase/perf/metrics/NetworkRequestMetricBuilderTest.java

@@ -242,7 +242,7 @@ public void testSessionIdNotAddedIfPerfSessionIsNull() {
 
     int numberOfSessionIds = metricBuilder.getSessions().size();
 
-    new SessionManager(mock(GaugeManager.class), null, mock(AppStateMonitor.class));
+    new SessionManager(mock(GaugeManager.class), null);
 
     assertThat(metricBuilder.getSessions()).hasSize(numberOfSessionIds);
   }

firebase-perf/src/test/java/com/google/firebase/perf/metrics/TraceTest.java

@@ -1032,7 +1032,7 @@ public void testSessionIdNotAddedIfPerfSessionIsNull() {
 
     int numberOfSessionIds = trace.getSessions().size();
 
-    new SessionManager(mock(GaugeManager.class), null, mock(AppStateMonitor.class));
+    new SessionManager(mock(GaugeManager.class), null);
 
     assertThat(trace.getSessions()).hasSize(numberOfSessionIds);
 

firebase-perf/src/test/java/com/google/firebase/perf/session/SessionManagerTest.java

@@ -75,8 +75,7 @@ public void setApplicationContext_initializeGaugeMetadataManager()
       throws ExecutionException, InterruptedException {
     when(mockPerfSession.isVerbose()).thenReturn(true);
     InOrder inOrder = Mockito.inOrder(mockGaugeManager);
-    SessionManager testSessionManager =
-        new SessionManager(mockGaugeManager, mockPerfSession, mockAppStateMonitor);
+    SessionManager testSessionManager = new SessionManager(mockGaugeManager, mockPerfSession);
     testSessionManager.setApplicationContext(mockApplicationContext);
 
     inOrder.verify(mockGaugeManager).initializeGaugeMetadataManager(any());
@@ -90,8 +89,7 @@ public void setApplicationContext_initializeGaugeMetadataManager()
   public void testUpdatePerfSessionMakesGaugeManagerStopCollectingGaugesIfSessionIsNonVerbose() {
     forceNonVerboseSession();
 
-    SessionManager testSessionManager =
-        new SessionManager(mockGaugeManager, mockPerfSession, mockAppStateMonitor);
+    SessionManager testSessionManager = new SessionManager(mockGaugeManager, mockPerfSession);
     testSessionManager.updatePerfSession(createTestSession(1));
 
     verify(mockGaugeManager).stopCollectingGauges();
@@ -101,8 +99,7 @@ public void testUpdatePerfSessionMakesGaugeManagerStopCollectingGaugesIfSessionI
   public void testUpdatePerfSessionMakesGaugeManagerStopCollectingGaugesWhenSessionsDisabled() {
     forceSessionsFeatureDisabled();
 
-    SessionManager testSessionManager =
-        new SessionManager(mockGaugeManager, createTestSession(1), mockAppStateMonitor);
+    SessionManager testSessionManager = new SessionManager(mockGaugeManager, createTestSession(1));
     testSessionManager.updatePerfSession(createTestSession(2));
 
     verify(mockGaugeManager).stopCollectingGauges();
@@ -114,8 +111,7 @@ public void testSessionIdDoesNotUpdateIfPerfSessionRunsTooLong() {
     when(mockClock.getTime()).thenReturn(mockTimer);
 
     PerfSession session = new PerfSession(testSessionId(1), mockClock);
-    SessionManager testSessionManager =
-        new SessionManager(mockGaugeManager, session, mockAppStateMonitor);
+    SessionManager testSessionManager = new SessionManager(mockGaugeManager, session);
 
     assertThat(session.isSessionRunningTooLong()).isFalse();
 
@@ -138,8 +134,7 @@ public void testUpdatePerfSessionStartsCollectingGaugesIfSessionIsVerbose() {
     PerfSession newSession = createTestSession(2);
     newSession.setGaugeAndEventCollectionEnabled(true);
 
-    SessionManager testSessionManager =
-        new SessionManager(mockGaugeManager, previousSession, mockAppStateMonitor);
+    SessionManager testSessionManager = new SessionManager(mockGaugeManager, previousSession);
     testSessionManager.updatePerfSession(newSession);
     testSessionManager.setApplicationContext(mockApplicationContext);
 
@@ -149,8 +144,7 @@ public void testUpdatePerfSessionStartsCollectingGaugesIfSessionIsVerbose() {
 
   @Test
   public void testPerfSession_sessionAwareObjects_doesntNotifyIfNotRegistered() {
-    SessionManager testSessionManager =
-        new SessionManager(mockGaugeManager, mockPerfSession, mockAppStateMonitor);
+    SessionManager testSessionManager = new SessionManager(mockGaugeManager, mockPerfSession);
 
     FakeSessionAwareObject spySessionAwareObjectOne = spy(new FakeSessionAwareObject());
     FakeSessionAwareObject spySessionAwareObjectTwo = spy(new FakeSessionAwareObject());
@@ -165,8 +159,7 @@ public void testPerfSession_sessionAwareObjects_doesntNotifyIfNotRegistered() {
 
   @Test
   public void testPerfSession_sessionAwareObjects_NotifiesIfRegistered() {
-    SessionManager testSessionManager =
-        new SessionManager(mockGaugeManager, mockPerfSession, mockAppStateMonitor);
+    SessionManager testSessionManager = new SessionManager(mockGaugeManager, mockPerfSession);
 
     FakeSessionAwareObject spySessionAwareObjectOne = spy(new FakeSessionAwareObject());
     FakeSessionAwareObject spySessionAwareObjectTwo = spy(new FakeSessionAwareObject());
@@ -185,8 +178,7 @@ public void testPerfSession_sessionAwareObjects_NotifiesIfRegistered() {
 
   @Test
   public void testPerfSession_sessionAwareObjects_DoesNotNotifyIfUnregistered() {
-    SessionManager testSessionManager =
-        new SessionManager(mockGaugeManager, mockPerfSession, mockAppStateMonitor);
+    SessionManager testSessionManager = new SessionManager(mockGaugeManager, mockPerfSession);
 
     FakeSessionAwareObject spySessionAwareObjectOne = spy(new FakeSessionAwareObject());
     FakeSessionAwareObject spySessionAwareObjectTwo = spy(new FakeSessionAwareObject());