Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit bffbb141 authored by bkchoi's avatar bkchoi
Browse files

Make getAliveUsers() and getUsers() as @TestApi. 

To replace deprecated UserManager#getUsers(boolean, boolean, boolean),
which is used in CTS, we need to make the alternative methods Test API.

Bug: 253528462

Test: atest UserManagerTest BackgroundActivityLaunchTest

Change-Id: I07b356d210661fe1e0231a7d0aa6b061af032f04
parent 530f462f

core/api/test-current.txt

+2 −0
Original line number Diff line number Diff line
@@ -2374,10 +2374,12 @@ package android.os { 
    method @Nullable @RequiresPermission(anyOf={android.Manifest.permission.MANAGE_USERS, android.Manifest.permission.CREATE_USERS}) public android.content.pm.UserInfo createProfileForUser(@Nullable String, @NonNull String, int, int, @Nullable String[]);

    method @Nullable @RequiresPermission(anyOf={android.Manifest.permission.MANAGE_USERS, android.Manifest.permission.CREATE_USERS}) public android.content.pm.UserInfo createRestrictedProfile(@Nullable String);

    method @Nullable @RequiresPermission(anyOf={android.Manifest.permission.MANAGE_USERS, android.Manifest.permission.CREATE_USERS}) public android.content.pm.UserInfo createUser(@Nullable String, @NonNull String, int);

    method @NonNull @RequiresPermission(anyOf={android.Manifest.permission.MANAGE_USERS, android.Manifest.permission.CREATE_USERS}) public java.util.List<android.content.pm.UserInfo> getAliveUsers();

    method @NonNull @RequiresPermission(anyOf={android.Manifest.permission.MANAGE_USERS, android.Manifest.permission.CREATE_USERS}) public android.os.UserHandle getBootUser();

    method public int getMainDisplayIdAssignedToUser();

    method @Nullable @RequiresPermission(anyOf={android.Manifest.permission.MANAGE_USERS, android.Manifest.permission.CREATE_USERS}) public java.util.Set<java.lang.String> getPreInstallableSystemPackages(@NonNull String);

    method @NonNull @RequiresPermission(anyOf={android.Manifest.permission.MANAGE_USERS, android.Manifest.permission.CREATE_USERS, android.Manifest.permission.QUERY_USERS}) public String getUserType();

    method @NonNull @RequiresPermission(anyOf={android.Manifest.permission.MANAGE_USERS, android.Manifest.permission.CREATE_USERS}) public java.util.List<android.content.pm.UserInfo> getUsers();

    method @Deprecated @NonNull @RequiresPermission(anyOf={android.Manifest.permission.MANAGE_USERS, android.Manifest.permission.CREATE_USERS}) public java.util.List<android.content.pm.UserInfo> getUsers(boolean, boolean, boolean);

    method @RequiresPermission(anyOf={android.Manifest.permission.MANAGE_USERS, android.Manifest.permission.CREATE_USERS}) public boolean hasBaseUserRestriction(@NonNull String, @NonNull android.os.UserHandle);

    method @RequiresPermission(anyOf={android.Manifest.permission.MANAGE_USERS, android.Manifest.permission.CREATE_USERS}) public boolean isUserTypeEnabled(@NonNull String);

core/java/android/os/UserManager.java

+4 −3
Original line number Diff line number Diff line
@@ -4257,7 +4257,8 @@ public class UserManager { 
            android.Manifest.permission.MANAGE_USERS,

            android.Manifest.permission.CREATE_USERS

    })

    public List<UserInfo> getUsers() {

    @TestApi

    public @NonNull List<UserInfo> getUsers() {

        return getUsers(/*excludePartial= */ true, /* excludeDying= */ false,

                /* excludePreCreated= */ true);

    }
@@ -4278,6 +4279,7 @@ public class UserManager { 
            android.Manifest.permission.MANAGE_USERS,

            android.Manifest.permission.CREATE_USERS

    })

    @TestApi

    public @NonNull List<UserInfo> getAliveUsers() {

        return getUsers(/*excludePartial= */ true, /* excludeDying= */ true,

                /* excludePreCreated= */ true);
@@ -4304,8 +4306,7 @@ public class UserManager { 
     * Returns information for all users on this device, based on the filtering parameters.

     *

     * @deprecated Pre-created users are deprecated and no longer supported.

     *             Use {@link #getUsers()}, {@link #getUsers(boolean)}, or {@link #getAliveUsers()}

     *             instead.

     *             Use {@link #getUsers()}, or {@link #getAliveUsers()} instead.

     * @hide

     */

    @Deprecated