diff options
Diffstat (limited to 'javatests/com/google/android/libraries/mobiledatadownload/internal/logging/MddEventLoggerTest.java')
-rw-r--r-- | javatests/com/google/android/libraries/mobiledatadownload/internal/logging/MddEventLoggerTest.java | 75 |
1 files changed, 43 insertions, 32 deletions
diff --git a/javatests/com/google/android/libraries/mobiledatadownload/internal/logging/MddEventLoggerTest.java b/javatests/com/google/android/libraries/mobiledatadownload/internal/logging/MddEventLoggerTest.java index dd1f027..d31534a 100644 --- a/javatests/com/google/android/libraries/mobiledatadownload/internal/logging/MddEventLoggerTest.java +++ b/javatests/com/google/android/libraries/mobiledatadownload/internal/logging/MddEventLoggerTest.java @@ -18,14 +18,18 @@ package com.google.android.libraries.mobiledatadownload.internal.logging; import static com.google.common.util.concurrent.Futures.immediateFuture; import static com.google.common.util.concurrent.MoreExecutors.directExecutor; + import static org.junit.Assert.assertFalse; import static org.junit.Assert.assertTrue; import static org.mockito.ArgumentMatchers.eq; import static org.mockito.Mockito.verify; +import static org.mockito.Mockito.verifyNoInteractions; import android.content.Context; import android.content.SharedPreferences; + import androidx.test.core.app.ApplicationProvider; + import com.google.android.libraries.mobiledatadownload.Logger; import com.google.android.libraries.mobiledatadownload.internal.logging.EventLogger.FileGroupStatusWithDetails; import com.google.android.libraries.mobiledatadownload.testing.FakeTimeSource; @@ -40,8 +44,7 @@ import com.google.mobiledatadownload.LogProto.MddDeviceInfo; import com.google.mobiledatadownload.LogProto.MddFileGroupStatus; import com.google.mobiledatadownload.LogProto.MddLogData; import com.google.mobiledatadownload.LogProto.StableSamplingInfo; -import java.security.SecureRandom; -import java.util.Random; + import org.junit.Before; import org.junit.Rule; import org.junit.Test; @@ -51,16 +54,21 @@ import org.mockito.junit.MockitoJUnit; import org.mockito.junit.MockitoRule; import org.robolectric.RobolectricTestRunner; +import java.security.SecureRandom; +import java.util.Random; + @RunWith(RobolectricTestRunner.class) public class MddEventLoggerTest { - @Rule public final MockitoRule mocks = MockitoJUnit.rule(); + @Rule + public final MockitoRule mocks = MockitoJUnit.rule(); private static final int SOME_MODULE_VERSION = 42; private static final int SAMPLING_ALWAYS = 1; private static final int SAMPLING_NEVER = 0; - @Mock private Logger mockLogger; + @Mock + private Logger mockLogger; private MddEventLogger mddEventLogger; private final Context context = ApplicationProvider.getApplicationContext(); @@ -79,7 +87,8 @@ public class MddEventLoggerTest { flags); mddEventLogger.setLoggingStateStore( SharedPreferencesLoggingState.create( - () -> loggingStateSharedPrefs, new FakeTimeSource(), directExecutor(), new Random(0))); + () -> loggingStateSharedPrefs, new FakeTimeSource(), directExecutor(), + new Random(0))); } private MddLogData.Builder newLogDataBuilderWithClientInfo() { @@ -105,31 +114,31 @@ public class MddEventLoggerTest { assertTrue(LogUtil.shouldSampleInterval(1)); } -// TODO: (b/239218521) Test will be ready once mockto v4 is available -// @Test -// public void testLogMddEvents_noLog() throws Exception { -// overrideDefaultSampleInterval(SAMPLING_NEVER); - -// DataDownloadFileGroupStats fileGroupStats = -// DataDownloadFileGroupStats.newBuilder() -// .setFileGroupName("fileGroup") -// .setFileGroupVersionNumber(1) -// .setBuildId(123) -// .setVariantId("testVariant") -// .build(); -// MddFileGroupStatus fileGroupStatus = -// MddFileGroupStatus.newBuilder() -// .setFileGroupDownloadStatus(MddFileGroupDownloadStatus.Code.COMPLETE) -// .build(); -// FileGroupStatusWithDetails fileGroupStatusWithDetails = -// FileGroupStatusWithDetails.create(fileGroupStatus, fileGroupStats); - -// mddEventLogger -// .logMddFileGroupStats(() -> immediateFuture(ImmutableList.of(fileGroupStatusWithDetails))) -// .get(); - -// verifyNoInteractions(mockLogger); -// } + @Test + public void testLogMddEvents_noLog() throws Exception { + overrideDefaultSampleInterval(SAMPLING_NEVER); + + DataDownloadFileGroupStats fileGroupStats = + DataDownloadFileGroupStats.newBuilder() + .setFileGroupName("fileGroup") + .setFileGroupVersionNumber(1) + .setBuildId(123) + .setVariantId("testVariant") + .build(); + MddFileGroupStatus fileGroupStatus = + MddFileGroupStatus.newBuilder() + .setFileGroupDownloadStatus(MddFileGroupDownloadStatus.Code.COMPLETE) + .build(); + FileGroupStatusWithDetails fileGroupStatusWithDetails = + FileGroupStatusWithDetails.create(fileGroupStatus, fileGroupStats); + + mddEventLogger + .logMddFileGroupStats( + () -> immediateFuture(ImmutableList.of(fileGroupStatusWithDetails))) + .get(); + + verifyNoInteractions(mockLogger); + } @Test public void testLogMddEvents() throws Exception { @@ -159,11 +168,13 @@ public class MddEventLoggerTest { .build(); mddEventLogger - .logMddFileGroupStats(() -> immediateFuture(ImmutableList.of(fileGroupStatusWithDetails))) + .logMddFileGroupStats( + () -> immediateFuture(ImmutableList.of(fileGroupStatusWithDetails))) .get(); verify(mockLogger) - .log(eq(expectedData), eq(MddClientEvent.Code.DATA_DOWNLOAD_FILE_GROUP_STATUS_VALUE)); + .log(eq(expectedData), + eq(MddClientEvent.Code.DATA_DOWNLOAD_FILE_GROUP_STATUS_VALUE)); } private void overrideDefaultSampleInterval(int sampleInterval) { |