diff options
author | Alexander Dorokhine <adorokhine@google.com> | 2020-10-02 23:42:38 -0700 |
---|---|---|
committer | Alexander Dorokhine <adorokhine@google.com> | 2020-10-13 11:45:45 -0700 |
commit | 6fbec140f2654dc0318d8224afc7b8b1b524b0d8 (patch) | |
tree | c847707a688e0bf1c50c020603f1e4790d427995 /services/robotests | |
parent | 9c0f722e2eccdec98bf93f3aeb0a0e3e5b79c569 (diff) |
Migrate away from deprecated Truth APIs.
This is a transitional step towards truth 1.0.1, where these APIs have
been completely removed.
Bug: 168765701
Test: m checkbuild
Exempt-From-Owner-Approval: Cherry-pick of no-op refactor into another branch
Merged-In: I26ab5ab82bb939bbd9553c05387ac8641eb468b4
Change-Id: I26ab5ab82bb939bbd9553c05387ac8641eb468b4
(cherry picked from commit 4697f76edd28cc9363c5ca099a6f9c311c1aee50)
Diffstat (limited to 'services/robotests')
-rw-r--r-- | services/robotests/backup/src/com/android/server/backup/keyvalue/KeyValueBackupTaskTest.java | 5 | ||||
-rw-r--r-- | services/robotests/backup/src/com/android/server/backup/testing/TestUtils.java | 18 |
2 files changed, 11 insertions, 12 deletions
diff --git a/services/robotests/backup/src/com/android/server/backup/keyvalue/KeyValueBackupTaskTest.java b/services/robotests/backup/src/com/android/server/backup/keyvalue/KeyValueBackupTaskTest.java index ec56e1ebc8e0..bd37e587aee7 100644 --- a/services/robotests/backup/src/com/android/server/backup/keyvalue/KeyValueBackupTaskTest.java +++ b/services/robotests/backup/src/com/android/server/backup/keyvalue/KeyValueBackupTaskTest.java @@ -41,6 +41,7 @@ import static com.android.server.backup.testing.Utils.oneTimeIterable; import static com.android.server.backup.testing.Utils.transferStreamedData; import static com.google.common.truth.Truth.assertThat; +import static com.google.common.truth.Truth.assertWithMessage; import static org.mockito.ArgumentMatchers.any; import static org.mockito.ArgumentMatchers.anyInt; @@ -2814,8 +2815,8 @@ public class KeyValueBackupTaskTest { } private static IterableSubject assertDirectory(Path directory) throws IOException { - return assertThat(oneTimeIterable(Files.newDirectoryStream(directory).iterator())) - .named("directory " + directory); + return assertWithMessage("directory " + directory).that( + oneTimeIterable(Files.newDirectoryStream(directory).iterator())); } private static void assertJournalDoesNotContain( diff --git a/services/robotests/backup/src/com/android/server/backup/testing/TestUtils.java b/services/robotests/backup/src/com/android/server/backup/testing/TestUtils.java index 3fe1f3f90f2f..3114a751d556 100644 --- a/services/robotests/backup/src/com/android/server/backup/testing/TestUtils.java +++ b/services/robotests/backup/src/com/android/server/backup/testing/TestUtils.java @@ -17,6 +17,7 @@ package com.android.server.backup.testing; import static com.google.common.truth.Truth.assertThat; +import static com.google.common.truth.Truth.assertWithMessage; import static org.robolectric.Shadows.shadowOf; @@ -95,8 +96,8 @@ public class TestUtils { * logcat before that. */ public static void assertLogcatAtMost(String tag, int level) { - assertThat(ShadowLog.getLogsForTag(tag).stream().allMatch(logItem -> logItem.type <= level)) - .named("All logs <= " + level) + assertWithMessage("All logs <= " + level).that( + ShadowLog.getLogsForTag(tag).stream().allMatch(logItem -> logItem.type <= level)) .isTrue(); } @@ -105,8 +106,8 @@ public class TestUtils { * logcat before that. */ public static void assertLogcatAtLeast(String tag, int level) { - assertThat(ShadowLog.getLogsForTag(tag).stream().anyMatch(logItem -> logItem.type >= level)) - .named("Any log >= " + level) + assertWithMessage("Any log >= " + level).that( + ShadowLog.getLogsForTag(tag).stream().anyMatch(logItem -> logItem.type >= level)) .isTrue(); } @@ -121,11 +122,10 @@ public class TestUtils { * that uses logcat before that. */ public static void assertLogcat(String tag, int... logs) { - assertThat( + assertWithMessage("Log items (specified per level)").that( ShadowLog.getLogsForTag(tag).stream() .map(logItem -> logItem.type) .collect(toSet())) - .named("Log items (specified per level)") .containsExactly(IntStream.of(logs).boxed().toArray()); } @@ -135,15 +135,13 @@ public class TestUtils { /** Declare shadow {@link ShadowEventLog} to use this. */ public static void assertEventLogged(int tag, Object... values) { - assertThat(ShadowEventLog.getEntries()) - .named("Event logs") + assertWithMessage("Event logs").that(ShadowEventLog.getEntries()) .contains(new ShadowEventLog.Entry(tag, Arrays.asList(values))); } /** Declare shadow {@link ShadowEventLog} to use this. */ public static void assertEventNotLogged(int tag, Object... values) { - assertThat(ShadowEventLog.getEntries()) - .named("Event logs") + assertWithMessage("Event logs").that(ShadowEventLog.getEntries()) .doesNotContain(new ShadowEventLog.Entry(tag, Arrays.asList(values))); } |