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

Commit 1660a27b authored by Jason Monk's avatar Jason Monk
Browse files

Fix flaky test? 

 - Fix testAttachDetach, process more messages so it actually
   does attach and detach
 - Don't inflate Clocks in QSFragmentTest because they are doing
   something weird...

Test: runtest systemui
Change-Id: I05360630ee8d96158b6ab36660f20588ad158a28
Fixes: 37773362
parent 0c215009

packages/SystemUI/tests/src/com/android/systemui/qs/QSFragmentTest.java

+2 −0
Original line number Diff line number Diff line
@@ -32,6 +32,7 @@ import android.testing.AndroidTestingRunner; 


import com.android.systemui.SysuiBaseFragmentTest;

import com.android.systemui.statusbar.phone.StatusBarIconController;

import com.android.systemui.statusbar.policy.Clock;

import com.android.systemui.statusbar.policy.UserSwitcherController;

import android.testing.LayoutInflaterBuilder;

import android.testing.TestableLooper;
@@ -64,6 +65,7 @@ public class QSFragmentTest extends SysuiBaseFragmentTest { 
                                FrameLayout.class)

                        .replace("TextClock", View.class)

                        .replace(CarrierText.class, View.class)

                        .replace(Clock.class, View.class)

                        .build());



        mDependency.injectTestDependency(Dependency.BG_LOOPER,

tests/testables/src/android/testing/BaseFragmentTest.java

+2 −2
Original line number Diff line number Diff line
@@ -161,12 +161,12 @@ public abstract class BaseFragmentTest { 


    protected void attachFragmentToWindow() {

        ViewUtils.attachView(mView);

        TestableLooper.get(this).processMessages(1);

        TestableLooper.get(this).processAllMessages();

    }



    protected void detachFragmentToWindow() {

        ViewUtils.detachView(mView);

        TestableLooper.get(this).processMessages(1);

        TestableLooper.get(this).processAllMessages();

    }



    protected void destroyFragments() {