From c52a581bb730883ce3e2f322bfc01fbda830b0b4 Mon Sep 17 00:00:00 2001 From: Fan Zhang Date: Mon, 28 Jan 2019 16:32:56 -0800 Subject: [PATCH] Clean up ShadowUserManager Remove addManagedUser() and use default shadow for the same purpose. Test: make RunSettingsRoboTests -j40 ROBOTEST_FILTER=Redaction Change-Id: I2f86bd51bccbde23a8a37d19b28e30cadb2b9f93 --- .../notification/RedactionInterstitialTest.java | 13 +++++++++---- .../testutils/shadow/ShadowUserManager.java | 9 --------- 2 files changed, 9 insertions(+), 13 deletions(-) diff --git a/tests/robotests/src/com/android/settings/notification/RedactionInterstitialTest.java b/tests/robotests/src/com/android/settings/notification/RedactionInterstitialTest.java index 61fe0258682..a2ca62eda6b 100644 --- a/tests/robotests/src/com/android/settings/notification/RedactionInterstitialTest.java +++ b/tests/robotests/src/com/android/settings/notification/RedactionInterstitialTest.java @@ -12,6 +12,7 @@ import static org.robolectric.Robolectric.buildActivity; import android.content.ContentResolver; import android.content.Intent; import android.os.UserHandle; +import android.os.UserManager; import android.provider.Settings; import android.view.View; import android.widget.RadioButton; @@ -20,7 +21,6 @@ import com.android.settings.R; import com.android.settings.RestrictedRadioButton; import com.android.settings.notification.RedactionInterstitial.RedactionInterstitialFragment; import com.android.settings.testutils.shadow.ShadowRestrictedLockUtilsInternal; -import com.android.settings.testutils.shadow.ShadowUserManager; import com.android.settings.testutils.shadow.ShadowUtils; import org.junit.After; @@ -30,12 +30,13 @@ import org.junit.runner.RunWith; import org.robolectric.RobolectricTestRunner; import org.robolectric.RuntimeEnvironment; import org.robolectric.annotation.Config; +import org.robolectric.shadow.api.Shadow; +import org.robolectric.shadows.ShadowUserManager; @RunWith(RobolectricTestRunner.class) @Config(shadows = { ShadowUtils.class, ShadowRestrictedLockUtilsInternal.class, - ShadowUserManager.class, }) public class RedactionInterstitialTest { private RedactionInterstitial mActivity; @@ -103,7 +104,9 @@ public class RedactionInterstitialTest { @Test public void managedProfileNoRestrictionsTest() { setupSettings(1 /* show */, 1 /* showUnredacted */); - ShadowUserManager.getShadow().addManagedProfile(UserHandle.myUserId()); + final ShadowUserManager sum = + Shadow.extract(RuntimeEnvironment.application.getSystemService(UserManager.class)); + sum.setManagedProfile(true); setupActivity(); assertHideAllVisible(false); @@ -114,7 +117,9 @@ public class RedactionInterstitialTest { @Test public void managedProfileUnredactedRestrictionTest() { setupSettings(1 /* show */, 1 /* showUnredacted */); - ShadowUserManager.getShadow().addManagedProfile(UserHandle.myUserId()); + final ShadowUserManager sum = + Shadow.extract(RuntimeEnvironment.application.getSystemService(UserManager.class)); + sum.setManagedProfile(true); ShadowRestrictedLockUtilsInternal.setKeyguardDisabledFeatures( KEYGUARD_DISABLE_UNREDACTED_NOTIFICATIONS); setupActivity(); diff --git a/tests/robotests/src/com/android/settings/testutils/shadow/ShadowUserManager.java b/tests/robotests/src/com/android/settings/testutils/shadow/ShadowUserManager.java index d8703beb4b9..9a18c1f9aaf 100644 --- a/tests/robotests/src/com/android/settings/testutils/shadow/ShadowUserManager.java +++ b/tests/robotests/src/com/android/settings/testutils/shadow/ShadowUserManager.java @@ -98,15 +98,6 @@ public class ShadowUserManager extends org.robolectric.shadows.ShadowUserManager mRestrictionSources.put(restrictionKey + userHandle.getIdentifier(), enforcers); } - @Implementation - protected boolean isManagedProfile(@UserIdInt int userId) { - return mManagedProfiles.contains(userId); - } - - public void addManagedProfile(int userId) { - mManagedProfiles.add(userId); - } - @Implementation protected boolean isQuietModeEnabled(UserHandle userHandle) { return mIsQuietModeEnabled;