diff options
Diffstat (limited to 'java/tests/instrumentation/src/com/google/android/icing/IcingSearchEngineTest.java')
| -rw-r--r-- | java/tests/instrumentation/src/com/google/android/icing/IcingSearchEngineTest.java | 32 |
1 files changed, 14 insertions, 18 deletions
diff --git a/java/tests/instrumentation/src/com/google/android/icing/IcingSearchEngineTest.java b/java/tests/instrumentation/src/com/google/android/icing/IcingSearchEngineTest.java index 573ccfe..1ed2d9a 100644 --- a/java/tests/instrumentation/src/com/google/android/icing/IcingSearchEngineTest.java +++ b/java/tests/instrumentation/src/com/google/android/icing/IcingSearchEngineTest.java @@ -215,15 +215,13 @@ public final class IcingSearchEngineTest { assertThat(searchResultProto.getResultsCount()).isEqualTo(1); assertThat(searchResultProto.getResults(0).getDocument()).isEqualTo(emailDocument); - // TODO(b/236412954): Enable these JNI latency tests once cl/469819190 is synced to Jetpack - // Test that JNI latency has been set properly - // assertThat(searchResultProto.getQueryStats().hasNativeToJavaJniLatencyMs()).isTrue(); - // assertThat(searchResultProto.getQueryStats().hasNativeToJavaStartTimestampMs()).isTrue(); - // assertThat(searchResultProto.getQueryStats().hasJavaToNativeJniLatencyMs()).isTrue(); - // assertThat(searchResultProto.getQueryStats().getNativeToJavaJniLatencyMs()).isAtLeast(0); - // assertThat(searchResultProto.getQueryStats().getNativeToJavaStartTimestampMs()) - // .isGreaterThan(0); - // assertThat(searchResultProto.getQueryStats().getJavaToNativeJniLatencyMs()).isAtLeast(0); + assertThat(searchResultProto.getQueryStats().hasNativeToJavaStartTimestampMs()).isTrue(); + assertThat(searchResultProto.getQueryStats().hasNativeToJavaJniLatencyMs()).isTrue(); + assertThat(searchResultProto.getQueryStats().hasJavaToNativeJniLatencyMs()).isTrue(); + assertThat(searchResultProto.getQueryStats().getNativeToJavaStartTimestampMs()) + .isGreaterThan(0); + assertThat(searchResultProto.getQueryStats().getNativeToJavaJniLatencyMs()).isAtLeast(0); + assertThat(searchResultProto.getQueryStats().getJavaToNativeJniLatencyMs()).isAtLeast(0); } @Test @@ -266,15 +264,13 @@ public final class IcingSearchEngineTest { DocumentProto resultDocument = searchResultProto.getResults(0).getDocument(); assertThat(resultDocument).isEqualTo(documents.remove(resultDocument.getUri())); - // TODO(b/236412954): Enable these JNI latency tests once cl/469819190 is synced to Jetpack - // Test that JNI latency has been set - // assertThat(searchResultProto.getQueryStats().hasNativeToJavaJniLatencyMs()).isTrue(); - // assertThat(searchResultProto.getQueryStats().hasNativeToJavaStartTimestampMs()).isTrue(); - // assertThat(searchResultProto.getQueryStats().hasJavaToNativeJniLatencyMs()).isTrue(); - // assertThat(searchResultProto.getQueryStats().getNativeToJavaJniLatencyMs()).isAtLeast(0); - // assertThat(searchResultProto.getQueryStats().getNativeToJavaStartTimestampMs()) - // .isGreaterThan(0); - // assertThat(searchResultProto.getQueryStats().getJavaToNativeJniLatencyMs()).isAtLeast(0); + assertThat(searchResultProto.getQueryStats().hasNativeToJavaStartTimestampMs()).isTrue(); + assertThat(searchResultProto.getQueryStats().hasNativeToJavaJniLatencyMs()).isTrue(); + assertThat(searchResultProto.getQueryStats().hasJavaToNativeJniLatencyMs()).isTrue(); + assertThat(searchResultProto.getQueryStats().getNativeToJavaStartTimestampMs()) + .isGreaterThan(0); + assertThat(searchResultProto.getQueryStats().getNativeToJavaJniLatencyMs()).isAtLeast(0); + assertThat(searchResultProto.getQueryStats().getJavaToNativeJniLatencyMs()).isAtLeast(0); // fetch rest pages for (int i = 1; i < 5; i++) { |