Skip to content

Commit 92e508e

Browse files
committed
Fix unit test.
1 parent 2032a92 commit 92e508e

File tree

1 file changed

+5
-7
lines changed

1 file changed

+5
-7
lines changed

appcheck/firebase-appcheck/src/test/java/com/google/firebase/appcheck/internal/DefaultFirebaseAppCheckTest.java

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -20,12 +20,11 @@
2020
import static org.mockito.Mockito.never;
2121
import static org.mockito.Mockito.verify;
2222
import static org.mockito.Mockito.when;
23-
import static org.robolectric.Shadows.shadowOf;
2423

25-
import android.os.Looper;
2624
import androidx.test.core.app.ApplicationProvider;
2725
import com.google.android.gms.tasks.Task;
2826
import com.google.android.gms.tasks.Tasks;
27+
import com.google.common.util.concurrent.MoreExecutors;
2928
import com.google.firebase.FirebaseApp;
3029
import com.google.firebase.appcheck.AppCheckProvider;
3130
import com.google.firebase.appcheck.AppCheckProviderFactory;
@@ -49,7 +48,7 @@
4948
/** Tests for {@link DefaultFirebaseAppCheck}. */
5049
@RunWith(RobolectricTestRunner.class)
5150
@Config(manifest = Config.NONE)
52-
@LooperMode(Mode.PAUSED)
51+
@LooperMode(Mode.LEGACY)
5352
public class DefaultFirebaseAppCheckTest {
5453

5554
private static final String EXCEPTION_TEXT = "exceptionText";
@@ -79,14 +78,14 @@ public void setup() {
7978
when(mockAppCheckProviderFactory.create(any())).thenReturn(mockAppCheckProvider);
8079
when(mockAppCheckProvider.getToken()).thenReturn(Tasks.forResult(validDefaultAppCheckToken));
8180

81+
// TODO(b/258273630): Use TestOnlyExecutors.background() instead of
82+
// MoreExecutors.directExecutor().
8283
defaultFirebaseAppCheck =
8384
new DefaultFirebaseAppCheck(
8485
mockFirebaseApp,
8586
() -> mockHeartBeatController,
86-
TestOnlyExecutors.background(),
87+
MoreExecutors.directExecutor(),
8788
TestOnlyExecutors.blocking());
88-
89-
shadowOf(Looper.getMainLooper()).idle();
9089
}
9190

9291
@Test
@@ -343,7 +342,6 @@ public void testGetToken_existingValidToken_forceRefresh_requestsNewToken() {
343342
defaultFirebaseAppCheck.installAppCheckProviderFactory(mockAppCheckProviderFactory);
344343

345344
defaultFirebaseAppCheck.getToken(/* forceRefresh= */ true);
346-
shadowOf(Looper.getMainLooper()).idle();
347345

348346
verify(mockAppCheckProvider).getToken();
349347
}

0 commit comments

Comments
 (0)