Merge "DO NOT MERGE:Deflaky testLoggingStats_search_success" into sc-v2-dev
This commit is contained in:
@@ -391,11 +391,12 @@ public class AppSearchLoggerTest {
|
||||
InitializeStats iStats = initStatsBuilder.build();
|
||||
|
||||
assertThat(iStats).isNotNull();
|
||||
// If the process goes really fast, the total latency could be 0. Since the default of total
|
||||
// latency is also 0, we just remove the assert about NativeLatencyMillis.
|
||||
assertThat(iStats.getStatusCode()).isEqualTo(AppSearchResult.RESULT_OK);
|
||||
// Total latency captured in LocalStorage
|
||||
assertThat(iStats.getTotalLatencyMillis()).isEqualTo(0);
|
||||
assertThat(iStats.hasDeSync()).isFalse();
|
||||
assertThat(iStats.getNativeLatencyMillis()).isGreaterThan(0);
|
||||
assertThat(iStats.getDocumentStoreDataStatus())
|
||||
.isEqualTo(InitializeStatsProto.DocumentStoreDataStatus.NO_DATA_LOSS_VALUE);
|
||||
assertThat(iStats.getDocumentCount()).isEqualTo(0);
|
||||
@@ -442,11 +443,12 @@ public class AppSearchLoggerTest {
|
||||
InitializeStats iStats = initStatsBuilder.build();
|
||||
|
||||
assertThat(iStats).isNotNull();
|
||||
// If the process goes really fast, the total latency could be 0. Since the default of total
|
||||
// latency is also 0, we just remove the assert about NativeLatencyMillis.
|
||||
assertThat(iStats.getStatusCode()).isEqualTo(AppSearchResult.RESULT_OK);
|
||||
// Total latency captured in LocalStorage
|
||||
assertThat(iStats.getTotalLatencyMillis()).isEqualTo(0);
|
||||
assertThat(iStats.hasDeSync()).isFalse();
|
||||
assertThat(iStats.getNativeLatencyMillis()).isGreaterThan(0);
|
||||
assertThat(iStats.getDocumentStoreDataStatus())
|
||||
.isEqualTo(InitializeStatsProto.DocumentStoreDataStatus.NO_DATA_LOSS_VALUE);
|
||||
assertThat(iStats.getDocumentCount()).isEqualTo(2);
|
||||
@@ -673,10 +675,11 @@ public class AppSearchLoggerTest {
|
||||
SearchStats sStats = mLogger.mSearchStats;
|
||||
|
||||
assertThat(sStats).isNotNull();
|
||||
// If the process goes really fast, the total latency could be 0. Since the default of total
|
||||
// latency is also 0, we just remove the assert about NativeLatencyMillis.
|
||||
assertThat(sStats.getPackageName()).isEqualTo(testPackageName);
|
||||
assertThat(sStats.getDatabase()).isEqualTo(testDatabase);
|
||||
assertThat(sStats.getStatusCode()).isEqualTo(AppSearchResult.RESULT_OK);
|
||||
assertThat(sStats.getTotalLatencyMillis()).isGreaterThan(0);
|
||||
assertThat(sStats.getVisibilityScope()).isEqualTo(SearchStats.VISIBILITY_SCOPE_LOCAL);
|
||||
assertThat(sStats.getTermCount()).isEqualTo(2);
|
||||
assertThat(sStats.getQueryLength()).isEqualTo(queryStr.length());
|
||||
|
||||
Reference in New Issue
Block a user