Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Ignore checking TRANSPORT_USB when Android version is older than S" into sc-dev

Browse Source
This commit is contained in:
Lucas Lin
2021-04-06 08:40:23 +00:00
committed by Android (Google) Code Review
parent fd38f5d17b edaa67f486
commit 375467b87a
1 changed files with 10 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
tests/net/common/java/android/net/NetworkCapabilitiesTest.java
View File

@@ -48,6 +48,7 @@ import static android.os.Process.INVALID_UID;
import static com.android.modules.utils.build.SdkLevel.isAtLeastR; import static com.android.modules.utils.build.SdkLevel.isAtLeastR;
import static com.android.modules.utils.build.SdkLevel.isAtLeastS; import static com.android.modules.utils.build.SdkLevel.isAtLeastS;
import static com.android.net.module.util.NetworkCapabilitiesUtils.TRANSPORT_USB;
import static com.android.testutils.MiscAsserts.assertEmpty; import static com.android.testutils.MiscAsserts.assertEmpty;
import static com.android.testutils.MiscAsserts.assertThrows; import static com.android.testutils.MiscAsserts.assertThrows;
import static com.android.testutils.ParcelUtils.assertParcelSane; import static com.android.testutils.ParcelUtils.assertParcelSane;
@@ -959,6 +960,11 @@ public class NetworkCapabilitiesTest {
assertNotEquals(512, nc.getLinkUpstreamBandwidthKbps()); assertNotEquals(512, nc.getLinkUpstreamBandwidthKbps());
} }
private int getMaxTransport() {
if (!isAtLeastS() && MAX_TRANSPORT == TRANSPORT_USB) return MAX_TRANSPORT - 1;
return MAX_TRANSPORT;
}
@Test @Test
public void testSignalStrength() { public void testSignalStrength() {
final NetworkCapabilities nc = new NetworkCapabilities(); final NetworkCapabilities nc = new NetworkCapabilities();
@@ -970,7 +976,7 @@ public class NetworkCapabilitiesTest {
} }
private void assertNoTransport(NetworkCapabilities nc) { private void assertNoTransport(NetworkCapabilities nc) {
for (int i = MIN_TRANSPORT; i <= MAX_TRANSPORT; i++) { for (int i = MIN_TRANSPORT; i <= getMaxTransport(); i++) {
assertFalse(nc.hasTransport(i)); assertFalse(nc.hasTransport(i));
} }
} }
@@ -987,7 +993,7 @@ public class NetworkCapabilitiesTest {
assertFalse(nc.hasTransport(i)); assertFalse(nc.hasTransport(i));
} }
} }
for (int i = MAX_TRANSPORT; i > maxTransportType; i--) { for (int i = getMaxTransport(); i > maxTransportType; i--) {
if (positiveSequence) { if (positiveSequence) {
assertFalse(nc.hasTransport(i)); assertFalse(nc.hasTransport(i));
} else { } else {
@@ -1001,12 +1007,12 @@ public class NetworkCapabilitiesTest {
final NetworkCapabilities nc = new NetworkCapabilities(); final NetworkCapabilities nc = new NetworkCapabilities();
assertNoTransport(nc); assertNoTransport(nc);
// Test adding multiple transport types. // Test adding multiple transport types.
for (int i = MIN_TRANSPORT; i <= MAX_TRANSPORT; i++) { for (int i = MIN_TRANSPORT; i <= getMaxTransport(); i++) {
nc.addTransportType(i); nc.addTransportType(i);
checkCurrentTransportTypes(nc, i, true /* positiveSequence */); checkCurrentTransportTypes(nc, i, true /* positiveSequence */);
} }
// Test removing multiple transport types. // Test removing multiple transport types.
for (int i = MIN_TRANSPORT; i <= MAX_TRANSPORT; i++) { for (int i = MIN_TRANSPORT; i <= getMaxTransport(); i++) {
nc.removeTransportType(i); nc.removeTransportType(i);
checkCurrentTransportTypes(nc, i, false /* positiveSequence */); checkCurrentTransportTypes(nc, i, false /* positiveSequence */);
} }