Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Fix no guest visible while on multi user switcher" into tm-qpr-dev am: 95799ff292

Browse Source 
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/20581318

Change-Id: I23cfaca65fa0534e11f055d66d8affa95d66ce67
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Brad Hinegardner
2022-11-30 17:29:11 +00:00
committed by Automerger Merge Worker
parent 4c1289ff7a 95799ff292
commit 95b8f5b764
2 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
packages/SystemUI/src/com/android/systemui/user/domain/interactor/UserInteractor.kt
View File

@@ -114,9 +114,9 @@ constructor(
private val callbackMutex = Mutex()
private val callbacks = mutableSetOf<UserCallback>()
private val userInfos =
combine(repository.userSwitcherSettings, repository.userInfos) { settings, userInfos ->
userInfos.filter { !it.isGuest || canCreateGuestUser(settings) }.filter { it.isFull }
private val userInfos: Flow<List<UserInfo>> =
repository.userInfos.map { userInfos ->
userInfos.filter { it.isFull }
}
/** List of current on-device users to select from. */

6
packages/SystemUI/tests/src/com/android/systemui/user/domain/interactor/UserInteractorTest.kt
View File

@@ -761,7 +761,7 @@ class UserInteractorTest : SysuiTestCase() {
}
@Test
fun `users - secondary user - no guest user`() =
fun `users - secondary user - guest user can be switched to`() =
runBlocking(IMMEDIATE) {
val userInfos = createUserInfos(count = 3, includeGuest = true)
userRepository.setUserInfos(userInfos)
@@ -770,8 +770,8 @@ class UserInteractorTest : SysuiTestCase() {
var res: List<UserModel>? = null
val job = underTest.users.onEach { res = it }.launchIn(this)
assertThat(res?.size == 2).isTrue()
assertThat(res?.find { it.isGuest }).isNull()
assertThat(res?.size == 3).isTrue()
assertThat(res?.find { it.isGuest }).isNotNull()
job.cancel()
}