summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorNate Jiang <qiangjiang@google.com>2019-05-21 19:54:11 -0700
committerNate Jiang <qiangjiang@google.com>2019-05-22 17:41:55 -0700
commit8be98dc89d77103b86639add8ce3147560ae5231 (patch)
tree71761132f3dcbd8489526084ae0b7b946e7b85e3
parent7d44d68570537a387c6e0efdcb1ebe28a8367fbb (diff)
downloadandroid_frameworks_opt_net_wifi-8be98dc89d77103b86639add8ce3147560ae5231.tar.gz
android_frameworks_opt_net_wifi-8be98dc89d77103b86639add8ce3147560ae5231.tar.bz2
android_frameworks_opt_net_wifi-8be98dc89d77103b86639add8ce3147560ae5231.zip
[AWARE] send WifiAware state change when Aware down caused by iff down&up
Send broadcast to notify Apps WifiAware is down and need to re-attach. Test: ACTS NonConcurrencyTest Test: atest android.net.wifi Test: atest com.android.server.wifi Bug: 132943959 Change-Id: Iac00c249aea2f5aa83dd8de2447d443f10f0bbbd
-rw-r--r--service/java/com/android/server/wifi/aware/WifiAwareStateManager.java4
-rw-r--r--tests/wifitests/src/com/android/server/wifi/aware/WifiAwareStateManagerTest.java89
2 files changed, 74 insertions, 19 deletions
diff --git a/service/java/com/android/server/wifi/aware/WifiAwareStateManager.java b/service/java/com/android/server/wifi/aware/WifiAwareStateManager.java
index 36cabd646..9b096e525 100644
--- a/service/java/com/android/server/wifi/aware/WifiAwareStateManager.java
+++ b/service/java/com/android/server/wifi/aware/WifiAwareStateManager.java
@@ -1413,7 +1413,9 @@ public class WifiAwareStateManager implements WifiAwareShellCommand.DelegatedShe
*/
onAwareDownLocal();
-
+ if (reason != NanStatusType.SUCCESS) {
+ sendAwareStateChangedBroadcast(false);
+ }
break;
}
case NOTIFICATION_TYPE_ON_MESSAGE_SEND_SUCCESS: {
diff --git a/tests/wifitests/src/com/android/server/wifi/aware/WifiAwareStateManagerTest.java b/tests/wifitests/src/com/android/server/wifi/aware/WifiAwareStateManagerTest.java
index 04ecfe7d4..d6b1b38d1 100644
--- a/tests/wifitests/src/com/android/server/wifi/aware/WifiAwareStateManagerTest.java
+++ b/tests/wifitests/src/com/android/server/wifi/aware/WifiAwareStateManagerTest.java
@@ -357,7 +357,7 @@ public class WifiAwareStateManagerTest {
// (1) enable usage
mDut.enableUsage();
mMockLooper.dispatchAll();
- validateCorrectAwareStatusChangeBroadcast(inOrder, true);
+ validateCorrectAwareStatusChangeBroadcast(inOrder);
inOrder.verify(mMockNative).getCapabilities(transactionId.capture());
mDut.onCapabilitiesUpdateResponse(transactionId.getValue(), getCapabilities());
mMockLooper.dispatchAll();
@@ -403,7 +403,7 @@ public class WifiAwareStateManagerTest {
// (1) check initial state
mDut.enableUsage();
mMockLooper.dispatchAll();
- validateCorrectAwareStatusChangeBroadcast(inOrder, true);
+ validateCorrectAwareStatusChangeBroadcast(inOrder);
inOrder.verify(mMockNative).getCapabilities(transactionId.capture());
mDut.onCapabilitiesUpdateResponse(transactionId.getValue(), getCapabilities());
mMockLooper.dispatchAll();
@@ -415,7 +415,7 @@ public class WifiAwareStateManagerTest {
collector.checkThat("usage disabled", mDut.isUsageEnabled(), equalTo(false));
inOrder.verify(mMockNative).disable(transactionId.capture());
mDut.onDisableResponse(transactionId.getValue(), NanStatusType.SUCCESS);
- validateCorrectAwareStatusChangeBroadcast(inOrder, false);
+ validateCorrectAwareStatusChangeBroadcast(inOrder);
// (3) try connecting and validate that get failure callback (though app should be aware of
// non-availability through state change broadcast and/or query API)
@@ -448,7 +448,7 @@ public class WifiAwareStateManagerTest {
// (1) check initial state
mDut.enableUsage();
mMockLooper.dispatchAll();
- validateCorrectAwareStatusChangeBroadcast(inOrder, true);
+ validateCorrectAwareStatusChangeBroadcast(inOrder);
inOrderM.verify(mAwareMetricsMock).recordEnableUsage();
inOrder.verify(mMockNative).getCapabilities(transactionId.capture());
mDut.onCapabilitiesUpdateResponse(transactionId.getValue(), getCapabilities());
@@ -476,7 +476,7 @@ public class WifiAwareStateManagerTest {
inOrderM.verify(mAwareMetricsMock).recordAttachSessionDuration(anyLong());
inOrderM.verify(mAwareMetricsMock).recordDisableAware();
inOrderM.verify(mAwareMetricsMock).recordDisableUsage();
- validateCorrectAwareStatusChangeBroadcast(inOrder, false);
+ validateCorrectAwareStatusChangeBroadcast(inOrder);
validateInternalClientInfoCleanedUp(clientId);
mDut.onDisableResponse(transactionId.getValue(), NanStatusType.SUCCESS);
mMockLooper.dispatchAll();
@@ -498,7 +498,7 @@ public class WifiAwareStateManagerTest {
mMockLooper.dispatchAll();
inOrderM.verify(mAwareMetricsMock).recordEnableUsage();
collector.checkThat("usage enabled", mDut.isUsageEnabled(), equalTo(true));
- validateCorrectAwareStatusChangeBroadcast(inOrder, true);
+ validateCorrectAwareStatusChangeBroadcast(inOrder);
// (7) connect (should be successful)
mDut.connect(clientId, uid, pid, callingPackage, mockCallback, configRequest, false);
@@ -536,7 +536,7 @@ public class WifiAwareStateManagerTest {
// (1) check initial state
mDut.enableUsage();
mMockLooper.dispatchAll();
- validateCorrectAwareStatusChangeBroadcast(inOrder, true);
+ validateCorrectAwareStatusChangeBroadcast(inOrder);
inOrder.verify(mMockNative).getCapabilities(transactionId.capture());
mDut.onCapabilitiesUpdateResponse(transactionId.getValue(), getCapabilities());
mMockLooper.dispatchAll();
@@ -3148,7 +3148,8 @@ public class WifiAwareStateManagerTest {
mMockLooper.dispatchAll();
inOrder.verify(mMockNative).disable(transactionId.capture());
mDut.onDisableResponse(transactionId.getValue(), NanStatusType.SUCCESS);
- validateCorrectAwareStatusChangeBroadcast(inOrder, false);
+ collector.checkThat("usage disabled", mDut.isUsageEnabled(), equalTo(false));
+ validateCorrectAwareStatusChangeBroadcast(inOrder);
// (4) power state change: SCREEN ON (but DOZE still on - fakish but expect no changes)
simulatePowerStateChangeInteractive(false);
@@ -3167,7 +3168,8 @@ public class WifiAwareStateManagerTest {
// (5) power state change: DOZE OFF
simulatePowerStateChangeDoze(false);
mMockLooper.dispatchAll();
- validateCorrectAwareStatusChangeBroadcast(inOrder, true);
+ collector.checkThat("usage enabled", mDut.isUsageEnabled(), equalTo(true));
+ validateCorrectAwareStatusChangeBroadcast(inOrder);
verifyNoMoreInteractions(mMockNativeManager, mMockNative, mockCallback);
}
@@ -3212,7 +3214,8 @@ public class WifiAwareStateManagerTest {
mMockLooper.dispatchAll();
inOrder.verify(mMockNative).disable(transactionId.capture());
mDut.onDisableResponse(transactionId.getValue(), NanStatusType.SUCCESS);
- validateCorrectAwareStatusChangeBroadcast(inOrder, false);
+ collector.checkThat("usage disabled", mDut.isUsageEnabled(), equalTo(false));
+ validateCorrectAwareStatusChangeBroadcast(inOrder);
// disable other gating feature -> no change
simulatePowerStateChangeDoze(true);
@@ -3227,7 +3230,8 @@ public class WifiAwareStateManagerTest {
// (4) location mode change: enable
simulateLocationModeChange(true);
mMockLooper.dispatchAll();
- validateCorrectAwareStatusChangeBroadcast(inOrder, true);
+ collector.checkThat("usage enabled", mDut.isUsageEnabled(), equalTo(true));
+ validateCorrectAwareStatusChangeBroadcast(inOrder);
verifyNoMoreInteractions(mMockNativeManager, mMockNative, mockCallback);
}
@@ -3272,7 +3276,8 @@ public class WifiAwareStateManagerTest {
mMockLooper.dispatchAll();
inOrder.verify(mMockNative).disable(transactionId.capture());
mDut.onDisableResponse(transactionId.getValue(), NanStatusType.SUCCESS);
- validateCorrectAwareStatusChangeBroadcast(inOrder, false);
+ collector.checkThat("usage disabled", mDut.isUsageEnabled(), equalTo(false));
+ validateCorrectAwareStatusChangeBroadcast(inOrder);
// disable other gating feature -> no change
simulatePowerStateChangeDoze(true);
@@ -3287,11 +3292,63 @@ public class WifiAwareStateManagerTest {
// (4) wifi state change: enable
simulateWifiStateChange(true);
mMockLooper.dispatchAll();
- validateCorrectAwareStatusChangeBroadcast(inOrder, true);
+ collector.checkThat("usage enabled", mDut.isUsageEnabled(), equalTo(true));
+ validateCorrectAwareStatusChangeBroadcast(inOrder);
verifyNoMoreInteractions(mMockNativeManager, mMockNative, mockCallback);
}
+ /**
+ * Validate aware state change when get aware down from native
+ */
+ @Test
+ public void testWifiAwareStateChangeFromNative() throws Exception {
+ final int clientId = 12314;
+ final int uid = 1000;
+ final int pid = 2000;
+ final String callingPackage = "com.google.somePackage";
+ final ConfigRequest configRequest = new ConfigRequest.Builder().build();
+
+ IWifiAwareEventCallback mockCallback = mock(IWifiAwareEventCallback.class);
+ InOrder inOrder = inOrder(mMockContext, mMockNative, mockCallback);
+ ArgumentCaptor<Short> transactionId = ArgumentCaptor.forClass(Short.class);
+
+ // (0) check initial state
+ mDut.enableUsage();
+ mMockLooper.dispatchAll();
+ validateCorrectAwareStatusChangeBroadcast(inOrder);
+ inOrder.verify(mMockNative).getCapabilities(transactionId.capture());
+ mDut.onCapabilitiesUpdateResponse(transactionId.getValue(), getCapabilities());
+ mMockLooper.dispatchAll();
+ collector.checkThat("usage enabled", mDut.isUsageEnabled(), equalTo(true));
+
+ // (1) connect client
+ mDut.connect(clientId, uid, pid, callingPackage, mockCallback, configRequest, false);
+ mMockLooper.dispatchAll();
+ inOrder.verify(mMockNative).enableAndConfigure(transactionId.capture(),
+ eq(configRequest), eq(false), eq(true), eq(true), eq(false));
+ mDut.onConfigSuccessResponse(transactionId.getValue());
+ mMockLooper.dispatchAll();
+ inOrder.verify(mockCallback).onConnectSuccess(clientId);
+
+ // (2) Aware down notification from native
+ mDut.onAwareDownNotification(NanStatusType.UNSUPPORTED_CONCURRENCY_NAN_DISABLED);
+ mMockLooper.dispatchAll();
+ collector.checkThat("usage enabled", mDut.isUsageEnabled(), equalTo(true));
+ validateCorrectAwareStatusChangeBroadcast(inOrder);
+
+ // (3) try reconnect client
+ mDut.connect(clientId, uid, pid, callingPackage, mockCallback, configRequest, false);
+ mMockLooper.dispatchAll();
+ inOrder.verify(mMockNative).enableAndConfigure(transactionId.capture(),
+ eq(configRequest), eq(false), eq(true), eq(true), eq(false));
+ mDut.onConfigSuccessResponse(transactionId.getValue());
+ mMockLooper.dispatchAll();
+ inOrder.verify(mockCallback).onConnectSuccess(clientId);
+
+ verifyNoMoreInteractions(mMockNative, mockCallback);
+ }
+
/*
* Tests of internal state of WifiAwareStateManager: very limited (not usually
* a good idea). However, these test that the internal state is cleaned-up
@@ -3352,12 +3409,8 @@ public class WifiAwareStateManagerTest {
/**
* Validates that the broadcast sent on Aware status change is correct.
- *
- * @param expectedEnabled The expected change status - i.e. are we expected
- * to announce that Aware is enabled (true) or disabled (false).
*/
- private void validateCorrectAwareStatusChangeBroadcast(InOrder inOrder,
- boolean expectedEnabled) {
+ private void validateCorrectAwareStatusChangeBroadcast(InOrder inOrder) {
ArgumentCaptor<Intent> intent = ArgumentCaptor.forClass(Intent.class);
inOrder.verify(mMockContext).sendBroadcastAsUser(intent.capture(), eq(UserHandle.ALL));