Merge "Adding a check if the user exist before making a state mutation"

This commit is contained in:
Jackson Michael Moreira Miranda
2022-10-19 13:27:01 +00:00
committed by Android (Google) Code Review
2 changed files with 10 additions and 0 deletions

View File

@@ -2633,6 +2633,9 @@ public class UserManagerService extends IUserManager.Stub {
/** @return a specific user restriction that's in effect currently. */
@Override
public boolean hasUserRestriction(String restrictionKey, @UserIdInt int userId) {
if (!userExists(userId)) {
return false;
}
checkManageOrInteractPermissionIfCallerInOtherProfileGroup(userId, "hasUserRestriction");
return mLocalService.hasUserRestriction(restrictionKey, userId);
}

View File

@@ -175,6 +175,13 @@ public class UserManagerServiceTest {
assertThat(mUserManagerService.hasUserRestriction(DISALLOW_USER_SWITCH, userId)).isFalse();
}
@Test
public void testHasUserRestriction_NonExistentUserReturnsFalse() {
int nonExistentUserId = UserHandle.USER_NULL;
assertThat(mUserManagerService.hasUserRestriction(DISALLOW_USER_SWITCH, nonExistentUserId))
.isFalse();
}
@Test
public void testSetUserRestrictionWithIncorrectID() throws Exception {
int incorrectId = 1;