Merge changes Icd871654,I7c70328d am: 1efc2bc73f am: 0908c8a645
Original change: https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/2418999 Change-Id: I70d25e5ce7d895ba4fa1d8a276065790d9df8a01 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
commit
95bdfa6cb2
1 changed files with 19 additions and 7 deletions
|
|
@ -2678,35 +2678,47 @@ public class TetheringTest {
|
|||
public void testUpstreamNetworkChanged() {
|
||||
final Tethering.TetherMainSM stateMachine = (Tethering.TetherMainSM)
|
||||
mTetheringDependencies.mUpstreamNetworkMonitorSM;
|
||||
final InOrder inOrder = inOrder(mNotificationUpdater);
|
||||
|
||||
// Gain upstream.
|
||||
final UpstreamNetworkState upstreamState = buildMobileIPv4UpstreamState();
|
||||
initTetheringUpstream(upstreamState);
|
||||
stateMachine.chooseUpstreamType(true);
|
||||
mTetheringEventCallback.expectUpstreamChanged(upstreamState.network);
|
||||
verify(mNotificationUpdater)
|
||||
inOrder.verify(mNotificationUpdater)
|
||||
.onUpstreamCapabilitiesChanged(upstreamState.networkCapabilities);
|
||||
|
||||
// Set the upstream with the same network ID but different object.
|
||||
final UpstreamNetworkState upstreamState2 = buildMobileIPv4UpstreamState();
|
||||
initTetheringUpstream(upstreamState2);
|
||||
stateMachine.chooseUpstreamType(true);
|
||||
// Bug: duplicated upstream change event.
|
||||
mTetheringEventCallback.expectUpstreamChanged(upstreamState2.network);
|
||||
inOrder.verify(mNotificationUpdater)
|
||||
.onUpstreamCapabilitiesChanged(upstreamState2.networkCapabilities);
|
||||
|
||||
// Lose upstream.
|
||||
initTetheringUpstream(null);
|
||||
stateMachine.chooseUpstreamType(true);
|
||||
mTetheringEventCallback.expectUpstreamChanged(NULL_NETWORK);
|
||||
verify(mNotificationUpdater).onUpstreamCapabilitiesChanged(null);
|
||||
inOrder.verify(mNotificationUpdater).onUpstreamCapabilitiesChanged(null);
|
||||
}
|
||||
|
||||
@Test
|
||||
public void testUpstreamCapabilitiesChanged() {
|
||||
final Tethering.TetherMainSM stateMachine = (Tethering.TetherMainSM)
|
||||
mTetheringDependencies.mUpstreamNetworkMonitorSM;
|
||||
final InOrder inOrder = inOrder(mNotificationUpdater);
|
||||
final UpstreamNetworkState upstreamState = buildMobileIPv4UpstreamState();
|
||||
initTetheringUpstream(upstreamState);
|
||||
|
||||
stateMachine.chooseUpstreamType(true);
|
||||
inOrder.verify(mNotificationUpdater)
|
||||
.onUpstreamCapabilitiesChanged(upstreamState.networkCapabilities);
|
||||
|
||||
stateMachine.handleUpstreamNetworkMonitorCallback(EVENT_ON_CAPABILITIES, upstreamState);
|
||||
// Should have two onUpstreamCapabilitiesChanged().
|
||||
// One is called by reportUpstreamChanged(). One is called by EVENT_ON_CAPABILITIES.
|
||||
verify(mNotificationUpdater, times(2))
|
||||
inOrder.verify(mNotificationUpdater)
|
||||
.onUpstreamCapabilitiesChanged(upstreamState.networkCapabilities);
|
||||
reset(mNotificationUpdater);
|
||||
|
||||
// Verify that onUpstreamCapabilitiesChanged won't be called if not current upstream network
|
||||
// capabilities changed.
|
||||
|
|
@ -2714,7 +2726,7 @@ public class TetheringTest {
|
|||
upstreamState.linkProperties, upstreamState.networkCapabilities,
|
||||
new Network(WIFI_NETID));
|
||||
stateMachine.handleUpstreamNetworkMonitorCallback(EVENT_ON_CAPABILITIES, upstreamState2);
|
||||
verify(mNotificationUpdater, never()).onUpstreamCapabilitiesChanged(any());
|
||||
inOrder.verify(mNotificationUpdater, never()).onUpstreamCapabilitiesChanged(any());
|
||||
}
|
||||
|
||||
@Test
|
||||
|
|
|
|||
Loading…
Reference in a new issue