Merge "Enable Encryption & Creds UI for Secondary users" into tm-dev
diff --git a/src/com/android/settings/security/EncryptionAndCredential.java b/src/com/android/settings/security/EncryptionAndCredential.java
index 5362ad0..e3dd9d5 100644
--- a/src/com/android/settings/security/EncryptionAndCredential.java
+++ b/src/com/android/settings/security/EncryptionAndCredential.java
@@ -20,7 +20,6 @@
import android.app.settings.SettingsEnums;
import android.content.Context;
-import android.os.UserManager;
import com.android.settings.R;
import com.android.settings.dashboard.DashboardFragment;
@@ -96,9 +95,7 @@
@Override
protected boolean isPageSearchEnabled(Context context) {
- final UserManager um = (UserManager) context.getSystemService(
- Context.USER_SERVICE);
- return um.isAdminUser();
+ return true;
}
};
}
diff --git a/src/com/android/settings/security/EncryptionStatusPreferenceController.java b/src/com/android/settings/security/EncryptionStatusPreferenceController.java
index d835798..68d1181 100644
--- a/src/com/android/settings/security/EncryptionStatusPreferenceController.java
+++ b/src/com/android/settings/security/EncryptionStatusPreferenceController.java
@@ -44,11 +44,11 @@
public int getAvailabilityStatus() {
if (TextUtils.equals(getPreferenceKey(), PREF_KEY_ENCRYPTION_DETAIL_PAGE) &&
!mContext.getResources().getBoolean(
- R.bool.config_show_encryption_and_credentials_encryption_status)) {
+ R.bool.config_show_encryption_and_credentials_encryption_status)) {
return UNSUPPORTED_ON_DEVICE;
}
- return mUserManager.isAdminUser() ? AVAILABLE : DISABLED_FOR_USER;
+ return AVAILABLE;
}
@Override
diff --git a/tests/robotests/src/com/android/settings/security/EncryptionAndCredentialTest.java b/tests/robotests/src/com/android/settings/security/EncryptionAndCredentialTest.java
index 6f326ce..1d7a4a7 100644
--- a/tests/robotests/src/com/android/settings/security/EncryptionAndCredentialTest.java
+++ b/tests/robotests/src/com/android/settings/security/EncryptionAndCredentialTest.java
@@ -16,19 +16,13 @@
package com.android.settings.security;
-import static com.android.settings.security.EncryptionAndCredential.SEARCH_INDEX_DATA_PROVIDER;
-
import static com.google.common.truth.Truth.assertThat;
-import static org.mockito.Mockito.when;
-
import android.app.admin.DevicePolicyManager;
import android.content.Context;
import android.os.UserManager;
-import android.provider.SearchIndexableResource;
import com.android.internal.logging.nano.MetricsProto.MetricsEvent;
-import com.android.settings.search.BaseSearchIndexProvider;
import org.junit.Before;
import org.junit.Test;
@@ -39,9 +33,6 @@
import org.robolectric.RuntimeEnvironment;
import org.robolectric.shadows.ShadowApplication;
-import java.util.ArrayList;
-import java.util.List;
-
@RunWith(RobolectricTestRunner.class)
public class EncryptionAndCredentialTest {
@@ -67,19 +58,4 @@
assertThat(fragment.getMetricsCategory()).isEqualTo(MetricsEvent.ENCRYPTION_AND_CREDENTIAL);
}
- @Test
- public void getNonIndexableKeys_pageIsDisabled_shouldReturnAllKeysAsNonIndexable() {
- when(mUserManager.isAdminUser()).thenReturn(false);
-
- final List<SearchIndexableResource> index =
- SEARCH_INDEX_DATA_PROVIDER.getXmlResourcesToIndex(mContext, true /* enabled */);
- final List<String> expectedKeys = new ArrayList<>();
- for (SearchIndexableResource res : index) {
- expectedKeys.addAll(((BaseSearchIndexProvider) SEARCH_INDEX_DATA_PROVIDER)
- .getNonIndexableKeysFromXml(mContext, res.xmlResId, true /* suppressAll */));
- }
- final List<String> keys = SEARCH_INDEX_DATA_PROVIDER.getNonIndexableKeys(mContext);
-
- assertThat(keys).containsExactlyElementsIn(expectedKeys);
- }
}
diff --git a/tests/robotests/src/com/android/settings/security/EncryptionStatusPreferenceControllerTest.java b/tests/robotests/src/com/android/settings/security/EncryptionStatusPreferenceControllerTest.java
index 970564a..410198b 100644
--- a/tests/robotests/src/com/android/settings/security/EncryptionStatusPreferenceControllerTest.java
+++ b/tests/robotests/src/com/android/settings/security/EncryptionStatusPreferenceControllerTest.java
@@ -57,20 +57,6 @@
}
@Test
- public void isAvailable_admin_true() {
- mShadowUserManager.setIsAdminUser(true);
-
- assertThat(mController.isAvailable()).isTrue();
- }
-
- @Test
- public void isAvailable_notAdmin_false() {
- mShadowUserManager.setIsAdminUser(false);
-
- assertThat(mController.isAvailable()).isFalse();
- }
-
- @Test
@Config(qualifiers = "mcc999")
public void isAvailable_notVisible_false() {
assertThat(mController.isAvailable()).isFalse();
@@ -82,7 +68,6 @@
mController = new EncryptionStatusPreferenceController(mContext,
PREF_KEY_ENCRYPTION_SECURITY_PAGE);
- mShadowUserManager.setIsAdminUser(true);
assertThat(mController.isAvailable()).isTrue();
}