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

Commit a1d19764 authored by lucaslin's avatar lucaslin
Browse files

Fix NetworkCapabilitiesTest fail on Q device 

The getSSID() has changed to getSsid() in Android R, adding
isAtLeastR() to prevent NetworkCapabilitiesTest fail on Android
Q.

Bug: 151322799
Test: Run "atest CtsNetTestCasesLatestSdk:NetworkCapabilitiesTest"
      on Android Q & R device.
Change-Id: I602ae32dae1ad29fe3293c541fa6d2cef01b81d3
parent ac423f3a

tests/net/common/java/android/net/NetworkCapabilitiesTest.java

+9 −3
Original line number Diff line number Diff line
@@ -463,7 +463,9 @@ public class NetworkCapabilitiesTest { 


        nc1.setSSID(TEST_SSID);

        nc2.combineCapabilities(nc1);

        if (isAtLeastR()) {

            assertTrue(TEST_SSID.equals(nc2.getSsid()));

        }



        // Because they now have the same SSID, the following call should not throw

        nc2.combineCapabilities(nc1);
@@ -601,12 +603,16 @@ public class NetworkCapabilitiesTest { 
        // from nc2.

        assertFalse(nc2.hasCapability(NET_CAPABILITY_NOT_ROAMING));

        assertTrue(nc2.hasUnwantedCapability(NET_CAPABILITY_NOT_ROAMING));

        if (isAtLeastR()) {

            assertTrue(TEST_SSID.equals(nc2.getSsid()));

        }



        nc1.setSSID(DIFFERENT_TEST_SSID);

        nc2.set(nc1);

        assertEquals(nc1, nc2);

        if (isAtLeastR()) {

            assertTrue(DIFFERENT_TEST_SSID.equals(nc2.getSsid()));

        }



        nc1.setUids(uidRange(10, 13));

        nc2.set(nc1);  // Overwrites, as opposed to combineCapabilities