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

Commit 71d61f32 authored by Caitlin Shkuratov's avatar Caitlin Shkuratov
Browse files

[SB Refactor] Convert the WifiViewModel.wifiIcon tests to a 

parameterized test file to make all the possibilities clearer.

Bug: 238425913
Test: WifiViewModelIconParameterizedTest
Change-Id: I5250e903a1b99c2a1ea29b39c3cf3b3e8a5abcfc
parent 7f29e50a

packages/SystemUI/src/com/android/systemui/statusbar/pipeline/wifi/ui/viewmodel/HomeWifiViewModel.kt

+1 −1
Original line number Diff line number Diff line
@@ -28,7 +28,7 @@ import kotlinx.coroutines.flow.StateFlow 
 */

class HomeWifiViewModel(

    statusBarPipelineFlags: StatusBarPipelineFlags,

    wifiIcon: StateFlow<Icon?>,

    wifiIcon: StateFlow<Icon.Resource?>,

    isActivityInViewVisible: Flow<Boolean>,

    isActivityOutViewVisible: Flow<Boolean>,

    isActivityContainerVisible: Flow<Boolean>,

packages/SystemUI/src/com/android/systemui/statusbar/pipeline/wifi/ui/viewmodel/KeyguardWifiViewModel.kt

+1 −1
Original line number Diff line number Diff line
@@ -25,7 +25,7 @@ import kotlinx.coroutines.flow.StateFlow 
/** A view model for the wifi icon shown on keyguard (lockscreen). */

class KeyguardWifiViewModel(

    statusBarPipelineFlags: StatusBarPipelineFlags,

    wifiIcon: StateFlow<Icon?>,

    wifiIcon: StateFlow<Icon.Resource?>,

    isActivityInViewVisible: Flow<Boolean>,

    isActivityOutViewVisible: Flow<Boolean>,

    isActivityContainerVisible: Flow<Boolean>,

packages/SystemUI/src/com/android/systemui/statusbar/pipeline/wifi/ui/viewmodel/LocationBasedWifiViewModel.kt

+1 −1
Original line number Diff line number Diff line
@@ -34,7 +34,7 @@ abstract class LocationBasedWifiViewModel( 
    debugTint: Int,



    /** The wifi icon that should be displayed. Null if we shouldn't display any icon. */

    val wifiIcon: StateFlow<Icon?>,

    val wifiIcon: StateFlow<Icon.Resource?>,



    /** True if the activity in view should be visible. */

    val isActivityInViewVisible: Flow<Boolean>,

packages/SystemUI/src/com/android/systemui/statusbar/pipeline/wifi/ui/viewmodel/QsWifiViewModel.kt

+1 −1
Original line number Diff line number Diff line
@@ -25,7 +25,7 @@ import kotlinx.coroutines.flow.StateFlow 
/** A view model for the wifi icon shown in quick settings (when the shade is pulled down). */

class QsWifiViewModel(

    statusBarPipelineFlags: StatusBarPipelineFlags,

    wifiIcon: StateFlow<Icon?>,

    wifiIcon: StateFlow<Icon.Resource?>,

    isActivityInViewVisible: Flow<Boolean>,

    isActivityOutViewVisible: Flow<Boolean>,

    isActivityContainerVisible: Flow<Boolean>,

packages/SystemUI/src/com/android/systemui/statusbar/pipeline/wifi/ui/viewmodel/WifiViewModel.kt

+1 −1
Original line number Diff line number Diff line
@@ -121,7 +121,7 @@ constructor( 
    }



    /** The wifi icon that should be displayed. Null if we shouldn't display any icon. */

    private val wifiIcon: StateFlow<Icon?> =

    private val wifiIcon: StateFlow<Icon.Resource?> =

        combine(

            interactor.isEnabled,

            interactor.isForceHidden,