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

Commit 6bd6a90e authored by Hawkwood Glazier's avatar Hawkwood Glazier Committed by Automerger Merge Worker
Browse files

Merge "Ensure state is set correctly after animateAppearOnLockscreen is... 

Merge "Ensure state is set correctly after animateAppearOnLockscreen is called" into tm-qpr-dev am: 66f5377b

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



Change-Id: Ib3ca05d14959bd43f111637b51831d0689b629ac
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 56294c35 66f5377b

packages/SystemUI/shared/src/com/android/systemui/shared/clocks/AnimatableClockView.kt

+4 −4
Original line number Diff line number Diff line
@@ -23,6 +23,7 @@ import android.annotation.SuppressLint 
import android.app.compat.ChangeIdStateCache.invalidate

import android.content.Context

import android.graphics.Canvas

import android.text.Layout

import android.text.TextUtils

import android.text.format.DateFormat

import android.util.AttributeSet
@@ -78,6 +79,8 @@ class AnimatableClockView @JvmOverloads constructor( 
    private var textAnimator: TextAnimator? = null

    private var onTextAnimatorInitialized: Runnable? = null



    @VisibleForTesting var textAnimatorFactory: (Layout, () -> Unit) -> TextAnimator =

        { layout, invalidateCb -> TextAnimator(layout, invalidateCb) }

    @VisibleForTesting var isAnimationEnabled: Boolean = true

    @VisibleForTesting var timeOverrideInMillis: Long? = null
@@ -174,7 +177,7 @@ class AnimatableClockView @JvmOverloads constructor( 
        super.onMeasure(widthMeasureSpec, heightMeasureSpec)

        val animator = textAnimator

        if (animator == null) {

            textAnimator = TextAnimator(layout) { invalidate() }

            textAnimator = textAnimatorFactory(layout, ::invalidate)

            onTextAnimatorInitialized?.run()

            onTextAnimatorInitialized = null

        } else {
@@ -219,9 +222,6 @@ class AnimatableClockView @JvmOverloads constructor( 
    }



    fun animateAppearOnLockscreen() {

        if (isAnimationEnabled && textAnimator == null) {

            return

        }

        setTextStyle(

            weight = dozingWeight,

            textSize = -1f,

packages/SystemUI/tests/src/com/android/systemui/shared/clocks/AnimatableClockViewTest.kt

0 → 100644
+74 −0
Original line number Diff line number Diff line 
/*

 * Copyright (C) 2022 The Android Open Source Project

 *

 * Licensed under the Apache License, Version 2.0 (the "License");

 * you may not use this file except in compliance with the License.

 * You may obtain a copy of the License at

 *

 *      http://www.apache.org/licenses/LICENSE-2.0

 *

 * Unless required by applicable law or agreed to in writing, software

 * distributed under the License is distributed on an "AS IS" BASIS,

 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.

 * See the License for the specific language governing permissions and

 * limitations under the License

 */



package com.android.systemui.shared.clocks



import android.testing.AndroidTestingRunner

import android.view.LayoutInflater

import androidx.test.filters.SmallTest

import com.android.systemui.R

import com.android.systemui.SysuiTestCase

import com.android.systemui.animation.TextAnimator

import org.junit.Before

import org.junit.Rule

import org.junit.Test

import org.junit.runner.RunWith

import org.mockito.Mock

import org.mockito.Mockito.times

import org.mockito.Mockito.verify

import org.mockito.Mockito.verifyNoMoreInteractions

import org.mockito.junit.MockitoJUnit



@RunWith(AndroidTestingRunner::class)

@SmallTest

class AnimatableClockViewTest : SysuiTestCase() {



    @JvmField @Rule val mockito = MockitoJUnit.rule()



    @Mock private lateinit var mockTextAnimator: TextAnimator

    private lateinit var clockView: AnimatableClockView



    @Before

    fun setUp() {

        val layoutInflater = LayoutInflater.from(context)

        clockView =

            layoutInflater.inflate(R.layout.clock_default_small, null) as AnimatableClockView

        clockView.textAnimatorFactory = { _, _ -> mockTextAnimator }

    }



    @Test

    fun validateColorAnimationRunsBeforeMeasure() {

        clockView.setColors(100, 200)

        clockView.animateAppearOnLockscreen()

        clockView.measure(50, 50)



        verify(mockTextAnimator).glyphFilter = null

        verify(mockTextAnimator).setTextStyle(300, -1.0f, 200, false, 350L, null, 0L, null)

        verifyNoMoreInteractions(mockTextAnimator)

    }



    @Test

    fun validateColorAnimationRunsAfterMeasure() {

        clockView.setColors(100, 200)

        clockView.measure(50, 50)

        clockView.animateAppearOnLockscreen()



        verify(mockTextAnimator, times(2)).glyphFilter = null

        verify(mockTextAnimator).setTextStyle(100, -1.0f, 200, false, 0L, null, 0L, null)

        verify(mockTextAnimator).setTextStyle(300, -1.0f, 200, true, 350L, null, 0L, null)

        verifyNoMoreInteractions(mockTextAnimator)

    }

}