diff options
author | Michael Plass <mplass@google.com> | 2017-10-27 15:08:00 -0700 |
---|---|---|
committer | Andreas Gampe <agampe@google.com> | 2018-02-28 11:49:32 -0800 |
commit | 4260ae8d7c8dfe3600f167a973ad00f9c49c9e4d (patch) | |
tree | 5194a30f2bee060d8be4784ecedbfae7c6027718 | |
parent | ac0845af4a05f4e7a2c0a3501a7f12e4374976fe (diff) | |
download | wifi-android-p-preview-1.tar.gz |
WifiMetricsTest fixesandroid-p-preview-1android-o-mr1-iot-preview-7o-mr1-iot-preview-7
Add missing @Test.
Add missing verifications for StaEvents generated from mTestStaLogInts.
(cherry picked from commit 38e4d5224e8870484d1f6e028763fc2d6dd62974)
(cherry picked from commit bb3272b409d725dc1aada2cb871f78423ac37c48)
Test: Yes, it is
Bug: 68271729
Bug: 71708302
Merged-In: I505656bc61a87e94edb674db4bbac2f30caa4d48
Change-Id: I505656bc61a87e94edb674db4bbac2f30caa4d48
-rw-r--r-- | tests/wifitests/src/com/android/server/wifi/WifiMetricsTest.java | 17 |
1 files changed, 17 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 65427beb2..fd21437d1 100644 --- a/tests/wifitests/src/com/android/server/wifi/WifiMetricsTest.java +++ b/tests/wifitests/src/com/android/server/wifi/WifiMetricsTest.java @@ -1142,6 +1142,7 @@ public class WifiMetricsTest { } } private void verifyDeserializedStaEvents(WifiMetricsProto.WifiLog wifiLog) { + assertNotNull(mTestWifiConfig); assertEquals(NUM_TEST_STA_EVENTS, wifiLog.staEventList.length); int j = 0; // De-serialized event index for (int i = 0; i < mTestStaMessageInts.length; i++) { @@ -1161,6 +1162,21 @@ public class WifiMetricsTest { j++; } } + for (int i = 0; i < mTestStaLogInts.length; i++) { + StaEvent event = wifiLog.staEventList[j]; + int[] evs = mExpectedValues[j]; + assertEquals(evs[0], event.type); + assertEquals(evs[1], event.reason); + assertEquals(evs[2], event.status); + assertEquals(evs[3] == 1 ? true : false, event.localGen); + assertEquals(evs[4], event.authFailureReason); + assertEquals(evs[5] == 1 ? true : false, event.associationTimedOut); + assertEquals(evs[6], event.supplicantStateChangesBitmask); + assertConfigInfoEqualsWifiConfig( + evs[7] == 1 ? mTestWifiConfig : null, event.configInfo); + j++; + } + assertEquals(mExpectedValues.length, j); } /** @@ -1274,6 +1290,7 @@ public class WifiMetricsTest { * Test Open Network Notification blacklist size and feature state are not cleared when proto * is dumped. */ + @Test public void testOpenNetworkNotificationBlacklistSizeAndFeatureStateNotCleared() throws Exception { mWifiMetrics.setOpenNetworkRecommenderBlacklistSize( |