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

Commit a30002a7 authored by Vania Januar's avatar Vania Januar Committed by Automerger Merge Worker
Browse files

Merge "Do not dismiss USI notification when stylus is bluetooth connected."... 

Merge "Do not dismiss USI notification when stylus is bluetooth connected." into udc-dev am: d4bb4117 am: 5284dd48 am: 78f223b0

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



Change-Id: I8fa51908d192561a05bf0aaa3e77749ac21c72ae
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 5b2ed5dc 78f223b0

packages/SystemUI/src/com/android/systemui/stylus/StylusUsiPowerStartable.kt

+0 −8
Original line number Diff line number Diff line
@@ -50,14 +50,6 @@ constructor( 
        }

    }



    override fun onStylusBluetoothConnected(deviceId: Int, btAddress: String) {

        stylusUsiPowerUi.refresh()

    }



    override fun onStylusBluetoothDisconnected(deviceId: Int, btAddress: String) {

        stylusUsiPowerUi.refresh()

    }



    override fun onStylusUsiBatteryStateChanged(

        deviceId: Int,

        eventTimeMillis: Long,

packages/SystemUI/src/com/android/systemui/stylus/StylusUsiPowerUI.kt

+6 −1
Original line number Diff line number Diff line
@@ -94,12 +94,17 @@ constructor( 
                return@refreshNotification

            }



            // Only hide notification in two cases: battery has been recharged above the

            // threshold, or user has dismissed or clicked notification ("suppression").

            if (suppressed || !batteryBelowThreshold) {

                hideNotification()

            }



            if (!batteryBelowThreshold) {

                // Reset suppression when stylus battery is recharged, so that the next time

                // it reaches a low battery, the notification will show again.

                suppressed = false

            }

            hideNotification()

        }

    }

packages/SystemUI/tests/src/com/android/systemui/stylus/StylusUsiPowerStartableTest.kt

+0 −14
Original line number Diff line number Diff line
@@ -112,20 +112,6 @@ class StylusUsiPowerStartableTest : SysuiTestCase() { 
        verifyZeroInteractions(stylusUsiPowerUi)

    }



    @Test

    fun onStylusBluetoothConnected_refreshesNotification() {

        startable.onStylusBluetoothConnected(STYLUS_DEVICE_ID, "ANY")



        verify(stylusUsiPowerUi, times(1)).refresh()

    }



    @Test

    fun onStylusBluetoothDisconnected_refreshesNotification() {

        startable.onStylusBluetoothDisconnected(STYLUS_DEVICE_ID, "ANY")



        verify(stylusUsiPowerUi, times(1)).refresh()

    }



    @Test

    fun onStylusUsiBatteryStateChanged_batteryPresentValidCapacity_refreshesNotification() {

        val batteryState = FixedCapacityBatteryState(0.1f)

packages/SystemUI/tests/src/com/android/systemui/stylus/StylusUsiPowerUiTest.kt

+4 −11
Original line number Diff line number Diff line
@@ -45,6 +45,7 @@ import org.junit.runner.RunWith 
import org.mockito.ArgumentCaptor

import org.mockito.Captor

import org.mockito.Mock

import org.mockito.Mockito.clearInvocations

import org.mockito.Mockito.doNothing

import org.mockito.Mockito.inOrder

import org.mockito.Mockito.never
@@ -194,22 +195,14 @@ class StylusUsiPowerUiTest : SysuiTestCase() { 
    }



    @Test

    fun refresh_hasConnectedBluetoothStylus_cancelsNotification() {

        whenever(inputManager.inputDeviceIds).thenReturn(intArrayOf(0))



        stylusUsiPowerUi.refresh()



        verify(notificationManager).cancel(R.string.stylus_battery_low_percentage)

    }



    @Test

    fun refresh_hasConnectedBluetoothStylus_existingNotification_cancelsNotification() {

    fun refresh_hasConnectedBluetoothStylus_existingNotification_doesNothing() {

        stylusUsiPowerUi.updateBatteryState(0, FixedCapacityBatteryState(0.1f))

        whenever(inputManager.inputDeviceIds).thenReturn(intArrayOf(0))

        clearInvocations(notificationManager)



        stylusUsiPowerUi.refresh()



        verify(notificationManager).cancel(R.string.stylus_battery_low_percentage)

        verifyNoMoreInteractions(notificationManager)

    }



    @Test