diff options
| author | Chalard Jean <jchalard@google.com> | 2022-02-25 16:12:12 +0900 |
|---|---|---|
| committer | Sooraj Sasindran <sasindran@google.com> | 2022-03-16 17:58:20 +0000 |
| commit | de665266cdc969e203e716601649d11de34f887a (patch) | |
| tree | b8cc3c3bddb22399f05ec44430e63b55fbb7f108 /tests/unit/java/com/android/server/ConnectivityServiceTest.java | |
| parent | 2ccbfe9d8f7bd4e66cd0a18ffe613a909573f640 (diff) | |
Rename setAccessUids to setAllowedUids
Bug: 217725769
Test: ConnectivityServiceTest CtsNetTestCases
Change-Id: Ic8a3f91553d1462b7f54259c467fb90a950bdd59
Merged-In: I8860fbb353eedf5d01e9dc248e4d765046bd562c
Diffstat (limited to 'tests/unit/java/com/android/server/ConnectivityServiceTest.java')
| -rw-r--r-- | tests/unit/java/com/android/server/ConnectivityServiceTest.java | 46 |
1 files changed, 23 insertions, 23 deletions
diff --git a/tests/unit/java/com/android/server/ConnectivityServiceTest.java b/tests/unit/java/com/android/server/ConnectivityServiceTest.java index 0ee9e0bc73..6d802afbb8 100644 --- a/tests/unit/java/com/android/server/ConnectivityServiceTest.java +++ b/tests/unit/java/com/android/server/ConnectivityServiceTest.java @@ -3906,14 +3906,14 @@ public class ConnectivityServiceTest { } @Test - public void testNoAccessUidsInNetworkRequests() throws Exception { + public void testNoAllowedUidsInNetworkRequests() throws Exception { final PendingIntent pendingIntent = PendingIntent.getBroadcast( mContext, 0 /* requestCode */, new Intent("a"), FLAG_IMMUTABLE); final NetworkRequest r = new NetworkRequest.Builder().build(); - final ArraySet<Integer> accessUids = new ArraySet<>(); - accessUids.add(6); - accessUids.add(9); - r.networkCapabilities.setAccessUids(accessUids); + final ArraySet<Integer> allowedUids = new ArraySet<>(); + allowedUids.add(6); + allowedUids.add(9); + r.networkCapabilities.setAllowedUids(allowedUids); final Handler handler = new Handler(ConnectivityThread.getInstanceLooper()); final NetworkCallback cb = new NetworkCallback(); @@ -3928,7 +3928,7 @@ public class ConnectivityServiceTest { // Make sure that resetting the access UIDs to the empty set will allow calling // requestNetwork and registerNetworkCallback. - r.networkCapabilities.setAccessUids(Collections.emptySet()); + r.networkCapabilities.setAllowedUids(Collections.emptySet()); mCm.requestNetwork(r, cb); mCm.unregisterNetworkCallback(cb); mCm.registerNetworkCallback(r, cb); @@ -14688,7 +14688,7 @@ public class ConnectivityServiceTest { } @Test - public void testAccessUids() throws Exception { + public void testAllowedUids() throws Exception { final int preferenceOrder = ConnectivityService.PREFERENCE_ORDER_IRRELEVANT_BECAUSE_NOT_DEFAULT; mServiceContext.setPermission(NETWORK_FACTORY, PERMISSION_GRANTED); @@ -14705,7 +14705,7 @@ public class ConnectivityServiceTest { final NetworkCapabilities nc = new NetworkCapabilities.Builder() .addTransportType(TRANSPORT_TEST) .removeCapability(NET_CAPABILITY_NOT_RESTRICTED) - .setAccessUids(uids) + .setAllowedUids(uids) .build(); final TestNetworkAgentWrapper agent = new TestNetworkAgentWrapper(TRANSPORT_TEST, new LinkProperties(), nc); @@ -14723,10 +14723,10 @@ public class ConnectivityServiceTest { uids.add(300); uids.add(400); - nc.setAccessUids(uids); + nc.setAllowedUids(uids); agent.setNetworkCapabilities(nc, true /* sendToConnectivityService */); if (SdkLevel.isAtLeastT()) { - cb.expectCapabilitiesThat(agent, caps -> caps.getAccessUids().equals(uids)); + cb.expectCapabilitiesThat(agent, caps -> caps.getAllowedUids().equals(uids)); } else { cb.assertNoCallback(); } @@ -14740,10 +14740,10 @@ public class ConnectivityServiceTest { inOrder.verify(mMockNetd, times(1)).networkAddUidRangesParcel(uids300400Parcel); } - nc.setAccessUids(uids); + nc.setAllowedUids(uids); agent.setNetworkCapabilities(nc, true /* sendToConnectivityService */); if (SdkLevel.isAtLeastT()) { - cb.expectCapabilitiesThat(agent, caps -> caps.getAccessUids().equals(uids)); + cb.expectCapabilitiesThat(agent, caps -> caps.getAllowedUids().equals(uids)); inOrder.verify(mMockNetd, times(1)).networkRemoveUidRangesParcel(uids200Parcel); } else { cb.assertNoCallback(); @@ -14751,10 +14751,10 @@ public class ConnectivityServiceTest { uids.clear(); uids.add(600); - nc.setAccessUids(uids); + nc.setAllowedUids(uids); agent.setNetworkCapabilities(nc, true /* sendToConnectivityService */); if (SdkLevel.isAtLeastT()) { - cb.expectCapabilitiesThat(agent, caps -> caps.getAccessUids().equals(uids)); + cb.expectCapabilitiesThat(agent, caps -> caps.getAllowedUids().equals(uids)); } else { cb.assertNoCallback(); } @@ -14768,10 +14768,10 @@ public class ConnectivityServiceTest { } uids.clear(); - nc.setAccessUids(uids); + nc.setAllowedUids(uids); agent.setNetworkCapabilities(nc, true /* sendToConnectivityService */); if (SdkLevel.isAtLeastT()) { - cb.expectCapabilitiesThat(agent, caps -> caps.getAccessUids().isEmpty()); + cb.expectCapabilitiesThat(agent, caps -> caps.getAllowedUids().isEmpty()); inOrder.verify(mMockNetd, times(1)).networkRemoveUidRangesParcel(uids600Parcel); } else { cb.assertNoCallback(); @@ -14782,7 +14782,7 @@ public class ConnectivityServiceTest { } @Test - public void testCbsAccessUids() throws Exception { + public void testCbsAllowedUids() throws Exception { mServiceContext.setPermission(NETWORK_FACTORY, PERMISSION_GRANTED); mServiceContext.setPermission(MANAGE_TEST_NETWORKS, PERMISSION_GRANTED); @@ -14819,29 +14819,29 @@ public class ConnectivityServiceTest { new LinkProperties(), ncb.build()); mCellNetworkAgent.connect(true); cb.expectAvailableThenValidatedCallbacks(mCellNetworkAgent); - ncb.setAccessUids(serviceUidSet); + ncb.setAllowedUids(serviceUidSet); mCellNetworkAgent.setNetworkCapabilities(ncb.build(), true /* sendToCS */); if (SdkLevel.isAtLeastT()) { cb.expectCapabilitiesThat(mCellNetworkAgent, - caps -> caps.getAccessUids().equals(serviceUidSet)); + caps -> caps.getAllowedUids().equals(serviceUidSet)); } else { // S must ignore access UIDs. cb.assertNoCallback(TEST_CALLBACK_TIMEOUT_MS); } // ...but not to some other UID. Rejection sets UIDs to the empty set - ncb.setAccessUids(nonServiceUidSet); + ncb.setAllowedUids(nonServiceUidSet); mCellNetworkAgent.setNetworkCapabilities(ncb.build(), true /* sendToCS */); if (SdkLevel.isAtLeastT()) { cb.expectCapabilitiesThat(mCellNetworkAgent, - caps -> caps.getAccessUids().isEmpty()); + caps -> caps.getAllowedUids().isEmpty()); } else { // S must ignore access UIDs. cb.assertNoCallback(TEST_CALLBACK_TIMEOUT_MS); } // ...and also not to multiple UIDs even including the service UID - ncb.setAccessUids(serviceUidSetPlus); + ncb.setAllowedUids(serviceUidSetPlus); mCellNetworkAgent.setNetworkCapabilities(ncb.build(), true /* sendToCS */); cb.assertNoCallback(TEST_CALLBACK_TIMEOUT_MS); @@ -14864,7 +14864,7 @@ public class ConnectivityServiceTest { new LinkProperties(), ncb.build()); mWiFiNetworkAgent.connect(true); cb.expectAvailableThenValidatedCallbacks(mWiFiNetworkAgent); - ncb.setAccessUids(serviceUidSet); + ncb.setAllowedUids(serviceUidSet); mWiFiNetworkAgent.setNetworkCapabilities(ncb.build(), true /* sendToCS */); cb.assertNoCallback(TEST_CALLBACK_TIMEOUT_MS); mCm.unregisterNetworkCallback(cb); |
