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

Commit 85d3a826 authored by Roland Levillain's avatar Roland Levillain Committed by android-build-merger
Browse files

Merge "Revert "ConnectivityServiceTest: more informative assert failures"" am:... 

Merge "Revert "ConnectivityServiceTest: more informative assert failures"" am: d10790c9 am: 8f682322 am: 3b2b9b62
am: b566fa91

Change-Id: I914c3ad6074e99e313c65572e3f38144a7f8a0d9
parents 8fd0bf1e b566fa91

tests/net/java/com/android/server/ConnectivityServiceTest.java

+11 −22
Original line number Diff line number Diff line
@@ -891,7 +891,7 @@ public class ConnectivityServiceTest extends AndroidTestCase { 
        assertNull(mCm.getActiveNetwork());

        assertNull(mCm.getActiveNetworkForUid(Process.myUid()));

        // Test getAllNetworks()

        assertEmpty(mCm.getAllNetworks());

        assertEquals(0, mCm.getAllNetworks().length);

    }



    /**
@@ -932,7 +932,7 @@ public class ConnectivityServiceTest extends AndroidTestCase { 
        mCellNetworkAgent.connect(true);

        waitFor(cv);

        verifyActiveNetwork(TRANSPORT_CELLULAR);

        assertLength(2, mCm.getAllNetworks());

        assertEquals(2, mCm.getAllNetworks().length);

        assertTrue(mCm.getAllNetworks()[0].equals(mCm.getActiveNetwork()) ||

                mCm.getAllNetworks()[1].equals(mCm.getActiveNetwork()));

        assertTrue(mCm.getAllNetworks()[0].equals(mWiFiNetworkAgent.getNetwork()) ||
@@ -942,7 +942,7 @@ public class ConnectivityServiceTest extends AndroidTestCase { 
        mWiFiNetworkAgent.connect(true);

        waitFor(cv);

        verifyActiveNetwork(TRANSPORT_WIFI);

        assertLength(2, mCm.getAllNetworks());

        assertEquals(2, mCm.getAllNetworks().length);

        assertTrue(mCm.getAllNetworks()[0].equals(mCm.getActiveNetwork()) ||

                mCm.getAllNetworks()[1].equals(mCm.getActiveNetwork()));

        assertTrue(mCm.getAllNetworks()[0].equals(mCellNetworkAgent.getNetwork()) ||
@@ -950,9 +950,9 @@ public class ConnectivityServiceTest extends AndroidTestCase { 
        // Test cellular linger timeout.

        waitFor(mCellNetworkAgent.getDisconnectedCV());

        waitForIdle();

        assertLength(1, mCm.getAllNetworks());

        assertEquals(1, mCm.getAllNetworks().length);

        verifyActiveNetwork(TRANSPORT_WIFI);

        assertLength(1, mCm.getAllNetworks());

        assertEquals(1, mCm.getAllNetworks().length);

        assertEquals(mCm.getAllNetworks()[0], mCm.getActiveNetwork());

        // Test WiFi disconnect.

        cv = waitForConnectivityBroadcasts(1);
@@ -1892,7 +1892,7 @@ public class ConnectivityServiceTest extends AndroidTestCase { 
        mCellNetworkAgent.connectWithoutInternet();

        waitFor(cv);

        waitForIdle();

        assertEmpty(mCm.getAllNetworks());

        assertEquals(0, mCm.getAllNetworks().length);

        verifyNoNetwork();



        // Test bringing up validated WiFi.
@@ -2566,7 +2566,7 @@ public class ConnectivityServiceTest extends AndroidTestCase { 
        assertTrue(testFactory.getMyStartRequested());



        // Bring up cell data and check that the factory stops looking.

        assertLength(1, mCm.getAllNetworks());

        assertEquals(1, mCm.getAllNetworks().length);

        mCellNetworkAgent = new MockNetworkAgent(TRANSPORT_CELLULAR);

        testFactory.expectAddRequests(2);  // Because the cell request changes score twice.

        mCellNetworkAgent.connect(true);
@@ -2577,7 +2577,7 @@ public class ConnectivityServiceTest extends AndroidTestCase { 
        // Check that cell data stays up.

        waitForIdle();

        verifyActiveNetwork(TRANSPORT_WIFI);

        assertLength(2, mCm.getAllNetworks());

        assertEquals(2, mCm.getAllNetworks().length);



        // Turn off mobile data always on and expect the request to disappear...

        testFactory.expectRemoveRequests(1);
@@ -2586,7 +2586,7 @@ public class ConnectivityServiceTest extends AndroidTestCase { 


        // ...  and cell data to be torn down.

        cellNetworkCallback.expectCallback(CallbackState.LOST, mCellNetworkAgent);

        assertLength(1, mCm.getAllNetworks());

        assertEquals(1, mCm.getAllNetworks().length);



        testFactory.unregister();

        mCm.unregisterNetworkCallback(cellNetworkCallback);
@@ -3291,7 +3291,7 @@ public class ConnectivityServiceTest extends AndroidTestCase { 
        assertNull(mCm.getActiveNetworkInfo());



        Network[] allNetworks = mCm.getAllNetworks();

        assertLength(1, allNetworks);

        assertEquals(1, allNetworks.length);

        Network network = allNetworks[0];

        NetworkCapabilities capabilities = mCm.getNetworkCapabilities(network);

        assertTrue(capabilities.hasTransport(TRANSPORT_LOWPAN));
@@ -3309,7 +3309,7 @@ public class ConnectivityServiceTest extends AndroidTestCase { 
        assertNull(mCm.getActiveNetwork());

        // TODO: getAllNetworkInfo is dirty and returns a non-empty array rght from the start

        // of this test. Fix it and uncomment the assert below.

        //assertEmpty(mCm.getAllNetworkInfo());

        //assertEquals(0, mCm.getAllNetworkInfo().length);



        // Disconnect lowpan.

        lowpanNetwork.disconnect();
@@ -3355,17 +3355,6 @@ public class ConnectivityServiceTest extends AndroidTestCase { 
        }

    }



    private static <T> void assertEmpty(T[] ts) {

        int length = ts.length;

        assertEquals("expected empty array, but length was " + length, 0, length);

    }



    private static <T> void assertLength(int expected, T[] got) {

        int length = got.length;

        assertEquals(String.format("expected array of length %s, but length was %s for %s",

                expected, length, Arrays.toString(got)), expected, length);

    }



    /* test utilities */

    // TODO: eliminate all usages of sleepFor and replace by proper timeouts/waitForIdle.

    static private void sleepFor(int ms) {