From 93c143ed0a0e20f388cae4e754d34fa3754ab6ab Mon Sep 17 00:00:00 2001 From: Sorin Basca Date: Tue, 5 Jul 2022 18:37:05 +0100 Subject: [PATCH] Use Mockito 4.6.1 API Bug: 236636175 Test: m RunSettingsRoboTests Change-Id: I2cfda684059520f6ddd1e72c55f1ab1ec9c99e8b Merged-In: I2cfda684059520f6ddd1e72c55f1ab1ec9c99e8b --- color-check-baseline.xml | 192 ++++++++++++++++++ ...ithAdminGrantedPermissionsCounterTest.java | 28 +-- ...WithAdminGrantedPermissionsListerTest.java | 26 +-- .../applications/InstalledAppListerTest.java | 10 +- ...cialAppAccessPreferenceControllerTest.java | 2 +- ...DefaultAssistPreferenceControllerTest.java | 4 +- .../FingerprintEnrollEnrollingTest.java | 8 +- .../BluetoothDeviceDetailsFragmentTest.java | 2 +- .../DashboardFeatureProviderImplTest.java | 4 +- .../settings/datausage/AppDataUsageTest.java | 10 +- .../datausage/BillingCycleSettingsTest.java | 17 +- .../datausage/lib/DataUsageLibTest.java | 2 +- ...ZoneDetectionPreferenceControllerTest.java | 4 +- ...ModuleVersionPreferenceControllerTest.java | 4 +- ...rmissionsPreferenceControllerTestBase.java | 6 +- ...alledPackagesPreferenceControllerTest.java | 4 +- .../AdvancedPowerUsageDetailTest.java | 8 +- .../fuelgauge/BatteryBackupHelperTest.java | 10 +- ...BatteryHeaderPreferenceControllerTest.java | 6 +- .../fuelgauge/BatteryOptimizeUtilsTest.java | 16 +- ...atteryAppListPreferenceControllerTest.java | 2 +- .../BatteryChartPreferenceControllerTest.java | 4 +- .../batteryusage/BatteryDiffEntryTest.java | 6 +- .../TetherPreferenceControllerTest.java | 7 +- ...utomaticRulesPreferenceControllerTest.java | 17 +- ...ConversationsPreferenceControllerTest.java | 4 +- ...ioritySendersPreferenceControllerTest.java | 4 +- ...arredContactsPreferenceControllerTest.java | 2 +- ...intJobMessagePreferenceControllerTest.java | 10 +- .../PrintJobPreferenceControllerTest.java | 6 +- .../actionbar/SearchMenuControllerTest.java | 8 +- ...ghlightablePreferenceGroupAdapterTest.java | 8 +- .../NetworkRequestDialogFragmentTest.java | 2 +- .../DisclaimerItemListAdapterTest.java | 8 +- ...EmergencyCallLimitationDisclaimerTest.java | 4 +- .../calling/LocationPolicyDisclaimerTest.java | 4 +- .../WifiCallingDisclaimerFragmentTest.java | 9 +- .../WifiCallingSettingsForSubTest.java | 11 +- .../WifiDetailPreferenceController2Test.java | 12 +- 39 files changed, 345 insertions(+), 146 deletions(-) diff --git a/color-check-baseline.xml b/color-check-baseline.xml index b4915b937bd..6f0ff094e4b 100644 --- a/color-check-baseline.xml +++ b/color-check-baseline.xml @@ -525,6 +525,86 @@ column="5"/> + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + diff --git a/tests/robotests/src/com/android/settings/applications/AppWithAdminGrantedPermissionsCounterTest.java b/tests/robotests/src/com/android/settings/applications/AppWithAdminGrantedPermissionsCounterTest.java index 4e08f9841df..71c532c7e2f 100644 --- a/tests/robotests/src/com/android/settings/applications/AppWithAdminGrantedPermissionsCounterTest.java +++ b/tests/robotests/src/com/android/settings/applications/AppWithAdminGrantedPermissionsCounterTest.java @@ -20,10 +20,10 @@ import static com.android.settings.testutils.ApplicationTestUtils.buildInfo; import static com.google.common.truth.Truth.assertThat; -import static org.mockito.ArgumentMatchers.anyInt; -import static org.mockito.ArgumentMatchers.anyObject; -import static org.mockito.ArgumentMatchers.eq; +import static org.mockito.Mockito.any; +import static org.mockito.Mockito.anyInt; import static org.mockito.Mockito.atLeast; +import static org.mockito.Mockito.eq; import static org.mockito.Mockito.verify; import static org.mockito.Mockito.verifyNoMoreInteractions; import static org.mockito.Mockito.when; @@ -39,7 +39,6 @@ import android.os.RemoteException; import android.os.UserHandle; import android.os.UserManager; -import java.util.Collections; import org.junit.Before; import org.junit.Test; import org.junit.runner.RunWith; @@ -49,6 +48,7 @@ import org.robolectric.RobolectricTestRunner; import org.robolectric.shadows.ShadowApplication; import java.util.Arrays; +import java.util.Collections; @RunWith(RobolectricTestRunner.class) public final class AppWithAdminGrantedPermissionsCounterTest { @@ -129,7 +129,7 @@ public final class AppWithAdminGrantedPermissionsCounterTest { verify(mPackageManager).getInstalledApplicationsAsUser(anyInt(), eq(MAIN_USER_ID)); verify(mPackageManager).getInstalledApplicationsAsUser(anyInt(), eq(MANAGED_PROFILE_ID)); - verify(mPackageManager, atLeast(0)).getInstallReason(anyObject(), anyObject()); + verify(mPackageManager, atLeast(0)).getInstallReason(any(), any()); verifyNoMoreInteractions(mPackageManager); } @@ -166,9 +166,9 @@ public final class AppWithAdminGrantedPermissionsCounterTest { } private void configureInstallTimePermissions() throws RemoteException { - when(mPackageManagerService.checkUidPermission(anyObject(), eq(APP_1_UID))) + when(mPackageManagerService.checkUidPermission(any(), eq(APP_1_UID))) .thenReturn(PackageManager.PERMISSION_DENIED); - when(mPackageManagerService.checkUidPermission(anyObject(), eq(APP_2_UID))) + when(mPackageManagerService.checkUidPermission(any(), eq(APP_2_UID))) .thenReturn(PackageManager.PERMISSION_DENIED); when(mPackageManagerService.checkUidPermission(PERMISSION_1, APP_3_UID)) .thenReturn(PackageManager.PERMISSION_DENIED); @@ -178,9 +178,9 @@ public final class AppWithAdminGrantedPermissionsCounterTest { .thenReturn(PackageManager.PERMISSION_DENIED); when(mPackageManagerService.checkUidPermission(PERMISSION_2, APP_4_UID)) .thenReturn(PackageManager.PERMISSION_GRANTED); - when(mPackageManagerService.checkUidPermission(anyObject(), eq(APP_5_UID))) + when(mPackageManagerService.checkUidPermission(any(), eq(APP_5_UID))) .thenReturn(PackageManager.PERMISSION_DENIED); - when(mPackageManagerService.checkUidPermission(anyObject(), eq(APP_6_UID))) + when(mPackageManagerService.checkUidPermission(any(), eq(APP_6_UID))) .thenReturn(PackageManager.PERMISSION_DENIED); } @@ -189,15 +189,15 @@ public final class AppWithAdminGrantedPermissionsCounterTest { .thenReturn(DevicePolicyManager.PERMISSION_GRANT_STATE_GRANTED); when(mDevicePolicyManager.getPermissionGrantState(null, APP_1, PERMISSION_2)) .thenReturn(DevicePolicyManager.PERMISSION_GRANT_STATE_DENIED); - when(mDevicePolicyManager.getPermissionGrantState(eq(null), eq(APP_2), anyObject())) + when(mDevicePolicyManager.getPermissionGrantState(eq(null), eq(APP_2), any())) .thenReturn(DevicePolicyManager.PERMISSION_GRANT_STATE_DENIED); - when(mDevicePolicyManager.getPermissionGrantState(eq(null), eq(APP_3), anyObject())) + when(mDevicePolicyManager.getPermissionGrantState(eq(null), eq(APP_3), any())) .thenReturn(DevicePolicyManager.PERMISSION_GRANT_STATE_DENIED); - when(mDevicePolicyManager.getPermissionGrantState(eq(null), eq(APP_4), anyObject())) + when(mDevicePolicyManager.getPermissionGrantState(eq(null), eq(APP_4), any())) .thenReturn(DevicePolicyManager.PERMISSION_GRANT_STATE_DENIED); - when(mDevicePolicyManager.getPermissionGrantState(eq(null), eq(APP_5), anyObject())) + when(mDevicePolicyManager.getPermissionGrantState(eq(null), eq(APP_5), any())) .thenReturn(DevicePolicyManager.PERMISSION_GRANT_STATE_DENIED); - when(mDevicePolicyManager.getPermissionGrantState(eq(null), eq(APP_6), anyObject())) + when(mDevicePolicyManager.getPermissionGrantState(eq(null), eq(APP_6), any())) .thenReturn(DevicePolicyManager.PERMISSION_GRANT_STATE_GRANTED); } diff --git a/tests/robotests/src/com/android/settings/applications/AppWithAdminGrantedPermissionsListerTest.java b/tests/robotests/src/com/android/settings/applications/AppWithAdminGrantedPermissionsListerTest.java index a8186373d38..9a4978b9d6e 100644 --- a/tests/robotests/src/com/android/settings/applications/AppWithAdminGrantedPermissionsListerTest.java +++ b/tests/robotests/src/com/android/settings/applications/AppWithAdminGrantedPermissionsListerTest.java @@ -20,10 +20,10 @@ import static com.android.settings.testutils.ApplicationTestUtils.buildInfo; import static com.google.common.truth.Truth.assertThat; -import static org.mockito.ArgumentMatchers.anyInt; -import static org.mockito.ArgumentMatchers.anyObject; -import static org.mockito.ArgumentMatchers.eq; +import static org.mockito.Mockito.any; +import static org.mockito.Mockito.anyInt; import static org.mockito.Mockito.atLeast; +import static org.mockito.Mockito.eq; import static org.mockito.Mockito.verify; import static org.mockito.Mockito.verifyNoMoreInteractions; import static org.mockito.Mockito.when; @@ -122,19 +122,19 @@ public final class AppWithAdminGrantedPermissionsListerTest { .thenReturn(DevicePolicyManager.PERMISSION_GRANT_STATE_GRANTED); when(mDevicePolicyManager.getPermissionGrantState(null, APP_1, PERMISSION_2)) .thenReturn(DevicePolicyManager.PERMISSION_GRANT_STATE_DENIED); - when(mDevicePolicyManager.getPermissionGrantState(eq(null), eq(APP_2), anyObject())) + when(mDevicePolicyManager.getPermissionGrantState(eq(null), eq(APP_2), any())) .thenReturn(DevicePolicyManager.PERMISSION_GRANT_STATE_DENIED); - when(mDevicePolicyManager.getPermissionGrantState(eq(null), eq(APP_3), anyObject())) + when(mDevicePolicyManager.getPermissionGrantState(eq(null), eq(APP_3), any())) .thenReturn(DevicePolicyManager.PERMISSION_GRANT_STATE_DENIED); - when(mDevicePolicyManager.getPermissionGrantState(eq(null), eq(APP_4), anyObject())) + when(mDevicePolicyManager.getPermissionGrantState(eq(null), eq(APP_4), any())) .thenReturn(DevicePolicyManager.PERMISSION_GRANT_STATE_DENIED); - when(mDevicePolicyManager.getPermissionGrantState(eq(null), eq(APP_5), anyObject())) + when(mDevicePolicyManager.getPermissionGrantState(eq(null), eq(APP_5), any())) .thenReturn(DevicePolicyManager.PERMISSION_GRANT_STATE_DENIED); // Grant install-time permissions as appropriate. - when(mPackageManagerService.checkUidPermission(anyObject(), eq(APP_1_UID))) + when(mPackageManagerService.checkUidPermission(any(), eq(APP_1_UID))) .thenReturn(PackageManager.PERMISSION_DENIED); - when(mPackageManagerService.checkUidPermission(anyObject(), eq(APP_2_UID))) + when(mPackageManagerService.checkUidPermission(any(), eq(APP_2_UID))) .thenReturn(PackageManager.PERMISSION_DENIED); when(mPackageManagerService.checkUidPermission(PERMISSION_1, APP_3_UID)) .thenReturn(PackageManager.PERMISSION_DENIED); @@ -144,7 +144,7 @@ public final class AppWithAdminGrantedPermissionsListerTest { .thenReturn(PackageManager.PERMISSION_DENIED); when(mPackageManagerService.checkUidPermission(PERMISSION_2, APP_4_UID)) .thenReturn(PackageManager.PERMISSION_GRANTED); - when(mPackageManagerService.checkUidPermission(anyObject(), eq(APP_5_UID))) + when(mPackageManagerService.checkUidPermission(any(), eq(APP_5_UID))) .thenReturn(PackageManager.PERMISSION_DENIED); // app3 and app5 were installed by enterprise policy. @@ -168,11 +168,11 @@ public final class AppWithAdminGrantedPermissionsListerTest { buildInfo(APP_6_UID, APP_6, 0 /* flags */, Build.VERSION_CODES.M))); // Grant run-time permissions as appropriate. - when(mDevicePolicyManager.getPermissionGrantState(eq(null), eq(APP_6), anyObject())) + when(mDevicePolicyManager.getPermissionGrantState(eq(null), eq(APP_6), any())) .thenReturn(DevicePolicyManager.PERMISSION_GRANT_STATE_GRANTED); // Grant install-time permissions as appropriate. - when(mPackageManagerService.checkUidPermission(anyObject(), eq(APP_6_UID))) + when(mPackageManagerService.checkUidPermission(any(), eq(APP_6_UID))) .thenReturn(PackageManager.PERMISSION_DENIED); // app6 was not installed by enterprise policy. @@ -202,7 +202,7 @@ public final class AppWithAdminGrantedPermissionsListerTest { verify(mPackageManager).getInstalledApplicationsAsUser(anyInt(), eq(MAIN_USER_ID)); verify(mPackageManager).getInstalledApplicationsAsUser(anyInt(), eq(MANAGED_PROFILE_ID)); - verify(mPackageManager, atLeast(0)).getInstallReason(anyObject(), anyObject()); + verify(mPackageManager, atLeast(0)).getInstallReason(any(), any()); verifyNoMoreInteractions(mPackageManager); } diff --git a/tests/robotests/src/com/android/settings/applications/InstalledAppListerTest.java b/tests/robotests/src/com/android/settings/applications/InstalledAppListerTest.java index e2aed368f80..602d678c9d8 100644 --- a/tests/robotests/src/com/android/settings/applications/InstalledAppListerTest.java +++ b/tests/robotests/src/com/android/settings/applications/InstalledAppListerTest.java @@ -20,11 +20,11 @@ import static com.android.settings.testutils.ApplicationTestUtils.buildInfo; import static com.google.common.truth.Truth.assertThat; -import static org.mockito.ArgumentMatchers.anyInt; -import static org.mockito.ArgumentMatchers.anyObject; -import static org.mockito.ArgumentMatchers.argThat; -import static org.mockito.ArgumentMatchers.eq; +import static org.mockito.Mockito.any; +import static org.mockito.Mockito.anyInt; +import static org.mockito.Mockito.argThat; import static org.mockito.Mockito.atLeast; +import static org.mockito.Mockito.eq; import static org.mockito.Mockito.verify; import static org.mockito.Mockito.when; @@ -170,7 +170,7 @@ public final class InstalledAppListerTest { verify(mPackageManager).getInstalledApplicationsAsUser(anyInt(), eq(MAIN_USER_ID)); verify(mPackageManager).getInstalledApplicationsAsUser(anyInt(), eq(MANAGED_PROFILE_ID)); verify(mPackageManager, atLeast(0)) - .queryIntentActivitiesAsUser(anyObject(), anyInt(), anyInt()); + .queryIntentActivitiesAsUser(any(), anyInt(), anyInt()); } public static boolean checkAppFound(List mInstalledAppList, String appId, diff --git a/tests/robotests/src/com/android/settings/applications/SpecialAppAccessPreferenceControllerTest.java b/tests/robotests/src/com/android/settings/applications/SpecialAppAccessPreferenceControllerTest.java index e0a08364a8c..da5ada783ad 100644 --- a/tests/robotests/src/com/android/settings/applications/SpecialAppAccessPreferenceControllerTest.java +++ b/tests/robotests/src/com/android/settings/applications/SpecialAppAccessPreferenceControllerTest.java @@ -20,7 +20,7 @@ import static com.android.settings.core.BasePreferenceController.AVAILABLE; import static com.google.common.truth.Truth.assertThat; -import static org.mockito.Matchers.anyInt; +import static org.mockito.Mockito.anyInt; import static org.mockito.Mockito.doReturn; import static org.mockito.Mockito.mock; import static org.mockito.Mockito.spy; diff --git a/tests/robotests/src/com/android/settings/applications/assist/DefaultAssistPreferenceControllerTest.java b/tests/robotests/src/com/android/settings/applications/assist/DefaultAssistPreferenceControllerTest.java index 3e46027c362..178a26f8453 100644 --- a/tests/robotests/src/com/android/settings/applications/assist/DefaultAssistPreferenceControllerTest.java +++ b/tests/robotests/src/com/android/settings/applications/assist/DefaultAssistPreferenceControllerTest.java @@ -24,7 +24,7 @@ import static org.mockito.ArgumentMatchers.anyInt; import static org.mockito.Mockito.doReturn; import static org.mockito.Mockito.mock; import static org.mockito.Mockito.spy; -import static org.mockito.Mockito.verifyZeroInteractions; +import static org.mockito.Mockito.verifyNoInteractions; import static org.mockito.Mockito.when; import android.Manifest; @@ -142,6 +142,6 @@ public class DefaultAssistPreferenceControllerTest { controller.getSettingIntent(null); - verifyZeroInteractions(assistUtils); + verifyNoInteractions(assistUtils); } } diff --git a/tests/robotests/src/com/android/settings/biometrics/fingerprint/FingerprintEnrollEnrollingTest.java b/tests/robotests/src/com/android/settings/biometrics/fingerprint/FingerprintEnrollEnrollingTest.java index b8992068a8e..ee6ae223169 100644 --- a/tests/robotests/src/com/android/settings/biometrics/fingerprint/FingerprintEnrollEnrollingTest.java +++ b/tests/robotests/src/com/android/settings/biometrics/fingerprint/FingerprintEnrollEnrollingTest.java @@ -25,11 +25,11 @@ import static com.android.settings.biometrics.fingerprint.FingerprintEnrollEnrol import static com.google.common.truth.Truth.assertThat; -import static org.mockito.ArgumentMatchers.any; -import static org.mockito.ArgumentMatchers.anyInt; -import static org.mockito.ArgumentMatchers.anyString; -import static org.mockito.Matchers.eq; +import static org.mockito.Mockito.any; +import static org.mockito.Mockito.anyInt; +import static org.mockito.Mockito.anyString; import static org.mockito.Mockito.doReturn; +import static org.mockito.Mockito.eq; import static org.mockito.Mockito.never; import static org.mockito.Mockito.spy; import static org.mockito.Mockito.times; diff --git a/tests/robotests/src/com/android/settings/bluetooth/BluetoothDeviceDetailsFragmentTest.java b/tests/robotests/src/com/android/settings/bluetooth/BluetoothDeviceDetailsFragmentTest.java index fac8b584f0a..81f485986b9 100644 --- a/tests/robotests/src/com/android/settings/bluetooth/BluetoothDeviceDetailsFragmentTest.java +++ b/tests/robotests/src/com/android/settings/bluetooth/BluetoothDeviceDetailsFragmentTest.java @@ -21,8 +21,8 @@ import static android.bluetooth.BluetoothDevice.BOND_NONE; import static com.google.common.truth.Truth.assertThat; import static org.mockito.ArgumentMatchers.any; -import static org.mockito.Matchers.eq; import static org.mockito.Mockito.doReturn; +import static org.mockito.Mockito.eq; import static org.mockito.Mockito.mock; import static org.mockito.Mockito.spy; import static org.mockito.Mockito.verify; diff --git a/tests/robotests/src/com/android/settings/dashboard/DashboardFeatureProviderImplTest.java b/tests/robotests/src/com/android/settings/dashboard/DashboardFeatureProviderImplTest.java index f2b0acddbed..37b8ab630fb 100644 --- a/tests/robotests/src/com/android/settings/dashboard/DashboardFeatureProviderImplTest.java +++ b/tests/robotests/src/com/android/settings/dashboard/DashboardFeatureProviderImplTest.java @@ -40,7 +40,7 @@ import static org.mockito.Mockito.mock; import static org.mockito.Mockito.never; import static org.mockito.Mockito.spy; import static org.mockito.Mockito.verify; -import static org.mockito.Mockito.verifyZeroInteractions; +import static org.mockito.Mockito.verifyNoInteractions; import static org.mockito.Mockito.when; import android.content.Context; @@ -268,7 +268,7 @@ public class DashboardFeatureProviderImplTest { mImpl.bindPreferenceToTileAndGetObservers(mActivity, mFragment, mForceRoundedIcon, null /* keys */, tile, "123", Preference.DEFAULT_ORDER); - verifyZeroInteractions(tile); + verifyNoInteractions(tile); } @Test diff --git a/tests/robotests/src/com/android/settings/datausage/AppDataUsageTest.java b/tests/robotests/src/com/android/settings/datausage/AppDataUsageTest.java index bccec3ed8b0..e7b59a35fe5 100644 --- a/tests/robotests/src/com/android/settings/datausage/AppDataUsageTest.java +++ b/tests/robotests/src/com/android/settings/datausage/AppDataUsageTest.java @@ -19,14 +19,14 @@ package com.android.settings.datausage; import static com.google.common.truth.Truth.assertThat; import static org.junit.Assert.assertTrue; -import static org.mockito.ArgumentMatchers.any; -import static org.mockito.ArgumentMatchers.anyBoolean; -import static org.mockito.ArgumentMatchers.anyInt; -import static org.mockito.ArgumentMatchers.anyString; -import static org.mockito.Matchers.eq; import static org.mockito.Mockito.RETURNS_DEEP_STUBS; +import static org.mockito.Mockito.any; +import static org.mockito.Mockito.anyBoolean; +import static org.mockito.Mockito.anyInt; +import static org.mockito.Mockito.anyString; import static org.mockito.Mockito.doNothing; import static org.mockito.Mockito.doReturn; +import static org.mockito.Mockito.eq; import static org.mockito.Mockito.mock; import static org.mockito.Mockito.spy; import static org.mockito.Mockito.verify; diff --git a/tests/robotests/src/com/android/settings/datausage/BillingCycleSettingsTest.java b/tests/robotests/src/com/android/settings/datausage/BillingCycleSettingsTest.java index 861b4e395fa..cef5928c717 100644 --- a/tests/robotests/src/com/android/settings/datausage/BillingCycleSettingsTest.java +++ b/tests/robotests/src/com/android/settings/datausage/BillingCycleSettingsTest.java @@ -19,16 +19,15 @@ import static android.net.NetworkPolicy.CYCLE_NONE; import static com.google.common.truth.Truth.assertThat; -import static org.mockito.ArgumentMatchers.any; -import static org.mockito.ArgumentMatchers.anyInt; -import static org.mockito.ArgumentMatchers.anyLong; -import static org.mockito.ArgumentMatchers.anyObject; -import static org.mockito.ArgumentMatchers.anyString; -import static org.mockito.ArgumentMatchers.nullable; +import static org.mockito.Mockito.any; +import static org.mockito.Mockito.anyInt; +import static org.mockito.Mockito.anyLong; +import static org.mockito.Mockito.anyString; import static org.mockito.Mockito.doNothing; import static org.mockito.Mockito.doReturn; import static org.mockito.Mockito.mock; import static org.mockito.Mockito.never; +import static org.mockito.Mockito.nullable; import static org.mockito.Mockito.spy; import static org.mockito.Mockito.verify; import static org.mockito.Mockito.when; @@ -133,9 +132,9 @@ public class BillingCycleSettingsTest { mDataLimit, mDataWarning, mEnableDataLimit, mEnableDataWarning); doReturn("some-string").when(billingCycleSettings).getString(anyInt(), anyInt()); - when(mNetworkPolicyEditor.getPolicyCycleDay(anyObject())).thenReturn(CYCLE_NONE + 1); - when(mNetworkPolicyEditor.getPolicyLimitBytes(anyObject())).thenReturn(2000L); - when(mNetworkPolicyEditor.getPolicyWarningBytes(anyObject())).thenReturn(1000L); + when(mNetworkPolicyEditor.getPolicyCycleDay(any())).thenReturn(CYCLE_NONE + 1); + when(mNetworkPolicyEditor.getPolicyLimitBytes(any())).thenReturn(2000L); + when(mNetworkPolicyEditor.getPolicyWarningBytes(any())).thenReturn(1000L); billingCycleSettings.updatePrefs(); diff --git a/tests/robotests/src/com/android/settings/datausage/lib/DataUsageLibTest.java b/tests/robotests/src/com/android/settings/datausage/lib/DataUsageLibTest.java index 70a1a38fa0e..4391425d2ba 100644 --- a/tests/robotests/src/com/android/settings/datausage/lib/DataUsageLibTest.java +++ b/tests/robotests/src/com/android/settings/datausage/lib/DataUsageLibTest.java @@ -18,7 +18,7 @@ package com.android.settings.datausage.lib; import static com.google.common.truth.Truth.assertThat; -import static org.mockito.Matchers.anyInt; +import static org.mockito.Mockito.anyInt; import static org.mockito.Mockito.spy; import static org.mockito.Mockito.when; diff --git a/tests/robotests/src/com/android/settings/datetime/LocationTimeZoneDetectionPreferenceControllerTest.java b/tests/robotests/src/com/android/settings/datetime/LocationTimeZoneDetectionPreferenceControllerTest.java index b9c4edc9af2..9fb364ed62e 100644 --- a/tests/robotests/src/com/android/settings/datetime/LocationTimeZoneDetectionPreferenceControllerTest.java +++ b/tests/robotests/src/com/android/settings/datetime/LocationTimeZoneDetectionPreferenceControllerTest.java @@ -24,7 +24,7 @@ import static com.google.common.truth.Truth.assertThat; import static org.mockito.Mockito.spy; import static org.mockito.Mockito.verify; -import static org.mockito.Mockito.verifyZeroInteractions; +import static org.mockito.Mockito.verifyNoInteractions; import static org.mockito.Mockito.when; import android.app.time.Capabilities; @@ -99,7 +99,7 @@ public class LocationTimeZoneDetectionPreferenceControllerTest { mController.setChecked(true); // Verify the TimeManager was not called. - verifyZeroInteractions(mTimeManager); + verifyNoInteractions(mTimeManager); } @Test diff --git a/tests/robotests/src/com/android/settings/deviceinfo/firmwareversion/MainlineModuleVersionPreferenceControllerTest.java b/tests/robotests/src/com/android/settings/deviceinfo/firmwareversion/MainlineModuleVersionPreferenceControllerTest.java index 3f49b15b634..35e044cc2f3 100644 --- a/tests/robotests/src/com/android/settings/deviceinfo/firmwareversion/MainlineModuleVersionPreferenceControllerTest.java +++ b/tests/robotests/src/com/android/settings/deviceinfo/firmwareversion/MainlineModuleVersionPreferenceControllerTest.java @@ -23,8 +23,8 @@ import static com.android.settings.deviceinfo.firmwareversion.MainlineModuleVers import static com.google.common.truth.Truth.assertThat; -import static org.mockito.Matchers.anyInt; -import static org.mockito.Matchers.eq; +import static org.mockito.Mockito.anyInt; +import static org.mockito.Mockito.eq; import static org.mockito.Mockito.spy; import static org.mockito.Mockito.when; diff --git a/tests/robotests/src/com/android/settings/enterprise/AdminGrantedPermissionsPreferenceControllerTestBase.java b/tests/robotests/src/com/android/settings/enterprise/AdminGrantedPermissionsPreferenceControllerTestBase.java index 09d1c84f994..06e41073716 100644 --- a/tests/robotests/src/com/android/settings/enterprise/AdminGrantedPermissionsPreferenceControllerTestBase.java +++ b/tests/robotests/src/com/android/settings/enterprise/AdminGrantedPermissionsPreferenceControllerTestBase.java @@ -18,10 +18,10 @@ package com.android.settings.enterprise; import static com.google.common.truth.Truth.assertThat; -import static org.mockito.ArgumentMatchers.anyObject; -import static org.mockito.ArgumentMatchers.eq; +import static org.mockito.Mockito.any; import static org.mockito.Mockito.doAnswer; import static org.mockito.Mockito.doReturn; +import static org.mockito.Mockito.eq; import static org.mockito.Mockito.mock; import static org.mockito.Mockito.when; @@ -79,7 +79,7 @@ public abstract class AdminGrantedPermissionsPreferenceControllerTestBase { } }).when(mFeatureFactory.applicationFeatureProvider) .calculateNumberOfAppsWithAdminGrantedPermissions(eq(mPermissions), - eq(async), anyObject()); + eq(async), any()); } @Test diff --git a/tests/robotests/src/com/android/settings/enterprise/EnterpriseInstalledPackagesPreferenceControllerTest.java b/tests/robotests/src/com/android/settings/enterprise/EnterpriseInstalledPackagesPreferenceControllerTest.java index f39b4b8821b..c32d2d79256 100644 --- a/tests/robotests/src/com/android/settings/enterprise/EnterpriseInstalledPackagesPreferenceControllerTest.java +++ b/tests/robotests/src/com/android/settings/enterprise/EnterpriseInstalledPackagesPreferenceControllerTest.java @@ -18,7 +18,7 @@ package com.android.settings.enterprise; import static com.google.common.truth.Truth.assertThat; -import static org.mockito.Mockito.anyObject; +import static org.mockito.Mockito.any; import static org.mockito.Mockito.doAnswer; import static org.mockito.Mockito.doReturn; import static org.mockito.Mockito.eq; @@ -74,7 +74,7 @@ public class EnterpriseInstalledPackagesPreferenceControllerTest { return null; } }).when(mFeatureFactory.applicationFeatureProvider) - .calculateNumberOfPolicyInstalledApps(eq(async), anyObject()); + .calculateNumberOfPolicyInstalledApps(eq(async), any()); } @Test diff --git a/tests/robotests/src/com/android/settings/fuelgauge/AdvancedPowerUsageDetailTest.java b/tests/robotests/src/com/android/settings/fuelgauge/AdvancedPowerUsageDetailTest.java index 90e8caf85a2..8c650868836 100644 --- a/tests/robotests/src/com/android/settings/fuelgauge/AdvancedPowerUsageDetailTest.java +++ b/tests/robotests/src/com/android/settings/fuelgauge/AdvancedPowerUsageDetailTest.java @@ -30,7 +30,7 @@ import static org.mockito.Mockito.doReturn; import static org.mockito.Mockito.mock; import static org.mockito.Mockito.spy; import static org.mockito.Mockito.verify; -import static org.mockito.Mockito.verifyZeroInteractions; +import static org.mockito.Mockito.verifyNoInteractions; import static org.mockito.Mockito.when; import android.app.AppOpsManager; @@ -859,7 +859,7 @@ public class AdvancedPowerUsageDetailTest { mFragment.onRadioButtonClicked(mOptimizePreference); mFragment.onPause(); - verifyZeroInteractions(mMetricsFeatureProvider); + verifyNoInteractions(mMetricsFeatureProvider); } @Test @@ -870,7 +870,7 @@ public class AdvancedPowerUsageDetailTest { mFragment.notifyBackupManager(); - verifyZeroInteractions(mBackupManager); + verifyNoInteractions(mBackupManager); } @Test @@ -893,6 +893,6 @@ public class AdvancedPowerUsageDetailTest { mFragment.onPause(); - verifyZeroInteractions(mBackupManager); + verifyNoInteractions(mBackupManager); } } diff --git a/tests/robotests/src/com/android/settings/fuelgauge/BatteryBackupHelperTest.java b/tests/robotests/src/com/android/settings/fuelgauge/BatteryBackupHelperTest.java index 283df7bcc43..55ff93b964a 100644 --- a/tests/robotests/src/com/android/settings/fuelgauge/BatteryBackupHelperTest.java +++ b/tests/robotests/src/com/android/settings/fuelgauge/BatteryBackupHelperTest.java @@ -34,7 +34,7 @@ import static org.mockito.Mockito.inOrder; import static org.mockito.Mockito.never; import static org.mockito.Mockito.spy; import static org.mockito.Mockito.verify; -import static org.mockito.Mockito.verifyZeroInteractions; +import static org.mockito.Mockito.verifyNoInteractions; import android.app.AppOpsManager; import android.app.backup.BackupDataInputStream; @@ -254,7 +254,7 @@ public final class BatteryBackupHelperTest { mBatteryBackupHelper.restoreEntity(mBackupDataInputStream); - verifyZeroInteractions(mBackupDataInputStream); + verifyNoInteractions(mBackupDataInputStream); } @Test @@ -292,13 +292,13 @@ public final class BatteryBackupHelperTest { @Test public void restoreOptimizationMode_nullBytesData_skipRestore() throws Exception { mBatteryBackupHelper.restoreOptimizationMode(new byte[0]); - verifyZeroInteractions(mBatteryOptimizeUtils); + verifyNoInteractions(mBatteryOptimizeUtils); mBatteryBackupHelper.restoreOptimizationMode("invalid data format".getBytes()); - verifyZeroInteractions(mBatteryOptimizeUtils); + verifyNoInteractions(mBatteryOptimizeUtils); mBatteryBackupHelper.restoreOptimizationMode(DELIMITER.getBytes()); - verifyZeroInteractions(mBatteryOptimizeUtils); + verifyNoInteractions(mBatteryOptimizeUtils); } @Test diff --git a/tests/robotests/src/com/android/settings/fuelgauge/BatteryHeaderPreferenceControllerTest.java b/tests/robotests/src/com/android/settings/fuelgauge/BatteryHeaderPreferenceControllerTest.java index 6389dc95fe1..78534e138f0 100644 --- a/tests/robotests/src/com/android/settings/fuelgauge/BatteryHeaderPreferenceControllerTest.java +++ b/tests/robotests/src/com/android/settings/fuelgauge/BatteryHeaderPreferenceControllerTest.java @@ -23,7 +23,7 @@ import static org.mockito.ArgumentMatchers.any; import static org.mockito.Mockito.doReturn; import static org.mockito.Mockito.spy; import static org.mockito.Mockito.verify; -import static org.mockito.Mockito.verifyZeroInteractions; +import static org.mockito.Mockito.verifyNoInteractions; import static org.mockito.Mockito.when; import android.app.Activity; @@ -267,7 +267,7 @@ public class BatteryHeaderPreferenceControllerTest { mController.updateHeaderByBatteryTips(null, mBatteryInfo); - verifyZeroInteractions(mBatteryUsageProgressBarPref); + verifyNoInteractions(mBatteryUsageProgressBarPref); } @Test @@ -277,7 +277,7 @@ public class BatteryHeaderPreferenceControllerTest { mController.updateHeaderByBatteryTips(lowBatteryTip, null); - verifyZeroInteractions(mBatteryUsageProgressBarPref); + verifyNoInteractions(mBatteryUsageProgressBarPref); } @Test diff --git a/tests/robotests/src/com/android/settings/fuelgauge/BatteryOptimizeUtilsTest.java b/tests/robotests/src/com/android/settings/fuelgauge/BatteryOptimizeUtilsTest.java index b223d0bc755..f92c3223b90 100644 --- a/tests/robotests/src/com/android/settings/fuelgauge/BatteryOptimizeUtilsTest.java +++ b/tests/robotests/src/com/android/settings/fuelgauge/BatteryOptimizeUtilsTest.java @@ -25,16 +25,16 @@ import static com.google.common.truth.Truth.assertThat; import static org.junit.Assert.assertFalse; import static org.junit.Assert.assertNull; import static org.junit.Assert.assertTrue; -import static org.mockito.ArgumentMatchers.anyInt; -import static org.mockito.ArgumentMatchers.anyLong; -import static org.mockito.ArgumentMatchers.anyString; +import static org.mockito.Mockito.anyInt; +import static org.mockito.Mockito.anyLong; +import static org.mockito.Mockito.anyString; import static org.mockito.Mockito.doReturn; import static org.mockito.Mockito.doThrow; import static org.mockito.Mockito.inOrder; import static org.mockito.Mockito.spy; import static org.mockito.Mockito.verify; +import static org.mockito.Mockito.verifyNoInteractions; import static org.mockito.Mockito.verifyNoMoreInteractions; -import static org.mockito.Mockito.verifyZeroInteractions; import static org.mockito.Mockito.when; import android.app.AppOpsManager; @@ -183,8 +183,8 @@ public class BatteryOptimizeUtilsTest { mBatteryOptimizeUtils.setAppUsageState(MODE_UNRESTRICTED); TimeUnit.SECONDS.sleep(1); - verifyZeroInteractions(mMockBackend); - verifyZeroInteractions(mMockBatteryUtils); + verifyNoInteractions(mMockBackend); + verifyNoInteractions(mMockBatteryUtils); } @Test @@ -247,7 +247,7 @@ public class BatteryOptimizeUtilsTest { AppOpsManager.MODE_ALLOWED, /* allowListed */ false, /* isSystemOrDefaultApp */ false); - verifyZeroInteractions(mMockBatteryUtils); + verifyNoInteractions(mMockBatteryUtils); final InOrder inOrder = inOrder(mMockBackend); inOrder.verify(mMockBackend).refreshList(); @@ -261,7 +261,7 @@ public class BatteryOptimizeUtilsTest { AppOpsManager.MODE_ALLOWED, /* allowListed */ true, /* isSystemOrDefaultApp */ true); - verifyZeroInteractions(mMockBatteryUtils); + verifyNoInteractions(mMockBatteryUtils); final InOrder inOrder = inOrder(mMockBackend); inOrder.verify(mMockBackend).refreshList(); diff --git a/tests/robotests/src/com/android/settings/fuelgauge/batteryusage/BatteryAppListPreferenceControllerTest.java b/tests/robotests/src/com/android/settings/fuelgauge/batteryusage/BatteryAppListPreferenceControllerTest.java index da397fd791e..5815bae6b6e 100644 --- a/tests/robotests/src/com/android/settings/fuelgauge/batteryusage/BatteryAppListPreferenceControllerTest.java +++ b/tests/robotests/src/com/android/settings/fuelgauge/batteryusage/BatteryAppListPreferenceControllerTest.java @@ -18,7 +18,7 @@ package com.android.settings.fuelgauge.batteryusage; import static com.google.common.truth.Truth.assertThat; -import static org.mockito.Matchers.anyInt; +import static org.mockito.Mockito.anyInt; import static org.mockito.Mockito.doReturn; import static org.mockito.Mockito.spy; import static org.mockito.Mockito.when; diff --git a/tests/robotests/src/com/android/settings/fuelgauge/batteryusage/BatteryChartPreferenceControllerTest.java b/tests/robotests/src/com/android/settings/fuelgauge/batteryusage/BatteryChartPreferenceControllerTest.java index 9fbcb1688db..151b8ae3166 100644 --- a/tests/robotests/src/com/android/settings/fuelgauge/batteryusage/BatteryChartPreferenceControllerTest.java +++ b/tests/robotests/src/com/android/settings/fuelgauge/batteryusage/BatteryChartPreferenceControllerTest.java @@ -18,9 +18,9 @@ package com.android.settings.fuelgauge.batteryusage; import static com.google.common.truth.Truth.assertThat; -import static org.mockito.ArgumentMatchers.anyFloat; -import static org.mockito.ArgumentMatchers.anyLong; import static org.mockito.Mockito.any; +import static org.mockito.Mockito.anyFloat; +import static org.mockito.Mockito.anyLong; import static org.mockito.Mockito.atLeast; import static org.mockito.Mockito.atLeastOnce; import static org.mockito.Mockito.doReturn; diff --git a/tests/robotests/src/com/android/settings/fuelgauge/batteryusage/BatteryDiffEntryTest.java b/tests/robotests/src/com/android/settings/fuelgauge/batteryusage/BatteryDiffEntryTest.java index 683f0fd2df4..07a5504f61d 100644 --- a/tests/robotests/src/com/android/settings/fuelgauge/batteryusage/BatteryDiffEntryTest.java +++ b/tests/robotests/src/com/android/settings/fuelgauge/batteryusage/BatteryDiffEntryTest.java @@ -17,10 +17,10 @@ package com.android.settings.fuelgauge.batteryusage; import static com.google.common.truth.Truth.assertThat; -import static org.mockito.ArgumentMatchers.any; -import static org.mockito.ArgumentMatchers.eq; -import static org.mockito.Matchers.anyInt; +import static org.mockito.Mockito.any; +import static org.mockito.Mockito.anyInt; import static org.mockito.Mockito.doReturn; +import static org.mockito.Mockito.eq; import static org.mockito.Mockito.spy; import android.content.ContentValues; diff --git a/tests/robotests/src/com/android/settings/network/TetherPreferenceControllerTest.java b/tests/robotests/src/com/android/settings/network/TetherPreferenceControllerTest.java index bafc021d9b2..9aa79ff4db6 100644 --- a/tests/robotests/src/com/android/settings/network/TetherPreferenceControllerTest.java +++ b/tests/robotests/src/com/android/settings/network/TetherPreferenceControllerTest.java @@ -16,16 +16,15 @@ package com.android.settings.network; -import static org.mockito.ArgumentMatchers.any; +import static org.mockito.Mockito.any; import static org.mockito.Mockito.doReturn; import static org.mockito.Mockito.mock; import static org.mockito.Mockito.spy; import static org.mockito.Mockito.verify; +import static org.mockito.Mockito.verifyNoInteractions; import static org.mockito.Mockito.verifyNoMoreInteractions; -import static org.mockito.Mockito.verifyZeroInteractions; import static org.mockito.Mockito.when; -import android.app.admin.DevicePolicyManager; import android.bluetooth.BluetoothAdapter; import android.bluetooth.BluetoothPan; import android.bluetooth.BluetoothProfile; @@ -162,7 +161,7 @@ public class TetherPreferenceControllerTest { mController.onResume(); - verifyZeroInteractions(mPreference); + verifyNoInteractions(mPreference); Settings.Global.putInt(context.getContentResolver(), Settings.Global.AIRPLANE_MODE_ON, 1); diff --git a/tests/robotests/src/com/android/settings/notification/zen/ZenModeAutomaticRulesPreferenceControllerTest.java b/tests/robotests/src/com/android/settings/notification/zen/ZenModeAutomaticRulesPreferenceControllerTest.java index 1660955e323..3791c23193e 100644 --- a/tests/robotests/src/com/android/settings/notification/zen/ZenModeAutomaticRulesPreferenceControllerTest.java +++ b/tests/robotests/src/com/android/settings/notification/zen/ZenModeAutomaticRulesPreferenceControllerTest.java @@ -18,6 +18,7 @@ package com.android.settings.notification.zen; import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertTrue; +import static org.junit.Assert.fail; import static org.mockito.ArgumentMatchers.any; import static org.mockito.ArgumentMatchers.anyInt; import static org.mockito.Mockito.doReturn; @@ -42,10 +43,10 @@ import org.junit.Test; import org.junit.runner.RunWith; import org.mockito.Mock; import org.mockito.MockitoAnnotations; -import org.mockito.internal.util.reflection.FieldSetter; import org.robolectric.RobolectricTestRunner; import org.robolectric.util.ReflectionHelpers; +import java.lang.reflect.Field; import java.util.HashMap; import java.util.Map; @@ -192,16 +193,26 @@ public class ZenModeAutomaticRulesPreferenceControllerTest { rule.setEnabled(false); rMap.put(testId, rule); mockGetAutomaticZenRules(NUM_RULES, rMap); - FieldSetter.setField(mZenRulePreference, ZenRulePreference.class.getDeclaredField("mId"), testId); + setZenRulePreferenceField("mId", testId); mController.updateState(mockPref); verify(mZenRulePreference, times(1)).updatePreference(any()); verify(mockPref, never()).removeAll(); assertEquals(NUM_RULES, mController.mZenRulePreferences.size()); } + private void setZenRulePreferenceField(String name, Object value) { + try { + Field field = ZenRulePreference.class.getDeclaredField("mId"); + field.setAccessible(true); + field.set(mZenRulePreference, value); + } catch (ReflectiveOperationException e) { + fail("Unable to set mZenRulePreference field: " + name); + } + } + private void mockGetAutomaticZenRules(int numRules, Map rules) { Map.Entry[] arr = new Map.Entry[numRules]; rules.entrySet().toArray(arr); when(mBackend.getAutomaticZenRules()).thenReturn(arr); } -} \ No newline at end of file +} diff --git a/tests/robotests/src/com/android/settings/notification/zen/ZenModePriorityConversationsPreferenceControllerTest.java b/tests/robotests/src/com/android/settings/notification/zen/ZenModePriorityConversationsPreferenceControllerTest.java index 6bf0538c33b..d5834f932c3 100644 --- a/tests/robotests/src/com/android/settings/notification/zen/ZenModePriorityConversationsPreferenceControllerTest.java +++ b/tests/robotests/src/com/android/settings/notification/zen/ZenModePriorityConversationsPreferenceControllerTest.java @@ -26,7 +26,7 @@ import static com.android.settings.notification.zen.ZenModePriorityConversations import static com.google.common.truth.Truth.assertThat; -import static org.mockito.Matchers.any; +import static org.mockito.Mockito.any; import static org.mockito.Mockito.mock; import static org.mockito.Mockito.never; import static org.mockito.Mockito.reset; @@ -146,4 +146,4 @@ public class ZenModePriorityConversationsPreferenceControllerTest { } return null; } -} \ No newline at end of file +} diff --git a/tests/robotests/src/com/android/settings/notification/zen/ZenModePrioritySendersPreferenceControllerTest.java b/tests/robotests/src/com/android/settings/notification/zen/ZenModePrioritySendersPreferenceControllerTest.java index 99fa8e6cdd4..c431c5332ab 100644 --- a/tests/robotests/src/com/android/settings/notification/zen/ZenModePrioritySendersPreferenceControllerTest.java +++ b/tests/robotests/src/com/android/settings/notification/zen/ZenModePrioritySendersPreferenceControllerTest.java @@ -30,8 +30,8 @@ import static com.android.settings.notification.zen.ZenPrioritySendersHelper.KEY import static com.android.settings.notification.zen.ZenPrioritySendersHelper.KEY_NONE; import static com.android.settings.notification.zen.ZenPrioritySendersHelper.UNKNOWN; -import static org.mockito.ArgumentMatchers.anyInt; -import static org.mockito.Matchers.any; +import static org.mockito.Mockito.any; +import static org.mockito.Mockito.anyInt; import static org.mockito.Mockito.mock; import static org.mockito.Mockito.never; import static org.mockito.Mockito.times; diff --git a/tests/robotests/src/com/android/settings/notification/zen/ZenRuleStarredContactsPreferenceControllerTest.java b/tests/robotests/src/com/android/settings/notification/zen/ZenRuleStarredContactsPreferenceControllerTest.java index 2f7dfe7f9b8..01f3dfabcb3 100644 --- a/tests/robotests/src/com/android/settings/notification/zen/ZenRuleStarredContactsPreferenceControllerTest.java +++ b/tests/robotests/src/com/android/settings/notification/zen/ZenRuleStarredContactsPreferenceControllerTest.java @@ -18,7 +18,7 @@ package com.android.settings.notification.zen; import static com.google.common.truth.Truth.assertThat; -import static org.mockito.Matchers.any; +import static org.mockito.Mockito.any; import static org.mockito.Mockito.mock; import static org.mockito.Mockito.when; diff --git a/tests/robotests/src/com/android/settings/print/PrintJobMessagePreferenceControllerTest.java b/tests/robotests/src/com/android/settings/print/PrintJobMessagePreferenceControllerTest.java index 96036d52755..b5d07cc5231 100644 --- a/tests/robotests/src/com/android/settings/print/PrintJobMessagePreferenceControllerTest.java +++ b/tests/robotests/src/com/android/settings/print/PrintJobMessagePreferenceControllerTest.java @@ -21,8 +21,8 @@ import static androidx.lifecycle.Lifecycle.Event.ON_STOP; import static com.google.common.truth.Truth.assertThat; -import static org.mockito.ArgumentMatchers.anyInt; -import static org.mockito.ArgumentMatchers.anyObject; +import static org.mockito.Mockito.any; +import static org.mockito.Mockito.anyInt; import static org.mockito.Mockito.spy; import static org.mockito.Mockito.verify; import static org.mockito.Mockito.when; @@ -72,7 +72,7 @@ public class PrintJobMessagePreferenceControllerTest { mPreference = new Preference(mContext); when(mContext.getSystemService(Context.PRINT_SERVICE)).thenReturn(mPrintManager); when(mPrintManager.getGlobalPrintManagerForUser(anyInt())).thenReturn(mPrintManager); - when(mPrintManager.getPrintJob(anyObject())).thenReturn(mPrintJob); + when(mPrintManager.getPrintJob(any())).thenReturn(mPrintJob); when(mPrintJob.getInfo()).thenReturn(mPrintJobInfo); mController = new PrintJobMessagePreferenceController(mContext, PREF_KEY); when(mScreen.findPreference(mController.getPreferenceKey())).thenReturn(mPreference); @@ -93,7 +93,7 @@ public class PrintJobMessagePreferenceControllerTest { @Test public void updateUi_visiblePreference() { - when(mPrintJobInfo.getStatus(anyObject())).thenReturn("TestPrint"); + when(mPrintJobInfo.getStatus(any())).thenReturn("TestPrint"); mLifecycle.handleLifecycleEvent(ON_START); assertThat(mPreference.isVisible()).isTrue(); @@ -103,7 +103,7 @@ public class PrintJobMessagePreferenceControllerTest { @Test public void updateUi_invisiblePreference() { - when(mPrintJobInfo.getStatus(anyObject())).thenReturn(null); + when(mPrintJobInfo.getStatus(any())).thenReturn(null); mLifecycle.handleLifecycleEvent(ON_START); assertThat(mPreference.isVisible()).isFalse(); diff --git a/tests/robotests/src/com/android/settings/print/PrintJobPreferenceControllerTest.java b/tests/robotests/src/com/android/settings/print/PrintJobPreferenceControllerTest.java index 09c5bf368af..da4d58c757b 100644 --- a/tests/robotests/src/com/android/settings/print/PrintJobPreferenceControllerTest.java +++ b/tests/robotests/src/com/android/settings/print/PrintJobPreferenceControllerTest.java @@ -21,8 +21,8 @@ import static androidx.lifecycle.Lifecycle.Event.ON_STOP; import static com.google.common.truth.Truth.assertThat; -import static org.mockito.ArgumentMatchers.anyInt; -import static org.mockito.ArgumentMatchers.anyObject; +import static org.mockito.Mockito.any; +import static org.mockito.Mockito.anyInt; import static org.mockito.Mockito.spy; import static org.mockito.Mockito.verify; import static org.mockito.Mockito.when; @@ -75,7 +75,7 @@ public class PrintJobPreferenceControllerTest { mTestLabel = "PrintTest"; when(mContext.getSystemService(Context.PRINT_SERVICE)).thenReturn(mPrintManager); when(mPrintManager.getGlobalPrintManagerForUser(anyInt())).thenReturn(mPrintManager); - when(mPrintManager.getPrintJob(anyObject())).thenReturn(mPrintJob); + when(mPrintManager.getPrintJob(any())).thenReturn(mPrintJob); when(mPrintJob.getInfo()).thenReturn(mPrintJobInfo); mController = new PrintJobPreferenceController(mContext, PREF_KEY); when(mScreen.findPreference(mController.getPreferenceKey())).thenReturn(mPreference); diff --git a/tests/robotests/src/com/android/settings/search/actionbar/SearchMenuControllerTest.java b/tests/robotests/src/com/android/settings/search/actionbar/SearchMenuControllerTest.java index c646a935192..2a7bd29a964 100644 --- a/tests/robotests/src/com/android/settings/search/actionbar/SearchMenuControllerTest.java +++ b/tests/robotests/src/com/android/settings/search/actionbar/SearchMenuControllerTest.java @@ -21,7 +21,7 @@ import static com.android.settings.search.actionbar.SearchMenuController.MENU_SE import static org.mockito.Mockito.mock; import static org.mockito.Mockito.spy; import static org.mockito.Mockito.verify; -import static org.mockito.Mockito.verifyZeroInteractions; +import static org.mockito.Mockito.verifyNoInteractions; import static org.mockito.Mockito.when; import android.app.settings.SettingsEnums; @@ -91,7 +91,7 @@ public class SearchMenuControllerTest { SearchMenuController.init(mHost); mHost.getSettingsLifecycle().onCreateOptionsMenu(mMenu, null /* inflater */); - verifyZeroInteractions(mMenu); + verifyNoInteractions(mMenu); } @Test @@ -100,7 +100,7 @@ public class SearchMenuControllerTest { SearchMenuController.init(mHost); mHost.getSettingsLifecycle().onCreateOptionsMenu(mMenu, null /* inflater */); - verifyZeroInteractions(mMenu); + verifyNoInteractions(mMenu); } @Test @@ -112,6 +112,6 @@ public class SearchMenuControllerTest { mHost.getSettingsLifecycle().onCreateOptionsMenu(mMenu, null /* inflater */); - verifyZeroInteractions(mMenu); + verifyNoInteractions(mMenu); } } diff --git a/tests/robotests/src/com/android/settings/widget/HighlightablePreferenceGroupAdapterTest.java b/tests/robotests/src/com/android/settings/widget/HighlightablePreferenceGroupAdapterTest.java index 62c7bd8091c..e297b788af9 100644 --- a/tests/robotests/src/com/android/settings/widget/HighlightablePreferenceGroupAdapterTest.java +++ b/tests/robotests/src/com/android/settings/widget/HighlightablePreferenceGroupAdapterTest.java @@ -26,7 +26,7 @@ import static org.mockito.Mockito.mock; import static org.mockito.Mockito.spy; import static org.mockito.Mockito.times; import static org.mockito.Mockito.verify; -import static org.mockito.Mockito.verifyZeroInteractions; +import static org.mockito.Mockito.verifyNoInteractions; import static org.mockito.Mockito.when; import android.content.Context; @@ -111,7 +111,7 @@ public class HighlightablePreferenceGroupAdapterTest { ReflectionHelpers.setField(mAdapter, "mHighlightRequested", false); mAdapter.requestHighlight(mRoot, null /* recyclerView */, mock(AppBarLayout.class)); - verifyZeroInteractions(mRoot); + verifyNoInteractions(mRoot); } @Test @@ -125,7 +125,7 @@ public class HighlightablePreferenceGroupAdapterTest { when(mFragment.getArguments()).thenReturn(null); when(mFragment.getPreferenceScreen()).thenReturn(screen); HighlightablePreferenceGroupAdapter.adjustInitialExpandedChildCount(mFragment); - verifyZeroInteractions(screen); + verifyNoInteractions(screen); } @Test @@ -150,7 +150,7 @@ public class HighlightablePreferenceGroupAdapterTest { HighlightablePreferenceGroupAdapter.adjustInitialExpandedChildCount(mFragment); verify(mFragment).getInitialExpandedChildCount(); - verifyZeroInteractions(screen); + verifyNoInteractions(screen); } @Test diff --git a/tests/robotests/src/com/android/settings/wifi/NetworkRequestDialogFragmentTest.java b/tests/robotests/src/com/android/settings/wifi/NetworkRequestDialogFragmentTest.java index e4d39828c7f..e5b13c4b114 100644 --- a/tests/robotests/src/com/android/settings/wifi/NetworkRequestDialogFragmentTest.java +++ b/tests/robotests/src/com/android/settings/wifi/NetworkRequestDialogFragmentTest.java @@ -195,7 +195,7 @@ public class NetworkRequestDialogFragmentTest { } private List createWifiEntryList() { - List wifiEntryList = spy(new ArrayList<>()); + List wifiEntryList = new ArrayList<>(); final WifiEntry wifiEntry1 = mock(WifiEntry.class); when(wifiEntry1.getSsid()).thenReturn("Test AP 1"); diff --git a/tests/robotests/src/com/android/settings/wifi/calling/DisclaimerItemListAdapterTest.java b/tests/robotests/src/com/android/settings/wifi/calling/DisclaimerItemListAdapterTest.java index 8478a5471ac..610e520e49b 100644 --- a/tests/robotests/src/com/android/settings/wifi/calling/DisclaimerItemListAdapterTest.java +++ b/tests/robotests/src/com/android/settings/wifi/calling/DisclaimerItemListAdapterTest.java @@ -21,9 +21,9 @@ import static com.android.settings.wifi.calling.DisclaimerItemListAdapter import static com.android.settings.wifi.calling.DisclaimerItemListAdapter .DisclaimerItemViewHolder.ID_DISCLAIMER_ITEM_TITLE; -import static org.mockito.Matchers.anyBoolean; -import static org.mockito.Matchers.anyInt; -import static org.mockito.Matchers.anyObject; +import static org.mockito.Mockito.any; +import static org.mockito.Mockito.anyBoolean; +import static org.mockito.Mockito.anyInt; import static org.mockito.Mockito.spy; import static org.mockito.Mockito.verify; import static org.mockito.Mockito.when; @@ -75,7 +75,7 @@ public class DisclaimerItemListAdapterTest { mDisclaimerItem = spy(new MockDisclaimerItem(mContext, 0 /* subId */)); mDisclaimerItemList.add(mDisclaimerItem); - when(mLayoutInflater.inflate(anyInt(), anyObject(), anyBoolean())).thenReturn(mView); + when(mLayoutInflater.inflate(anyInt(), any(), anyBoolean())).thenReturn(mView); when(mViewGroup.getContext()).thenReturn(mContext); when(mViewGroup.getContext().getSystemService(Context.LAYOUT_INFLATER_SERVICE)).thenReturn( mLayoutInflater); diff --git a/tests/robotests/src/com/android/settings/wifi/calling/EmergencyCallLimitationDisclaimerTest.java b/tests/robotests/src/com/android/settings/wifi/calling/EmergencyCallLimitationDisclaimerTest.java index 985edda69b5..c8e4cce6d0a 100644 --- a/tests/robotests/src/com/android/settings/wifi/calling/EmergencyCallLimitationDisclaimerTest.java +++ b/tests/robotests/src/com/android/settings/wifi/calling/EmergencyCallLimitationDisclaimerTest.java @@ -18,8 +18,8 @@ package com.android.settings.wifi.calling; import static com.google.common.truth.Truth.assertThat; -import static org.mockito.Matchers.anyInt; -import static org.mockito.Matchers.anyString; +import static org.mockito.Mockito.anyInt; +import static org.mockito.Mockito.anyString; import static org.mockito.Mockito.doReturn; import static org.mockito.Mockito.spy; import static org.mockito.Mockito.when; diff --git a/tests/robotests/src/com/android/settings/wifi/calling/LocationPolicyDisclaimerTest.java b/tests/robotests/src/com/android/settings/wifi/calling/LocationPolicyDisclaimerTest.java index 3fe9678c26e..148095aac02 100644 --- a/tests/robotests/src/com/android/settings/wifi/calling/LocationPolicyDisclaimerTest.java +++ b/tests/robotests/src/com/android/settings/wifi/calling/LocationPolicyDisclaimerTest.java @@ -18,8 +18,8 @@ package com.android.settings.wifi.calling; import static com.google.common.truth.Truth.assertThat; -import static org.mockito.Matchers.anyInt; -import static org.mockito.Matchers.anyString; +import static org.mockito.Mockito.anyInt; +import static org.mockito.Mockito.anyString; import static org.mockito.Mockito.doReturn; import static org.mockito.Mockito.spy; import static org.mockito.Mockito.when; diff --git a/tests/robotests/src/com/android/settings/wifi/calling/WifiCallingDisclaimerFragmentTest.java b/tests/robotests/src/com/android/settings/wifi/calling/WifiCallingDisclaimerFragmentTest.java index 207a2318c61..0ece537371f 100644 --- a/tests/robotests/src/com/android/settings/wifi/calling/WifiCallingDisclaimerFragmentTest.java +++ b/tests/robotests/src/com/android/settings/wifi/calling/WifiCallingDisclaimerFragmentTest.java @@ -16,10 +16,9 @@ package com.android.settings.wifi.calling; -import static org.mockito.Matchers.any; -import static org.mockito.Matchers.anyBoolean; -import static org.mockito.Matchers.anyInt; -import static org.mockito.Matchers.anyObject; +import static org.mockito.Mockito.any; +import static org.mockito.Mockito.anyBoolean; +import static org.mockito.Mockito.anyInt; import static org.mockito.Mockito.doNothing; import static org.mockito.Mockito.doReturn; import static org.mockito.Mockito.never; @@ -95,7 +94,7 @@ public class WifiCallingDisclaimerFragmentTest { doReturn(mActivity).when(mFragment).getActivity(); - when(mLayoutInflater.inflate(anyInt(), anyObject(), anyBoolean())).thenReturn(mView); + when(mLayoutInflater.inflate(anyInt(), any(), anyBoolean())).thenReturn(mView); when(mView.findViewById(R.id.agree_button)).thenReturn(mAgreeButton); when(mView.findViewById(R.id.disagree_button)).thenReturn(mDisagreeButton); when(mView.findViewById(R.id.disclaimer_item_list)).thenReturn(mRecyclerView); diff --git a/tests/robotests/src/com/android/settings/wifi/calling/WifiCallingSettingsForSubTest.java b/tests/robotests/src/com/android/settings/wifi/calling/WifiCallingSettingsForSubTest.java index e2c5ca33b4c..1faa611b970 100644 --- a/tests/robotests/src/com/android/settings/wifi/calling/WifiCallingSettingsForSubTest.java +++ b/tests/robotests/src/com/android/settings/wifi/calling/WifiCallingSettingsForSubTest.java @@ -16,17 +16,17 @@ package com.android.settings.wifi.calling; -import static junit.framework.Assert.assertEquals; - import static com.android.settings.SettingsActivity.EXTRA_SHOW_FRAGMENT; import static com.google.common.truth.Truth.assertThat; -import static org.mockito.Matchers.any; -import static org.mockito.Matchers.anyInt; -import static org.mockito.Matchers.eq; +import static junit.framework.Assert.assertEquals; + +import static org.mockito.Mockito.any; +import static org.mockito.Mockito.anyInt; import static org.mockito.Mockito.doNothing; import static org.mockito.Mockito.doReturn; +import static org.mockito.Mockito.eq; import static org.mockito.Mockito.mock; import static org.mockito.Mockito.never; import static org.mockito.Mockito.spy; @@ -44,7 +44,6 @@ import android.telephony.CarrierConfigManager; import android.telephony.TelephonyManager; import android.telephony.ims.ImsMmTelManager; import android.view.View; -import android.widget.TextView; import androidx.preference.Preference; import androidx.preference.PreferenceScreen; diff --git a/tests/robotests/src/com/android/settings/wifi/details2/WifiDetailPreferenceController2Test.java b/tests/robotests/src/com/android/settings/wifi/details2/WifiDetailPreferenceController2Test.java index 274ac16fbfc..c86a0230351 100644 --- a/tests/robotests/src/com/android/settings/wifi/details2/WifiDetailPreferenceController2Test.java +++ b/tests/robotests/src/com/android/settings/wifi/details2/WifiDetailPreferenceController2Test.java @@ -17,15 +17,16 @@ package com.android.settings.wifi.details2; import static com.google.common.truth.Truth.assertThat; -import static org.mockito.ArgumentMatchers.any; -import static org.mockito.ArgumentMatchers.anyBoolean; -import static org.mockito.ArgumentMatchers.anyInt; -import static org.mockito.ArgumentMatchers.nullable; +import static org.mockito.Mockito.any; +import static org.mockito.Mockito.anyBoolean; +import static org.mockito.Mockito.anyInt; +import static org.mockito.Mockito.anyString; import static org.mockito.Mockito.atLeastOnce; import static org.mockito.Mockito.doNothing; import static org.mockito.Mockito.inOrder; import static org.mockito.Mockito.mock; import static org.mockito.Mockito.never; +import static org.mockito.Mockito.nullable; import static org.mockito.Mockito.reset; import static org.mockito.Mockito.spy; import static org.mockito.Mockito.times; @@ -93,7 +94,6 @@ import org.mockito.Answers; import org.mockito.ArgumentCaptor; import org.mockito.Captor; import org.mockito.InOrder; -import org.mockito.Matchers; import org.mockito.Mock; import org.mockito.MockitoAnnotations; import org.robolectric.RobolectricTestRunner; @@ -1255,7 +1255,7 @@ public class WifiDetailPreferenceController2Test { mMockWifiConfig.creatorUid = doUid; ComponentName doComponent = new ComponentName(doPackage, "some.Class"); try { - when(mMockPackageManager.getPackageUidAsUser(Matchers.anyString(), Matchers.anyInt())) + when(mMockPackageManager.getPackageUidAsUser(anyString(), anyInt())) .thenReturn(doUid); } catch (PackageManager.NameNotFoundException e) { //do nothing