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

Commit d528756a authored by András Kurucz's avatar András Kurucz
Browse files

Re-enable and fix failing tests in NotificationStackScrollLayoutTest

Fixing some minor issues with tests.
Deleting the testAppearFractionCalculation(), because its content is no
longer relevant: we just return a value from
AmbientState#getAppearFractionCalculation().

Fixes: b/284136971
Test: atest NotificationStackScrollLayoutTest
Change-Id: Ie215aba0eb764c60e05fcf1baf96cc38b262dd25
parent 878b61b0
Loading
Loading
Loading
Loading
+2 −17
Original line number Diff line number Diff line
@@ -88,7 +88,6 @@ import com.android.systemui.statusbar.phone.UnlockedScreenOffAnimationController

import org.junit.Assert;
import org.junit.Before;
import org.junit.Ignore;
import org.junit.Rule;
import org.junit.Test;
import org.junit.runner.RunWith;
@@ -102,7 +101,6 @@ import java.util.ArrayList;
/**
 * Tests for {@link NotificationStackScrollLayout}.
 */
@Ignore("b/255552856")
@SmallTest
@RunWith(AndroidTestingRunner.class)
@TestableLooper.RunWithLooper
@@ -373,20 +371,6 @@ public class NotificationStackScrollLayoutTest extends SysuiTestCase {
        mStackScroller.setExpandedHeight(expectedHeight);
    }

    @Test
    public void testAppearFractionCalculation() {
        // appear start position
        when(mNotificationShelf.getIntrinsicHeight()).thenReturn(100);
        // because it's the same as shelf height, appear start position equals shelf height
        mStackScroller.mStatusBarHeight = 100;
        // appear end position
        when(mEmptyShadeView.getHeight()).thenReturn(200);

        assertEquals(0f, mStackScroller.calculateAppearFraction(100));
        assertEquals(1f, mStackScroller.calculateAppearFraction(200));
        assertEquals(0.5f, mStackScroller.calculateAppearFraction(150));
    }

    @Test
    public void testAppearFractionCalculationIsNotNegativeWhenShelfBecomesSmaller() {
        // this situation might occur if status bar height is defined in pixels while shelf height
@@ -592,6 +576,7 @@ public class NotificationStackScrollLayoutTest extends SysuiTestCase {

    @Test
    public void testReInflatesFooterViews() {
        when(mEmptyShadeView.getTextResource()).thenReturn(R.string.empty_shade_text);
        clearInvocations(mStackScroller);
        mStackScroller.reinflateViews();
        verify(mStackScroller).setFooterView(any());
@@ -874,7 +859,7 @@ public class NotificationStackScrollLayoutTest extends SysuiTestCase {
        mStackScroller.setHasFilteredOutSeenNotifications(true);
        mStackScroller.updateEmptyShadeView(true, false);

        verify(mEmptyShadeView).setFooterText(not(0));
        verify(mEmptyShadeView).setFooterText(not(eq(0)));
    }

    @Test