diff --git a/Tethering/src/com/android/networkstack/tethering/EntitlementManager.java b/Tethering/src/com/android/networkstack/tethering/EntitlementManager.java index fb162267bb..a9421660d9 100644 --- a/Tethering/src/com/android/networkstack/tethering/EntitlementManager.java +++ b/Tethering/src/com/android/networkstack/tethering/EntitlementManager.java @@ -154,7 +154,9 @@ public class EntitlementManager { // Only launch entitlement UI for the current user if it is allowed to // change tethering. This usually means the system user or the admin users in HSUM. - if (SdkLevel.isAtLeastT()) { + // TODO (b/382624069): Figure out whether it is safe to call createContextAsUser + // from secondary user. And re-enable the check or remove the code accordingly. + if (false) { // Create a user context for the current foreground user as UserManager#isAdmin() // operates on the context user. final int currentUserId = getCurrentUser(); diff --git a/Tethering/tests/unit/src/com/android/networkstack/tethering/EntitlementManagerTest.java b/Tethering/tests/unit/src/com/android/networkstack/tethering/EntitlementManagerTest.java index 51c2d56d09..16ebbbbdbb 100644 --- a/Tethering/tests/unit/src/com/android/networkstack/tethering/EntitlementManagerTest.java +++ b/Tethering/tests/unit/src/com/android/networkstack/tethering/EntitlementManagerTest.java @@ -38,7 +38,6 @@ import static android.telephony.SubscriptionManager.INVALID_SUBSCRIPTION_ID; import static com.android.dx.mockito.inline.extended.ExtendedMockito.doReturn; import static com.android.dx.mockito.inline.extended.ExtendedMockito.mockitoSession; import static com.android.networkstack.apishim.ConstantsShim.KEY_CARRIER_SUPPORTS_TETHERING_BOOL; -import static com.android.testutils.DevSdkIgnoreRule.IgnoreAfter; import static com.android.testutils.DevSdkIgnoreRule.IgnoreUpTo; import static com.android.testutils.DevSdkIgnoreRuleKt.SC_V2; @@ -592,16 +591,8 @@ public final class EntitlementManagerTest { .onTetherProvisioningFailed(TETHERING_WIFI, FAILED_TETHERING_REASON); } - @IgnoreUpTo(SC_V2) @Test - public void testUiProvisioningMultiUser_aboveT() { - doTestUiProvisioningMultiUser(true, 1); - doTestUiProvisioningMultiUser(false, 0); - } - - @IgnoreAfter(SC_V2) - @Test - public void testUiProvisioningMultiUser_belowT() { + public void testUiProvisioningMultiUser() { doTestUiProvisioningMultiUser(true, 1); doTestUiProvisioningMultiUser(false, 1); }