LineageOS/android_packages_modules_Connectivity
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge "Revert "Replace the usage of UidRange"" am: 83f77d83fd am: 0f97255402

Browse Source 
Original change: https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/1645769

Change-Id: I86886953ab47f4d176b11d64a915e637559b3987
This commit is contained in:
Anthony Stange
2021-03-18 17:41:40 +00:00
committed by Automerger Merge Worker
parent 54bf54b774 0f97255402
commit abc4b59c47
1 changed files with 6 additions and 30 deletions
Download Patch File Download Diff File Expand all files Collapse all files

36
tests/cts/net/src/android/net/cts/NetworkRequestTest.java
View File

@@ -39,20 +39,17 @@ import android.net.MatchAllNetworkSpecifier;
import android.net.NetworkCapabilities;
import android.net.NetworkRequest;
import android.net.NetworkSpecifier;
import android.net.UidRange;
import android.net.wifi.WifiNetworkSpecifier;
import android.os.Build;
import android.os.PatternMatcher;
import android.os.Process;
import android.util.ArraySet;
import android.util.Range;
import androidx.test.runner.AndroidJUnit4;
import com.android.modules.utils.build.SdkLevel;
import com.android.networkstack.apishim.ConstantsShim;
import com.android.networkstack.apishim.NetworkRequestShimImpl;
import com.android.networkstack.apishim.common.NetworkRequestShim;
import com.android.networkstack.apishim.common.UnsupportedApiLevelException;
import com.android.testutils.DevSdkIgnoreRule;
import com.android.testutils.DevSdkIgnoreRule.IgnoreUpTo;
@@ -60,8 +57,6 @@ import org.junit.Rule;
import org.junit.Test;
import org.junit.runner.RunWith;
import java.util.Set;
@RunWith(AndroidJUnit4.class)
public class NetworkRequestTest {
@Rule
@@ -230,14 +225,6 @@ public class NetworkRequestTest {
assertTrue(requestCellularInternet.canBeSatisfiedBy(capCellularVpnMmsInternet));
}
private void setUids(NetworkRequest.Builder builder, Set<Range<Integer>> ranges)
throws UnsupportedApiLevelException {
if (SdkLevel.isAtLeastS()) {
final NetworkRequestShim networkRequestShim = NetworkRequestShimImpl.newInstance();
networkRequestShim.setUids(builder, ranges);
}
}
@Test
@IgnoreUpTo(Build.VERSION_CODES.Q)
public void testInvariantInCanBeSatisfiedBy() {
@@ -245,26 +232,15 @@ public class NetworkRequestTest {
// NetworkCapabilities.satisfiedByNetworkCapabilities().
final LocalNetworkSpecifier specifier1 = new LocalNetworkSpecifier(1234 /* id */);
final int uid = Process.myUid();
final NetworkRequest.Builder nrBuilder = new NetworkRequest.Builder()
final ArraySet<UidRange> ranges = new ArraySet<>();
ranges.add(new UidRange(uid, uid));
final NetworkRequest requestCombination = new NetworkRequest.Builder()
.addTransportType(TRANSPORT_CELLULAR)
.addCapability(NET_CAPABILITY_INTERNET)
.setLinkUpstreamBandwidthKbps(1000)
.setNetworkSpecifier(specifier1)
.setSignalStrength(-123);
// The uid ranges should be set into the request, but setUids() takes a set of UidRange
// that is hidden and inaccessible from shims. Before, S setUids will be a no-op. But
// because NetworkRequest.Builder sets the UID of the request to the current UID, the
// request contains the current UID both on S and before S.
final Set<Range<Integer>> ranges = new ArraySet<>();
ranges.add(new Range<Integer>(uid, uid));
try {
setUids(nrBuilder, ranges);
} catch (UnsupportedApiLevelException e) {
// Not supported before API31.
}
final NetworkRequest requestCombination = nrBuilder.build();
.setSignalStrength(-123)
.setUids(ranges).build();
final NetworkCapabilities capCell = new NetworkCapabilities.Builder()
.addTransportType(TRANSPORT_CELLULAR).build();
assertCorrectlySatisfies(false, requestCombination, capCell);