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

Commit fc71616a authored by Treehugger Robot's avatar Treehugger Robot Committed by Automerger Merge Worker
Browse files

Merge "Fix CaptivePortalTest before R" am: 41dd05fd am: 547c982b

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

MUST ONLY BE SUBMITTED BY AUTOMERGER

Change-Id: Ib5bf1fc2b06fb5339ffd9b43d0eda893a4bbc4bd
parents 03297aac 547c982b
Loading
Loading
Loading
Loading
+21 −4
Original line number Diff line number Diff line
@@ -25,6 +25,7 @@ import androidx.test.filters.SmallTest;
import androidx.test.runner.AndroidJUnit4;

import com.android.testutils.DevSdkIgnoreRule;
import com.android.testutils.DevSdkIgnoreRule.IgnoreAfter;
import com.android.testutils.DevSdkIgnoreRule.IgnoreUpTo;

import org.junit.Rule;
@@ -53,6 +54,12 @@ public class CaptivePortalTest {
        public void appRequest(final int request) throws RemoteException {
            mCode = request;
        }

        // This is only @Override on R-
        public void logEvent(int eventId, String packageName) throws RemoteException {
            mCode = eventId;
            mPackageName = packageName;
        }
    }

    private interface TestFunctor {
@@ -91,14 +98,24 @@ public class CaptivePortalTest {
        assertEquals(result.mCode, CaptivePortal.APP_REQUEST_REEVALUATION_REQUIRED);
    }

    /**
     * Test testLogEvent is expected to do nothing but shouldn't crash, because the API logEvent
     * has been deprecated.
     */
    @IgnoreUpTo(Build.VERSION_CODES.R)
    @Test
    public void testLogEvent() {
        /**
        * From S testLogEvent is expected to do nothing but shouldn't crash (the API
        * logEvent has been deprecated).
        */
        final MyCaptivePortalImpl result = runCaptivePortalTest(c -> c.logEvent(
                0,
                TEST_PACKAGE_NAME));
    }

    @IgnoreAfter(Build.VERSION_CODES.R)
    @Test
    public void testLogEvent_UntilR() {
        final MyCaptivePortalImpl result = runCaptivePortalTest(c -> c.logEvent(
                42, TEST_PACKAGE_NAME));
        assertEquals(result.mCode, 42);
        assertEquals(result.mPackageName, TEST_PACKAGE_NAME);
    }
}