From d70c4a00361b87ad001d9322bc0a49b199c046df Mon Sep 17 00:00:00 2001 From: Andreas Gampe Date: Mon, 26 Feb 2018 11:28:21 -0800 Subject: [PATCH] Settings: Fix assertThat in tests Truth.assertThat is a fluent-style API. Fix the tests to actually do something. Found by errorprone. (cherry picked from commit 22bad556a1863bff80c4ec517ada315f7b01890e) Bug: 73513670 Test: m RunSettingsRoboTests Test: m javac-check RUN_ERROR_PRONE=true Exempt-From-Owner-Approval: Cherrypick Merged-In: Ib923de28ce5dcf79b9dec718306877e31e58e024 Change-Id: Ib923de28ce5dcf79b9dec718306877e31e58e024 --- .../com/android/settings/fuelgauge/PowerUsageSummaryTest.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/robotests/src/com/android/settings/fuelgauge/PowerUsageSummaryTest.java b/tests/robotests/src/com/android/settings/fuelgauge/PowerUsageSummaryTest.java index 48a48b1abce..4cda7fe5bdf 100644 --- a/tests/robotests/src/com/android/settings/fuelgauge/PowerUsageSummaryTest.java +++ b/tests/robotests/src/com/android/settings/fuelgauge/PowerUsageSummaryTest.java @@ -495,8 +495,8 @@ public class PowerUsageSummaryTest { TextView summary1 = mFragment.mBatteryLayoutPref.findViewById(R.id.summary1); TextView summary2 = mFragment.mBatteryLayoutPref.findViewById(R.id.summary2); Robolectric.flushBackgroundThreadScheduler(); - assertThat(summary2.getText().toString().contains(NEW_ML_EST_SUFFIX)); - assertThat(summary1.getText().toString().contains(OLD_EST_SUFFIX)); + assertThat(summary2.getText().toString()).contains(NEW_ML_EST_SUFFIX); + assertThat(summary1.getText().toString()).contains(OLD_EST_SUFFIX); } @Test