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

Commit b44a4044 authored by Jeff Chang's avatar Jeff Chang
Browse files

Fix splitscreen test: SnapToDismiss 

The test cases verify onSnappedToDismiss callbacks properly invoked
when the snap target indicates dismissing split. The callback is
invoked more than once since the start position is the same as the
end position when flingDividePosition is invoked. This CL updates
the current position to make sure the callback is invoked once.

Bug: 257348773
Test: atest SplitLayoutTests
Change-Id: I5742d3db1ab6d5c80c4b4ffa1a9465655c73ab11
parent c6e40030

libs/WindowManager/Shell/tests/unittest/src/com/android/wm/shell/common/split/SplitLayoutTests.java

+2 −2
Original line number Diff line number Diff line
@@ -140,7 +140,7 @@ public class SplitLayoutTests extends ShellTestCase { 
        DividerSnapAlgorithm.SnapTarget snapTarget = getSnapTarget(0 /* position */,

                DividerSnapAlgorithm.SnapTarget.FLAG_DISMISS_START);



        mSplitLayout.snapToTarget(0 /* currentPosition */, snapTarget);

        mSplitLayout.snapToTarget(mSplitLayout.getDividePosition(), snapTarget);

        waitDividerFlingFinished();

        verify(mSplitLayoutHandler).onSnappedToDismiss(eq(false), anyInt());

    }
@@ -152,7 +152,7 @@ public class SplitLayoutTests extends ShellTestCase { 
        DividerSnapAlgorithm.SnapTarget snapTarget = getSnapTarget(0 /* position */,

                DividerSnapAlgorithm.SnapTarget.FLAG_DISMISS_END);



        mSplitLayout.snapToTarget(0 /* currentPosition */, snapTarget);

        mSplitLayout.snapToTarget(mSplitLayout.getDividePosition(), snapTarget);

        waitDividerFlingFinished();

        verify(mSplitLayoutHandler).onSnappedToDismiss(eq(true), anyInt());

    }