diff --git a/tests/net/java/com/android/server/ConnectivityServiceTest.java b/tests/net/java/com/android/server/ConnectivityServiceTest.java index f946597b99..ed975eb5a6 100644 --- a/tests/net/java/com/android/server/ConnectivityServiceTest.java +++ b/tests/net/java/com/android/server/ConnectivityServiceTest.java @@ -1178,11 +1178,6 @@ public class ConnectivityServiceTest { return (mMockNetworkAgent == null) ? null : mMockNetworkAgent.getNetwork(); } - @Override - public int getNetId() { - return (mMockNetworkAgent == null) ? NETID_UNSET : mMockNetworkAgent.getNetwork().netId; - } - @Override public int getActiveVpnType() { return mVpnType; @@ -1207,10 +1202,12 @@ public class ConnectivityServiceTest { mNetworkCapabilities); mMockNetworkAgent.waitForIdle(TIMEOUT_MS); - verify(mMockNetd, times(1)).networkAddUidRanges(eq(mMockVpn.getNetId()), + final int expectedNetId = mMockVpn.getNetwork() == null ? NETID_UNSET + : mMockVpn.getNetwork().getNetId(); + verify(mMockNetd, times(1)).networkAddUidRanges(eq(expectedNetId), eq(toUidRangeStableParcels(uids))); verify(mMockNetd, never()) - .networkRemoveUidRanges(eq(mMockVpn.getNetId()), any()); + .networkRemoveUidRanges(eq(expectedNetId), any()); mAgentRegistered = true; updateState(NetworkInfo.DetailedState.CONNECTED, "registerAgent"); mNetworkCapabilities.set(mMockNetworkAgent.getNetworkCapabilities()); @@ -9802,11 +9799,14 @@ public class ConnectivityServiceTest { exemptUidCaptor.capture()); assertContainsExactly(exemptUidCaptor.getValue(), Process.VPN_UID, exemptUid); + final int expectedNetId = mMockVpn.getNetwork() == null ? NETID_UNSET + : mMockVpn.getNetwork().getNetId(); + if (add) { - inOrder.verify(mMockNetd, times(1)).networkAddUidRanges(eq(mMockVpn.getNetId()), + inOrder.verify(mMockNetd, times(1)).networkAddUidRanges(eq(expectedNetId), eq(toUidRangeStableParcels(vpnRanges))); } else { - inOrder.verify(mMockNetd, times(1)).networkRemoveUidRanges(eq(mMockVpn.getNetId()), + inOrder.verify(mMockNetd, times(1)).networkRemoveUidRanges(eq(expectedNetId), eq(toUidRangeStableParcels(vpnRanges))); }