From 8f67bdec5a8fb0e1ceccd6be6eb20e2a589bbf2f Mon Sep 17 00:00:00 2001 From: Lorenzo Colitti Date: Mon, 10 May 2021 01:13:22 +0900 Subject: [PATCH] Re-enable check for VpnTransportInfo session ID. This check was commented out because VpnTransportInfo was not correctly propagated to ConnectivityService. This is now fixed. Bug: 183938194 Test: atest CtsNetTestCases:NetworkAgentTest Change-Id: Iab9987c8b58487ee44d8479b02e030c7e1e4465f --- tests/cts/net/src/android/net/cts/NetworkAgentTest.kt | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/tests/cts/net/src/android/net/cts/NetworkAgentTest.kt b/tests/cts/net/src/android/net/cts/NetworkAgentTest.kt index 5cf7140567..32145319db 100644 --- a/tests/cts/net/src/android/net/cts/NetworkAgentTest.kt +++ b/tests/cts/net/src/android/net/cts/NetworkAgentTest.kt @@ -596,8 +596,7 @@ class NetworkAgentTest { assertNotNull(vpnNc) assertEquals(VpnManager.TYPE_VPN_SERVICE, (vpnNc.transportInfo as VpnTransportInfo).type) - // TODO: b/183938194 please fix the issue and enable following check. - // assertEquals(mySessionId, (vpnNc.transportInfo as VpnTransportInfo).sessionId) + assertEquals(mySessionId, (vpnNc.transportInfo as VpnTransportInfo).sessionId) val testAndVpn = intArrayOf(TRANSPORT_TEST, TRANSPORT_VPN) assertTrue(hasAllTransports(vpnNc, testAndVpn))