Merge "DO NOT MERGE:Deflaky testLoggingStats_search_success" into sc-dev
This commit is contained in:
@@ -351,11 +351,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);
|
||||
@@ -401,11 +402,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);
|
||||
@@ -628,10 +630,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