Skip to content

Commit 7f2cbc4

Browse files
committed
fix test
1 parent ec8c005 commit 7f2cbc4

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
@@ -72,6 +72,8 @@ public class CrashlyticsControllerTest extends CrashlyticsTestCase {
7272
private DataCollectionArbiter mockDataCollectionArbiter;
7373
private CrashlyticsNativeComponent mockNativeComponent = mock(CrashlyticsNativeComponent.class);
7474

75+
private CrashlyticsWorker diskWriteWorker = new CrashlyticsWorker(TestOnlyExecutors.background());
76+
7577
@Override
7678
protected void setUp() throws Exception {
7779
super.setUp();
@@ -182,7 +184,7 @@ public CrashlyticsController build() {
182184
nativeComponent,
183185
analyticsEventLogger,
184186
mock(CrashlyticsAppQualitySessionsSubscriber.class),
185-
new CrashlyticsWorker(TestOnlyExecutors.background()));
187+
diskWriteWorker);
186188
return controller;
187189
}
188190
}
@@ -366,13 +368,14 @@ public void testLoggedExceptionsAfterCrashOk() {
366368
*/
367369
// FIXME: Validate this test works as intended
368370
@SdkSuppress(minSdkVersion = 30) // ApplicationExitInfo
369-
public void testLogStringAfterCrashOk() {
371+
public void testLogStringAfterCrashOk() throws Exception {
370372
final CrashlyticsController controller = builder().build();
371373
controller.handleUncaughtException(
372374
testSettingsProvider, Thread.currentThread(), new RuntimeException());
373375

374376
// This should not throw.
375-
controller.writeToLog(System.currentTimeMillis(), "Hi");
377+
diskWriteWorker.submit(() -> controller.writeToLog(System.currentTimeMillis(), "Hi"));
378+
diskWriteWorker.await();
376379
}
377380

378381
/**

0 commit comments

Comments
 (0)