From: jeffreyhuang Date: Thu, 30 Nov 2017 18:35:58 +0000 (-0800) Subject: Update PickupGesturePrefCtrlTest to sdk 26 X-Git-Tag: android-x86-9.0-r1~145^2~28^2~2 X-Git-Url: http://git.osdn.net/view?a=commitdiff_plain;h=3d1acfeab7209ced545aba81af4856ff12834605;p=android-x86%2Fpackages-apps-Settings.git Update PickupGesturePrefCtrlTest to sdk 26 Test: make RunSettingsRoboTests -j40 Change-Id: Ia322a8c98760c1074732369d084e6a57a29903af --- diff --git a/src/com/android/settings/gestures/PickupGesturePreferenceController.java b/src/com/android/settings/gestures/PickupGesturePreferenceController.java index 314cbc36a2..02107c123d 100644 --- a/src/com/android/settings/gestures/PickupGesturePreferenceController.java +++ b/src/com/android/settings/gestures/PickupGesturePreferenceController.java @@ -24,6 +24,7 @@ import android.content.Intent; import android.content.SharedPreferences; import android.provider.Settings; import android.support.v7.preference.Preference; +import android.support.annotation.VisibleForTesting; import com.android.internal.hardware.AmbientDisplayConfiguration; import com.android.settings.R; @@ -90,7 +91,7 @@ public class PickupGesturePreferenceController extends GesturePreferenceControll @Override public boolean canHandleClicks() { - return mAmbientConfig.pulseOnPickupCanBeModified(mUserId); + return pulseOnPickupCanBeModified(); } @Override @@ -102,4 +103,9 @@ public class PickupGesturePreferenceController extends GesturePreferenceControll return new InlineSwitchPayload(SECURE_KEY, ResultPayload.SettingsSource.SECURE, ON /* onValue */, intent, isAvailable(), ON /* defaultValue */); } + + @VisibleForTesting + boolean pulseOnPickupCanBeModified() { + return mAmbientConfig.pulseOnPickupCanBeModified(mUserId); + } } diff --git a/tests/robotests/src/com/android/settings/gestures/PickupGesturePreferenceControllerTest.java b/tests/robotests/src/com/android/settings/gestures/PickupGesturePreferenceControllerTest.java index 8757a65906..ffd4928cb6 100644 --- a/tests/robotests/src/com/android/settings/gestures/PickupGesturePreferenceControllerTest.java +++ b/tests/robotests/src/com/android/settings/gestures/PickupGesturePreferenceControllerTest.java @@ -18,6 +18,8 @@ package com.android.settings.gestures; import static com.google.common.truth.Truth.assertThat; import static org.mockito.Matchers.anyInt; +import static org.mockito.Mockito.doReturn; +import static org.mockito.Mockito.spy; import static org.mockito.Mockito.when; import android.content.ContentResolver; @@ -46,7 +48,7 @@ import org.robolectric.RuntimeEnvironment; import org.robolectric.annotation.Config; @RunWith(SettingsRobolectricTestRunner.class) -@Config(manifest = TestConfig.MANIFEST_PATH, sdk = TestConfig.SDK_VERSION, shadows = { +@Config(manifest = TestConfig.MANIFEST_PATH, sdk = TestConfig.SDK_VERSION_O, shadows = { SettingsShadowResources.class }) public class PickupGesturePreferenceControllerTest { @@ -105,14 +107,16 @@ public class PickupGesturePreferenceControllerTest { @Test public void testCanHandleClicks_configIsSet_shouldReturnTrue() { - when(mAmbientDisplayConfiguration.pulseOnPickupCanBeModified(anyInt())).thenReturn(true); + mController = spy(mController); + doReturn(true).when(mController).pulseOnPickupCanBeModified(); assertThat(mController.canHandleClicks()).isTrue(); } @Test public void testCanHandleClicks_configIsNotSet_shouldReturnFalse() { - when(mAmbientDisplayConfiguration.pulseOnPickupCanBeModified(anyInt())).thenReturn(false); + mController = spy(mController); + doReturn(false).when(mController).pulseOnPickupCanBeModified(); assertThat(mController.canHandleClicks()).isFalse(); }