diff options
author | xshu <xshu@google.com> | 2019-12-12 15:17:10 -0800 |
---|---|---|
committer | xshu <xshu@google.com> | 2019-12-18 10:28:27 -0800 |
commit | 8417be01b133d54ffdcfff6936418c6abca57eb7 (patch) | |
tree | b8f89387881157f11a46337199a98f68d4b6d16f /tests | |
parent | a5280b9a68719e79910e51c5707f42b46318b2b9 (diff) |
BssidBlocklistMonitor metric
Add number of blocked BSSIDs to connection events.
Primarily interested in the Connection success and failures when the
number of blocked BSSIDs is non zero.
Bug: 139287182
Test: atest FrameworksWifiTests
Change-Id: Icdd51d96d94debe9e1306dc55ed92c6c1a352ca1
Diffstat (limited to 'tests')
-rw-r--r-- | tests/wifitests/src/com/android/server/wifi/WifiMetricsTest.java | 24 |
1 files changed, 24 insertions, 0 deletions
diff --git a/tests/wifitests/src/com/android/server/wifi/WifiMetricsTest.java b/tests/wifitests/src/com/android/server/wifi/WifiMetricsTest.java index c63cf487e..94d645646 100644 --- a/tests/wifitests/src/com/android/server/wifi/WifiMetricsTest.java +++ b/tests/wifitests/src/com/android/server/wifi/WifiMetricsTest.java @@ -143,6 +143,7 @@ public class WifiMetricsTest extends WifiBaseTest { @Mock Clock mClock; @Mock ScoringParams mScoringParams; @Mock WifiConfigManager mWcm; + @Mock BssidBlocklistMonitor mBssidBlocklistMonitor; @Mock PasspointManager mPpm; @Mock WifiNetworkSelector mWns; @Mock WifiPowerMetrics mWifiPowerMetrics; @@ -165,6 +166,7 @@ public class WifiMetricsTest extends WifiBaseTest { new WifiAwareMetrics(mClock), new RttMetrics(mClock), mWifiPowerMetrics, mWifiP2pMetrics, mDppMetrics); mWifiMetrics.setWifiConfigManager(mWcm); + mWifiMetrics.setBssidBlocklistMonitor(mBssidBlocklistMonitor); mWifiMetrics.setPasspointManager(mPpm); mWifiMetrics.setScoringParams(mScoringParams); mWifiMetrics.setWifiNetworkSelector(mWns); @@ -1476,6 +1478,28 @@ public class WifiMetricsTest extends WifiBaseTest { } /** + * Verify the logging of number of blocked BSSIDs in ConnectionEvent. + */ + @Test + public void testMetricNumBssidInBlocklist() throws Exception { + WifiConfiguration config = mock(WifiConfiguration.class); + config.SSID = "\"" + SSID + "\""; + when(config.getNetworkSelectionStatus()).thenReturn( + mock(WifiConfiguration.NetworkSelectionStatus.class)); + when(mBssidBlocklistMonitor.getNumBlockedBssidsForSsid(eq(config.SSID))).thenReturn(3); + mWifiMetrics.startConnectionEvent(config, "RED", + WifiMetricsProto.ConnectionEvent.ROAM_NONE); + mWifiMetrics.endConnectionEvent( + WifiMetrics.ConnectionEvent.FAILURE_ASSOCIATION_TIMED_OUT, + WifiMetricsProto.ConnectionEvent.HLF_NONE, + WifiMetricsProto.ConnectionEvent.FAILURE_REASON_UNKNOWN); + dumpProtoAndDeserialize(); + + assertEquals(1, mDecodedProto.connectionEvent.length); + assertEquals(3, mDecodedProto.connectionEvent[0].numBssidInBlocklist); + } + + /** * Test that WifiMetrics is serializing/deserializing authentication failure events. */ @Test |