Skip to content

Commit d4580f4

Browse files
committed
fix test
1 parent bad1a13 commit d4580f4

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

firebase-crashlytics/src/androidTest/java/com/google/firebase/crashlytics/internal/common/CrashlyticsControllerTest.java

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -74,6 +74,8 @@ public class CrashlyticsControllerTest extends CrashlyticsTestCase {
7474
private DataCollectionArbiter mockDataCollectionArbiter;
7575
private CrashlyticsNativeComponent mockNativeComponent = mock(CrashlyticsNativeComponent.class);
7676

77+
private CrashlyticsWorker diskWriteWorker = new CrashlyticsWorker(TestOnlyExecutors.background());
78+
7779
@Override
7880
protected void setUp() throws Exception {
7981
super.setUp();
@@ -187,7 +189,7 @@ public CrashlyticsController build() {
187189
nativeComponent,
188190
analyticsEventLogger,
189191
mock(CrashlyticsAppQualitySessionsSubscriber.class),
190-
new CrashlyticsWorker(TestOnlyExecutors.background()));
192+
diskWriteWorker);
191193
return controller;
192194
}
193195
}
@@ -377,13 +379,14 @@ public void testLoggedExceptionsAfterCrashOk() {
377379
*/
378380
// FIXME: Validate this test works as intended
379381
@SdkSuppress(minSdkVersion = 30) // ApplicationExitInfo
380-
public void testLogStringAfterCrashOk() {
382+
public void testLogStringAfterCrashOk() throws Exception {
381383
final CrashlyticsController controller = builder().build();
382384
controller.handleUncaughtException(
383385
testSettingsProvider, Thread.currentThread(), new RuntimeException());
384386

385387
// This should not throw.
386-
controller.writeToLog(System.currentTimeMillis(), "Hi");
388+
diskWriteWorker.submit(() -> controller.writeToLog(System.currentTimeMillis(), "Hi"));
389+
diskWriteWorker.await();
387390
}
388391

389392
/**

0 commit comments

Comments
 (0)