Merge "Add session ID to VpnTransportInfo" am: 320cd17220
Original change: https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/1652217 Change-Id: I2bcafaaeb3f9dd2030745f1f7fa52f10f60733dd
This commit is contained in:
@@ -547,6 +547,7 @@ class NetworkAgentTest {
|
|||||||
@Test
|
@Test
|
||||||
@IgnoreUpTo(Build.VERSION_CODES.R)
|
@IgnoreUpTo(Build.VERSION_CODES.R)
|
||||||
fun testSetUnderlyingNetworksAndVpnSpecifier() {
|
fun testSetUnderlyingNetworksAndVpnSpecifier() {
|
||||||
|
val mySessionId = "MySession12345"
|
||||||
val request = NetworkRequest.Builder()
|
val request = NetworkRequest.Builder()
|
||||||
.addTransportType(TRANSPORT_TEST)
|
.addTransportType(TRANSPORT_TEST)
|
||||||
.addTransportType(TRANSPORT_VPN)
|
.addTransportType(TRANSPORT_VPN)
|
||||||
@@ -560,7 +561,7 @@ class NetworkAgentTest {
|
|||||||
addTransportType(TRANSPORT_TEST)
|
addTransportType(TRANSPORT_TEST)
|
||||||
addTransportType(TRANSPORT_VPN)
|
addTransportType(TRANSPORT_VPN)
|
||||||
removeCapability(NET_CAPABILITY_NOT_VPN)
|
removeCapability(NET_CAPABILITY_NOT_VPN)
|
||||||
setTransportInfo(VpnTransportInfo(VpnManager.TYPE_VPN_SERVICE))
|
setTransportInfo(VpnTransportInfo(VpnManager.TYPE_VPN_SERVICE, mySessionId))
|
||||||
if (SdkLevel.isAtLeastS()) {
|
if (SdkLevel.isAtLeastS()) {
|
||||||
addCapability(NET_CAPABILITY_NOT_VCN_MANAGED)
|
addCapability(NET_CAPABILITY_NOT_VCN_MANAGED)
|
||||||
}
|
}
|
||||||
@@ -580,6 +581,8 @@ class NetworkAgentTest {
|
|||||||
assertNotNull(vpnNc)
|
assertNotNull(vpnNc)
|
||||||
assertEquals(VpnManager.TYPE_VPN_SERVICE,
|
assertEquals(VpnManager.TYPE_VPN_SERVICE,
|
||||||
(vpnNc.transportInfo as VpnTransportInfo).type)
|
(vpnNc.transportInfo as VpnTransportInfo).type)
|
||||||
|
// TODO: b/183938194 please fix the issue and enable following check.
|
||||||
|
// assertEquals(mySessionId, (vpnNc.transportInfo as VpnTransportInfo).sessionId)
|
||||||
|
|
||||||
val testAndVpn = intArrayOf(TRANSPORT_TEST, TRANSPORT_VPN)
|
val testAndVpn = intArrayOf(TRANSPORT_TEST, TRANSPORT_VPN)
|
||||||
assertTrue(hasAllTransports(vpnNc, testAndVpn))
|
assertTrue(hasAllTransports(vpnNc, testAndVpn))
|
||||||
|
|||||||
Reference in New Issue
Block a user