Skip to content

Commit 500d105

Browse files
committed
Fix flaky test
1 parent b09988d commit 500d105

2 files changed

Lines changed: 3 additions & 1 deletion

File tree

sentry-android-core/src/main/java/io/sentry/android/core/DefaultAndroidEventProcessor.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ final class DefaultAndroidEventProcessor implements EventProcessor {
3737

3838
private final @NotNull BuildInfoProvider buildInfoProvider;
3939
private final @NotNull SentryAndroidOptions options;
40-
private final @Nullable Future<DeviceInfoUtil> deviceInfoUtil;
40+
@TestOnly final @Nullable Future<DeviceInfoUtil> deviceInfoUtil;
4141
private final @NotNull LazyEvaluator<String> deviceFamily =
4242
new LazyEvaluator<>(() -> ContextUtils.getFamily(NoOpLogger.getInstance()));
4343

sentry-android-core/src/test/java/io/sentry/android/core/DefaultAndroidEventProcessorTest.kt

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -490,6 +490,7 @@ class DefaultAndroidEventProcessorTest {
490490
@Test
491491
fun `Event sets no device cpu info when there is none provided`() {
492492
val sut = fixture.getSut(context)
493+
sut.deviceInfoUtil?.get()
493494
CpuInfoUtils.getInstance().setCpuMaxFrequencies(emptyList())
494495
assertNotNull(sut.process(SentryEvent(), Hint())) {
495496
val device = it.contexts.device!!
@@ -501,6 +502,7 @@ class DefaultAndroidEventProcessorTest {
501502
@Test
502503
fun `Event sets rights device cpu info when there is one provided`() {
503504
val sut = fixture.getSut(context)
505+
sut.deviceInfoUtil?.get()
504506
CpuInfoUtils.getInstance().setCpuMaxFrequencies(listOf(800, 900))
505507

506508
assertNotNull(sut.process(SentryEvent(), Hint())) {

0 commit comments

Comments
 (0)