Merge "Address remaining comments on aosp/1607893" am: 08b0411d4f
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1640140 Change-Id: I49b1ea7e3461ef0cad563d5d7933b5535f94d08c
This commit is contained in:
@@ -11402,7 +11402,6 @@ public class ConnectivityServiceTest {
|
|||||||
mSystemDefaultNetworkCallback.expectCallback(CallbackEntry.LOST, mCellNetworkAgent);
|
mSystemDefaultNetworkCallback.expectCallback(CallbackEntry.LOST, mCellNetworkAgent);
|
||||||
mDefaultNetworkCallback.expectCallback(CallbackEntry.LOST, mCellNetworkAgent);
|
mDefaultNetworkCallback.expectCallback(CallbackEntry.LOST, mCellNetworkAgent);
|
||||||
mProfileDefaultNetworkCallback.assertNoCallback();
|
mProfileDefaultNetworkCallback.assertNoCallback();
|
||||||
waitForIdle();
|
|
||||||
inOrder.verify(mMockNetd).networkDestroy(mCellNetworkAgent.getNetwork().netId);
|
inOrder.verify(mMockNetd).networkDestroy(mCellNetworkAgent.getNetwork().netId);
|
||||||
|
|
||||||
mCellNetworkAgent = new TestNetworkAgentWrapper(TRANSPORT_CELLULAR);
|
mCellNetworkAgent = new TestNetworkAgentWrapper(TRANSPORT_CELLULAR);
|
||||||
@@ -11421,7 +11420,6 @@ public class ConnectivityServiceTest {
|
|||||||
assertNoCallbacks(mSystemDefaultNetworkCallback, mDefaultNetworkCallback);
|
assertNoCallbacks(mSystemDefaultNetworkCallback, mDefaultNetworkCallback);
|
||||||
inOrder.verify(mMockNetd).networkAddUidRanges(mCellNetworkAgent.getNetwork().netId,
|
inOrder.verify(mMockNetd).networkAddUidRanges(mCellNetworkAgent.getNetwork().netId,
|
||||||
uidRangeFor(testHandle));
|
uidRangeFor(testHandle));
|
||||||
waitForIdle();
|
|
||||||
inOrder.verify(mMockNetd).networkDestroy(workAgent.getNetwork().netId);
|
inOrder.verify(mMockNetd).networkDestroy(workAgent.getNetwork().netId);
|
||||||
|
|
||||||
mCellNetworkAgent.disconnect();
|
mCellNetworkAgent.disconnect();
|
||||||
@@ -11429,6 +11427,8 @@ public class ConnectivityServiceTest {
|
|||||||
mDefaultNetworkCallback.expectCallback(CallbackEntry.LOST, mCellNetworkAgent);
|
mDefaultNetworkCallback.expectCallback(CallbackEntry.LOST, mCellNetworkAgent);
|
||||||
mProfileDefaultNetworkCallback.expectCallback(CallbackEntry.LOST, mCellNetworkAgent);
|
mProfileDefaultNetworkCallback.expectCallback(CallbackEntry.LOST, mCellNetworkAgent);
|
||||||
|
|
||||||
|
// Waiting for the handler to be idle before checking for networkDestroy is necessary
|
||||||
|
// here because ConnectivityService calls onLost before the network is fully torn down.
|
||||||
waitForIdle();
|
waitForIdle();
|
||||||
inOrder.verify(mMockNetd).networkDestroy(mCellNetworkAgent.getNetwork().netId);
|
inOrder.verify(mMockNetd).networkDestroy(mCellNetworkAgent.getNetwork().netId);
|
||||||
|
|
||||||
@@ -11458,7 +11458,6 @@ public class ConnectivityServiceTest {
|
|||||||
workAgent2.disconnect();
|
workAgent2.disconnect();
|
||||||
mProfileDefaultNetworkCallback.expectCallback(CallbackEntry.LOST, workAgent2);
|
mProfileDefaultNetworkCallback.expectCallback(CallbackEntry.LOST, workAgent2);
|
||||||
assertNoCallbacks(mSystemDefaultNetworkCallback, mDefaultNetworkCallback);
|
assertNoCallbacks(mSystemDefaultNetworkCallback, mDefaultNetworkCallback);
|
||||||
waitForIdle();
|
|
||||||
inOrder.verify(mMockNetd).networkDestroy(workAgent2.getNetwork().netId);
|
inOrder.verify(mMockNetd).networkDestroy(workAgent2.getNetwork().netId);
|
||||||
|
|
||||||
assertNoCallbacks(mSystemDefaultNetworkCallback, mDefaultNetworkCallback,
|
assertNoCallbacks(mSystemDefaultNetworkCallback, mDefaultNetworkCallback,
|
||||||
|
|||||||
Reference in New Issue
Block a user