Merge "canAddPrivateProfile checks for private space creation test" into main
This commit is contained in:
committed by
Android (Google) Code Review
commit
1ddef782c8
@@ -18,6 +18,7 @@ package com.android.settings.privatespace;
|
|||||||
|
|
||||||
import static com.google.common.truth.Truth.assertThat;
|
import static com.google.common.truth.Truth.assertThat;
|
||||||
|
|
||||||
|
import static org.junit.Assume.assumeTrue;
|
||||||
import static org.mockito.ArgumentMatchers.any;
|
import static org.mockito.ArgumentMatchers.any;
|
||||||
import static org.mockito.Mockito.doNothing;
|
import static org.mockito.Mockito.doNothing;
|
||||||
import static org.mockito.Mockito.spy;
|
import static org.mockito.Mockito.spy;
|
||||||
@@ -27,6 +28,7 @@ import static org.mockito.Mockito.when;
|
|||||||
import android.content.Context;
|
import android.content.Context;
|
||||||
import android.content.Intent;
|
import android.content.Intent;
|
||||||
import android.os.Flags;
|
import android.os.Flags;
|
||||||
|
import android.os.UserManager;
|
||||||
import android.platform.test.annotations.RequiresFlagsEnabled;
|
import android.platform.test.annotations.RequiresFlagsEnabled;
|
||||||
import android.platform.test.flag.junit.CheckFlagsRule;
|
import android.platform.test.flag.junit.CheckFlagsRule;
|
||||||
import android.platform.test.flag.junit.DeviceFlagsValueProvider;
|
import android.platform.test.flag.junit.DeviceFlagsValueProvider;
|
||||||
@@ -91,6 +93,7 @@ public class PrivateSpaceAuthenticationActivityTest {
|
|||||||
@RequiresFlagsEnabled({Flags.FLAG_ALLOW_PRIVATE_PROFILE,
|
@RequiresFlagsEnabled({Flags.FLAG_ALLOW_PRIVATE_PROFILE,
|
||||||
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES})
|
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES})
|
||||||
public void whenPrivateProfileDoesNotExist_triggersSetupFlow() {
|
public void whenPrivateProfileDoesNotExist_triggersSetupFlow() {
|
||||||
|
assumeTrue(mContext.getSystemService(UserManager.class).canAddPrivateProfile());
|
||||||
when(mPrivateSpaceMaintainer.doesPrivateSpaceExist()).thenReturn(false);
|
when(mPrivateSpaceMaintainer.doesPrivateSpaceExist()).thenReturn(false);
|
||||||
|
|
||||||
final ArgumentCaptor<Intent> intentCaptor = ArgumentCaptor.forClass(Intent.class);
|
final ArgumentCaptor<Intent> intentCaptor = ArgumentCaptor.forClass(Intent.class);
|
||||||
|
@@ -81,6 +81,7 @@ public class PrivateSpaceMaintainerTest {
|
|||||||
public void deletePrivateSpace_psExists_deletesPS() {
|
public void deletePrivateSpace_psExists_deletesPS() {
|
||||||
mSetFlagsRule.enableFlags(
|
mSetFlagsRule.enableFlags(
|
||||||
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES);
|
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES);
|
||||||
|
assumeTrue(mContext.getSystemService(UserManager.class).canAddPrivateProfile());
|
||||||
PrivateSpaceMaintainer privateSpaceMaintainer =
|
PrivateSpaceMaintainer privateSpaceMaintainer =
|
||||||
PrivateSpaceMaintainer.getInstance(mContext);
|
PrivateSpaceMaintainer.getInstance(mContext);
|
||||||
privateSpaceMaintainer.createPrivateSpace();
|
privateSpaceMaintainer.createPrivateSpace();
|
||||||
@@ -114,6 +115,7 @@ public class PrivateSpaceMaintainerTest {
|
|||||||
public void createPrivateSpace_psDoesNotExist_createsPS() {
|
public void createPrivateSpace_psDoesNotExist_createsPS() {
|
||||||
mSetFlagsRule.enableFlags(
|
mSetFlagsRule.enableFlags(
|
||||||
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES);
|
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES);
|
||||||
|
assumeTrue(mContext.getSystemService(UserManager.class).canAddPrivateProfile());
|
||||||
PrivateSpaceMaintainer privateSpaceMaintainer =
|
PrivateSpaceMaintainer privateSpaceMaintainer =
|
||||||
PrivateSpaceMaintainer.getInstance(mContext);
|
PrivateSpaceMaintainer.getInstance(mContext);
|
||||||
privateSpaceMaintainer.deletePrivateSpace();
|
privateSpaceMaintainer.deletePrivateSpace();
|
||||||
@@ -129,6 +131,7 @@ public class PrivateSpaceMaintainerTest {
|
|||||||
public void createPrivateSpace_psExists_returnsFalse() {
|
public void createPrivateSpace_psExists_returnsFalse() {
|
||||||
mSetFlagsRule.enableFlags(
|
mSetFlagsRule.enableFlags(
|
||||||
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES);
|
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES);
|
||||||
|
assumeTrue(mContext.getSystemService(UserManager.class).canAddPrivateProfile());
|
||||||
PrivateSpaceMaintainer privateSpaceMaintainer =
|
PrivateSpaceMaintainer privateSpaceMaintainer =
|
||||||
PrivateSpaceMaintainer.getInstance(mContext);
|
PrivateSpaceMaintainer.getInstance(mContext);
|
||||||
privateSpaceMaintainer.deletePrivateSpace();
|
privateSpaceMaintainer.deletePrivateSpace();
|
||||||
@@ -145,6 +148,7 @@ public class PrivateSpaceMaintainerTest {
|
|||||||
public void createPrivateSpace_psDoesNotExist_resetsHidePSSettings() {
|
public void createPrivateSpace_psDoesNotExist_resetsHidePSSettings() {
|
||||||
mSetFlagsRule.enableFlags(
|
mSetFlagsRule.enableFlags(
|
||||||
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES);
|
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES);
|
||||||
|
assumeTrue(mContext.getSystemService(UserManager.class).canAddPrivateProfile());
|
||||||
PrivateSpaceMaintainer privateSpaceMaintainer =
|
PrivateSpaceMaintainer privateSpaceMaintainer =
|
||||||
PrivateSpaceMaintainer.getInstance(mContext);
|
PrivateSpaceMaintainer.getInstance(mContext);
|
||||||
Settings.Secure.putInt(
|
Settings.Secure.putInt(
|
||||||
@@ -168,6 +172,7 @@ public class PrivateSpaceMaintainerTest {
|
|||||||
Flags.FLAG_ALLOW_PRIVATE_PROFILE,
|
Flags.FLAG_ALLOW_PRIVATE_PROFILE,
|
||||||
android.multiuser.Flags.FLAG_ENABLE_PS_SENSITIVE_NOTIFICATIONS_TOGGLE,
|
android.multiuser.Flags.FLAG_ENABLE_PS_SENSITIVE_NOTIFICATIONS_TOGGLE,
|
||||||
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES);
|
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES);
|
||||||
|
assumeTrue(mContext.getSystemService(UserManager.class).canAddPrivateProfile());
|
||||||
PrivateSpaceMaintainer privateSpaceMaintainer =
|
PrivateSpaceMaintainer privateSpaceMaintainer =
|
||||||
PrivateSpaceMaintainer.getInstance(mContext);
|
PrivateSpaceMaintainer.getInstance(mContext);
|
||||||
privateSpaceMaintainer.deletePrivateSpace();
|
privateSpaceMaintainer.deletePrivateSpace();
|
||||||
@@ -185,6 +190,7 @@ public class PrivateSpaceMaintainerTest {
|
|||||||
public void createPrivateSpace_psExists_doesNotResetHidePSSettings() {
|
public void createPrivateSpace_psExists_doesNotResetHidePSSettings() {
|
||||||
mSetFlagsRule.enableFlags(
|
mSetFlagsRule.enableFlags(
|
||||||
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES);
|
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES);
|
||||||
|
assumeTrue(mContext.getSystemService(UserManager.class).canAddPrivateProfile());
|
||||||
PrivateSpaceMaintainer privateSpaceMaintainer =
|
PrivateSpaceMaintainer privateSpaceMaintainer =
|
||||||
PrivateSpaceMaintainer.getInstance(mContext);
|
PrivateSpaceMaintainer.getInstance(mContext);
|
||||||
privateSpaceMaintainer.createPrivateSpace();
|
privateSpaceMaintainer.createPrivateSpace();
|
||||||
@@ -230,6 +236,7 @@ public class PrivateSpaceMaintainerTest {
|
|||||||
public void lockPrivateSpace_psExistsAndPrivateProfileRunning_locksCreatedPrivateSpace() {
|
public void lockPrivateSpace_psExistsAndPrivateProfileRunning_locksCreatedPrivateSpace() {
|
||||||
mSetFlagsRule.enableFlags(
|
mSetFlagsRule.enableFlags(
|
||||||
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES);
|
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES);
|
||||||
|
assumeTrue(mContext.getSystemService(UserManager.class).canAddPrivateProfile());
|
||||||
PrivateSpaceMaintainer privateSpaceMaintainer =
|
PrivateSpaceMaintainer privateSpaceMaintainer =
|
||||||
PrivateSpaceMaintainer.getInstance(mContext);
|
PrivateSpaceMaintainer.getInstance(mContext);
|
||||||
privateSpaceMaintainer.createPrivateSpace();
|
privateSpaceMaintainer.createPrivateSpace();
|
||||||
@@ -248,6 +255,7 @@ public class PrivateSpaceMaintainerTest {
|
|||||||
public void lockPrivateSpace_psExistsAndPrivateProfileNotRunning_returnsFalse() {
|
public void lockPrivateSpace_psExistsAndPrivateProfileNotRunning_returnsFalse() {
|
||||||
mSetFlagsRule.enableFlags(
|
mSetFlagsRule.enableFlags(
|
||||||
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES);
|
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES);
|
||||||
|
assumeTrue(mContext.getSystemService(UserManager.class).canAddPrivateProfile());
|
||||||
PrivateSpaceMaintainer privateSpaceMaintainer =
|
PrivateSpaceMaintainer privateSpaceMaintainer =
|
||||||
PrivateSpaceMaintainer.getInstance(mContext);
|
PrivateSpaceMaintainer.getInstance(mContext);
|
||||||
privateSpaceMaintainer.createPrivateSpace();
|
privateSpaceMaintainer.createPrivateSpace();
|
||||||
@@ -284,6 +292,7 @@ public class PrivateSpaceMaintainerTest {
|
|||||||
public void createPrivateSpace_psDoesNotExist_setsUserSetupComplete() {
|
public void createPrivateSpace_psDoesNotExist_setsUserSetupComplete() {
|
||||||
mSetFlagsRule.enableFlags(
|
mSetFlagsRule.enableFlags(
|
||||||
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES);
|
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES);
|
||||||
|
assumeTrue(mContext.getSystemService(UserManager.class).canAddPrivateProfile());
|
||||||
PrivateSpaceMaintainer privateSpaceMaintainer =
|
PrivateSpaceMaintainer privateSpaceMaintainer =
|
||||||
PrivateSpaceMaintainer.getInstance(mContext);
|
PrivateSpaceMaintainer.getInstance(mContext);
|
||||||
privateSpaceMaintainer.createPrivateSpace();
|
privateSpaceMaintainer.createPrivateSpace();
|
||||||
@@ -298,6 +307,7 @@ public class PrivateSpaceMaintainerTest {
|
|||||||
public void createPrivateSpace_pSExists_doesNotChangeUserSetupSetting() {
|
public void createPrivateSpace_pSExists_doesNotChangeUserSetupSetting() {
|
||||||
mSetFlagsRule.enableFlags(
|
mSetFlagsRule.enableFlags(
|
||||||
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES);
|
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES);
|
||||||
|
assumeTrue(mContext.getSystemService(UserManager.class).canAddPrivateProfile());
|
||||||
PrivateSpaceMaintainer privateSpaceMaintainer =
|
PrivateSpaceMaintainer privateSpaceMaintainer =
|
||||||
PrivateSpaceMaintainer.getInstance(mContext);
|
PrivateSpaceMaintainer.getInstance(mContext);
|
||||||
privateSpaceMaintainer.createPrivateSpace();
|
privateSpaceMaintainer.createPrivateSpace();
|
||||||
@@ -316,6 +326,7 @@ public class PrivateSpaceMaintainerTest {
|
|||||||
Flags.FLAG_ALLOW_PRIVATE_PROFILE,
|
Flags.FLAG_ALLOW_PRIVATE_PROFILE,
|
||||||
android.multiuser.Flags.FLAG_SUPPORT_AUTOLOCK_FOR_PRIVATE_SPACE,
|
android.multiuser.Flags.FLAG_SUPPORT_AUTOLOCK_FOR_PRIVATE_SPACE,
|
||||||
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES);
|
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES);
|
||||||
|
assumeTrue(mContext.getSystemService(UserManager.class).canAddPrivateProfile());
|
||||||
final int autoLockOption = 2;
|
final int autoLockOption = 2;
|
||||||
PrivateSpaceMaintainer privateSpaceMaintainer =
|
PrivateSpaceMaintainer privateSpaceMaintainer =
|
||||||
PrivateSpaceMaintainer.getInstance(mContext);
|
PrivateSpaceMaintainer.getInstance(mContext);
|
||||||
@@ -340,6 +351,7 @@ public class PrivateSpaceMaintainerTest {
|
|||||||
Flags.FLAG_ALLOW_PRIVATE_PROFILE,
|
Flags.FLAG_ALLOW_PRIVATE_PROFILE,
|
||||||
android.multiuser.Flags.FLAG_SUPPORT_AUTOLOCK_FOR_PRIVATE_SPACE,
|
android.multiuser.Flags.FLAG_SUPPORT_AUTOLOCK_FOR_PRIVATE_SPACE,
|
||||||
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES);
|
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES);
|
||||||
|
assumeTrue(mContext.getSystemService(UserManager.class).canAddPrivateProfile());
|
||||||
final int privateSpaceAutLockValue = 1;
|
final int privateSpaceAutLockValue = 1;
|
||||||
PrivateSpaceMaintainer privateSpaceMaintainer =
|
PrivateSpaceMaintainer privateSpaceMaintainer =
|
||||||
PrivateSpaceMaintainer.getInstance(mContext);
|
PrivateSpaceMaintainer.getInstance(mContext);
|
||||||
|
@@ -20,6 +20,7 @@ import static com.android.settings.privatespace.PrivateSpaceMaintainer.ErrorDele
|
|||||||
|
|
||||||
import static com.google.common.truth.Truth.assertThat;
|
import static com.google.common.truth.Truth.assertThat;
|
||||||
|
|
||||||
|
import static org.junit.Assume.assumeTrue;
|
||||||
import static org.mockito.Mockito.doNothing;
|
import static org.mockito.Mockito.doNothing;
|
||||||
import static org.mockito.Mockito.doReturn;
|
import static org.mockito.Mockito.doReturn;
|
||||||
import static org.mockito.Mockito.spy;
|
import static org.mockito.Mockito.spy;
|
||||||
@@ -28,6 +29,7 @@ import static org.mockito.Mockito.verify;
|
|||||||
import android.app.settings.SettingsEnums;
|
import android.app.settings.SettingsEnums;
|
||||||
import android.content.Context;
|
import android.content.Context;
|
||||||
import android.os.Flags;
|
import android.os.Flags;
|
||||||
|
import android.os.UserManager;
|
||||||
import android.platform.test.flag.junit.SetFlagsRule;
|
import android.platform.test.flag.junit.SetFlagsRule;
|
||||||
|
|
||||||
import androidx.test.annotation.UiThreadTest;
|
import androidx.test.annotation.UiThreadTest;
|
||||||
@@ -90,6 +92,7 @@ public class PrivateSpaceDeletionProgressFragmentTest {
|
|||||||
doNothing().when(spyFragment).showSuccessfulDeletionToast();
|
doNothing().when(spyFragment).showSuccessfulDeletionToast();
|
||||||
mSetFlagsRule.enableFlags(Flags.FLAG_ALLOW_PRIVATE_PROFILE,
|
mSetFlagsRule.enableFlags(Flags.FLAG_ALLOW_PRIVATE_PROFILE,
|
||||||
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES);
|
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES);
|
||||||
|
assumeTrue(mContext.getSystemService(UserManager.class).canAddPrivateProfile());
|
||||||
|
|
||||||
mPrivateSpaceMaintainer.createPrivateSpace();
|
mPrivateSpaceMaintainer.createPrivateSpace();
|
||||||
spyFragment.deletePrivateSpace();
|
spyFragment.deletePrivateSpace();
|
||||||
@@ -104,6 +107,7 @@ public class PrivateSpaceDeletionProgressFragmentTest {
|
|||||||
doNothing().when(spyFragment).showSuccessfulDeletionToast();
|
doNothing().when(spyFragment).showSuccessfulDeletionToast();
|
||||||
mSetFlagsRule.enableFlags(Flags.FLAG_ALLOW_PRIVATE_PROFILE,
|
mSetFlagsRule.enableFlags(Flags.FLAG_ALLOW_PRIVATE_PROFILE,
|
||||||
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES);
|
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES);
|
||||||
|
assumeTrue(mContext.getSystemService(UserManager.class).canAddPrivateProfile());
|
||||||
|
|
||||||
mPrivateSpaceMaintainer.createPrivateSpace();
|
mPrivateSpaceMaintainer.createPrivateSpace();
|
||||||
spyFragment.deletePrivateSpace();
|
spyFragment.deletePrivateSpace();
|
||||||
@@ -128,6 +132,7 @@ public class PrivateSpaceDeletionProgressFragmentTest {
|
|||||||
doNothing().when(spyFragment).showDeletionInternalErrorToast();
|
doNothing().when(spyFragment).showDeletionInternalErrorToast();
|
||||||
mSetFlagsRule.enableFlags(Flags.FLAG_ALLOW_PRIVATE_PROFILE,
|
mSetFlagsRule.enableFlags(Flags.FLAG_ALLOW_PRIVATE_PROFILE,
|
||||||
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES);
|
android.multiuser.Flags.FLAG_ENABLE_PRIVATE_SPACE_FEATURES);
|
||||||
|
assumeTrue(mContext.getSystemService(UserManager.class).canAddPrivateProfile());
|
||||||
|
|
||||||
spyFragment.deletePrivateSpace();
|
spyFragment.deletePrivateSpace();
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user