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

Commit 4da81dc7 authored by Sarah Chin's avatar Sarah Chin
Browse files

Fix flaky override unmetered/congested test

The assert statement might be checking for state before the override
message is sent, so wait 100ms before checking.

Test: atest DataConnectionTest
Fix: 181793539
Change-Id: I3ddf3f0fed41d4af3671e4d09de3e44273340a03
parent ac591839
Loading
Loading
Loading
Loading
+4 −0
Original line number Diff line number Diff line
@@ -673,12 +673,14 @@ public class DataConnectionTest extends TelephonyTest {
        assertTrue(getNetworkCapabilities().hasCapability(NET_CAPABILITY_NOT_CONGESTED));

        mDc.onMeterednessChanged(true);
        waitForMs(100);

        assertFalse(getNetworkCapabilities().hasCapability(NET_CAPABILITY_NOT_METERED));
        assertTrue(getNetworkCapabilities().hasCapability(NET_CAPABILITY_TEMPORARILY_NOT_METERED));
        assertTrue(getNetworkCapabilities().hasCapability(NET_CAPABILITY_NOT_CONGESTED));

        mDc.onMeterednessChanged(false);
        waitForMs(100);

        assertFalse(getNetworkCapabilities().hasCapability(NET_CAPABILITY_NOT_METERED));
        assertFalse(getNetworkCapabilities().hasCapability(NET_CAPABILITY_TEMPORARILY_NOT_METERED));
@@ -697,12 +699,14 @@ public class DataConnectionTest extends TelephonyTest {
        assertTrue(getNetworkCapabilities().hasCapability(NET_CAPABILITY_NOT_CONGESTED));

        mDc.onCongestednessChanged(true);
        waitForMs(100);

        assertFalse(getNetworkCapabilities().hasCapability(NET_CAPABILITY_NOT_METERED));
        assertFalse(getNetworkCapabilities().hasCapability(NET_CAPABILITY_TEMPORARILY_NOT_METERED));
        assertFalse(getNetworkCapabilities().hasCapability(NET_CAPABILITY_NOT_CONGESTED));

        mDc.onCongestednessChanged(false);
        waitForMs(100);

        assertFalse(getNetworkCapabilities().hasCapability(NET_CAPABILITY_NOT_METERED));
        assertFalse(getNetworkCapabilities().hasCapability(NET_CAPABILITY_TEMPORARILY_NOT_METERED));