[VCN19] Rename get/setSubIds to get/setSubscriptionIds
Test: atest FrameworksNetTests FrameworksVcnTests Fix: 185215036 Merged-In: I9d90df5fc13b36d2cdc4920b456dcc87fcd2b3a7 Change-Id: I9d90df5fc13b36d2cdc4920b456dcc87fcd2b3a7 (cherry-picked from ag/14198665)
This commit is contained in:
@@ -311,7 +311,7 @@ public class NetworkCapabilitiesTest {
|
||||
.addCapability(NET_CAPABILITY_EIMS)
|
||||
.addCapability(NET_CAPABILITY_NOT_METERED);
|
||||
if (isAtLeastS()) {
|
||||
netCap.setSubIds(Set.of(TEST_SUBID1, TEST_SUBID2));
|
||||
netCap.setSubscriptionIds(Set.of(TEST_SUBID1, TEST_SUBID2));
|
||||
netCap.setUids(uids);
|
||||
}
|
||||
if (isAtLeastR()) {
|
||||
@@ -641,16 +641,16 @@ public class NetworkCapabilitiesTest {
|
||||
assertTrue(nc2.appliesToUid(22));
|
||||
|
||||
// Verify the subscription id list can be combined only when they are equal.
|
||||
nc1.setSubIds(Set.of(TEST_SUBID1, TEST_SUBID2));
|
||||
nc2.setSubIds(Set.of(TEST_SUBID2));
|
||||
nc1.setSubscriptionIds(Set.of(TEST_SUBID1, TEST_SUBID2));
|
||||
nc2.setSubscriptionIds(Set.of(TEST_SUBID2));
|
||||
assertThrows(IllegalStateException.class, () -> nc2.combineCapabilities(nc1));
|
||||
|
||||
nc2.setSubIds(Set.of());
|
||||
nc2.setSubscriptionIds(Set.of());
|
||||
assertThrows(IllegalStateException.class, () -> nc2.combineCapabilities(nc1));
|
||||
|
||||
nc2.setSubIds(Set.of(TEST_SUBID2, TEST_SUBID1));
|
||||
nc2.setSubscriptionIds(Set.of(TEST_SUBID2, TEST_SUBID1));
|
||||
nc2.combineCapabilities(nc1);
|
||||
assertEquals(Set.of(TEST_SUBID2, TEST_SUBID1), nc2.getSubIds());
|
||||
assertEquals(Set.of(TEST_SUBID2, TEST_SUBID1), nc2.getSubscriptionIds());
|
||||
}
|
||||
}
|
||||
|
||||
@@ -805,20 +805,20 @@ public class NetworkCapabilitiesTest {
|
||||
assertEquals(nc1, nc2);
|
||||
|
||||
if (isAtLeastS()) {
|
||||
assertThrows(NullPointerException.class, () -> nc1.setSubIds(null));
|
||||
nc1.setSubIds(Set.of());
|
||||
assertThrows(NullPointerException.class, () -> nc1.setSubscriptionIds(null));
|
||||
nc1.setSubscriptionIds(Set.of());
|
||||
nc2.set(nc1);
|
||||
assertEquals(nc1, nc2);
|
||||
|
||||
nc1.setSubIds(Set.of(TEST_SUBID1));
|
||||
nc1.setSubscriptionIds(Set.of(TEST_SUBID1));
|
||||
nc2.set(nc1);
|
||||
assertEquals(nc1, nc2);
|
||||
|
||||
nc2.setSubIds(Set.of(TEST_SUBID2, TEST_SUBID1));
|
||||
nc2.setSubscriptionIds(Set.of(TEST_SUBID2, TEST_SUBID1));
|
||||
nc2.set(nc1);
|
||||
assertEquals(nc1, nc2);
|
||||
|
||||
nc2.setSubIds(Set.of(TEST_SUBID3, TEST_SUBID2));
|
||||
nc2.setSubscriptionIds(Set.of(TEST_SUBID3, TEST_SUBID2));
|
||||
assertNotEquals(nc1, nc2);
|
||||
}
|
||||
}
|
||||
@@ -907,8 +907,8 @@ public class NetworkCapabilitiesTest {
|
||||
// satisfy these requests.
|
||||
final NetworkCapabilities nc = new NetworkCapabilities.Builder()
|
||||
.addCapability(NET_CAPABILITY_NOT_VCN_MANAGED)
|
||||
.setSubIds(new ArraySet<>(subIds)).build();
|
||||
assertEquals(new ArraySet<>(subIds), nc.getSubIds());
|
||||
.setSubscriptionIds(new ArraySet<>(subIds)).build();
|
||||
assertEquals(new ArraySet<>(subIds), nc.getSubscriptionIds());
|
||||
return nc;
|
||||
}
|
||||
|
||||
@@ -920,11 +920,11 @@ public class NetworkCapabilitiesTest {
|
||||
final NetworkCapabilities ncWithoutRequestedIds = capsWithSubIds(TEST_SUBID3);
|
||||
|
||||
final NetworkRequest requestWithoutId = new NetworkRequest.Builder().build();
|
||||
assertEmpty(requestWithoutId.networkCapabilities.getSubIds());
|
||||
assertEmpty(requestWithoutId.networkCapabilities.getSubscriptionIds());
|
||||
final NetworkRequest requestWithIds = new NetworkRequest.Builder()
|
||||
.setSubIds(Set.of(TEST_SUBID1, TEST_SUBID2)).build();
|
||||
.setSubscriptionIds(Set.of(TEST_SUBID1, TEST_SUBID2)).build();
|
||||
assertEquals(Set.of(TEST_SUBID1, TEST_SUBID2),
|
||||
requestWithIds.networkCapabilities.getSubIds());
|
||||
requestWithIds.networkCapabilities.getSubscriptionIds());
|
||||
|
||||
assertFalse(requestWithIds.canBeSatisfiedBy(ncWithoutId));
|
||||
assertTrue(requestWithIds.canBeSatisfiedBy(ncWithOtherIds));
|
||||
@@ -1132,8 +1132,8 @@ public class NetworkCapabilitiesTest {
|
||||
|
||||
if (isAtLeastS()) {
|
||||
final NetworkCapabilities nc2 = new NetworkCapabilities.Builder()
|
||||
.setSubIds(Set.of(TEST_SUBID1)).build();
|
||||
assertEquals(Set.of(TEST_SUBID1), nc2.getSubIds());
|
||||
.setSubscriptionIds(Set.of(TEST_SUBID1)).build();
|
||||
assertEquals(Set.of(TEST_SUBID1), nc2.getSubscriptionIds());
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user