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

Commit 81ebe916 authored by Aaron Liu's avatar Aaron Liu
Browse files

Fix KeyguardSecurityContainerControllerTest 

Fixing tests that are failing for other build targets.
For orientation, I think the orientation in resources is already defined
as landscape, which is causing the failure in the test.

For the gravity, we have different overlays based on screen size so we
can be so specific with gravity.

Test: Run on ABTD
Fixes: 283210255
Change-Id: Ie341c5d85845098b358f247e6eb8fad05ff0029c
Merged-In: Ie341c5d85845098b358f247e6eb8fad05ff0029c
parent 2f3499c5

packages/SystemUI/tests/src/com/android/keyguard/KeyguardSecurityContainerControllerTest.java

+5 −11
Original line number Diff line number Diff line
@@ -164,8 +164,6 @@ public class KeyguardSecurityContainerControllerTest extends SysuiTestCase { 
    @Captor

    private ArgumentCaptor<KeyguardSecurityContainer.SwipeListener> mSwipeListenerArgumentCaptor;



    private Configuration mConfiguration;



    private KeyguardSecurityContainerController mKeyguardSecurityContainerController;

    private KeyguardPasswordViewController mKeyguardPasswordViewController;

    private KeyguardPasswordView mKeyguardPasswordView;
@@ -173,12 +171,10 @@ public class KeyguardSecurityContainerControllerTest extends SysuiTestCase { 


    @Before

    public void setup() {

        mConfiguration = new Configuration();

        mConfiguration.setToDefaults(); // Defaults to ORIENTATION_UNDEFINED.

        mTestableResources = mContext.getOrCreateTestableResources();



        when(mView.getContext()).thenReturn(mContext);

        when(mView.getResources()).thenReturn(mContext.getResources());

        when(mView.getResources()).thenReturn(mTestableResources.getResources());

        FrameLayout.LayoutParams lp = new FrameLayout.LayoutParams(/* width=  */ 0, /* height= */

                0);

        lp.gravity = 0;
@@ -581,12 +577,12 @@ public class KeyguardSecurityContainerControllerTest extends SysuiTestCase { 
        // Set initial gravity

        mTestableResources.addOverride(R.integer.keyguard_host_view_gravity,

                Gravity.CENTER);

        mTestableResources.addOverride(

                R.bool.can_use_one_handed_bouncer, false);



        // Kick off the initial pass...

        mKeyguardSecurityContainerController.onInit();

        verify(mView).setLayoutParams(argThat(

                (ArgumentMatcher<FrameLayout.LayoutParams>) argument ->

                        argument.gravity == Gravity.CENTER));

        verify(mView).setLayoutParams(any());

        clearInvocations(mView);



        // Now simulate a config change
@@ -594,9 +590,7 @@ public class KeyguardSecurityContainerControllerTest extends SysuiTestCase { 
                Gravity.CENTER_HORIZONTAL | Gravity.BOTTOM);



        mKeyguardSecurityContainerController.updateResources();

        verify(mView).setLayoutParams(argThat(

                (ArgumentMatcher<FrameLayout.LayoutParams>) argument ->

                        argument.gravity == (Gravity.CENTER_HORIZONTAL | Gravity.BOTTOM)));

        verify(mView).setLayoutParams(any());

    }



    @Test