Ajay Nadathur | 7d176bc | 2016-10-24 16:55:24 -0700 | [diff] [blame] | 1 | /* |
| 2 | * Copyright (C) 2016 The Android Open Source Project |
| 3 | * |
| 4 | * Licensed under the Apache License, Version 2.0 (the "License"); |
| 5 | * you may not use this file except in compliance with the License. |
| 6 | * You may obtain a copy of the License at |
| 7 | * |
| 8 | * http://www.apache.org/licenses/LICENSE-2.0 |
| 9 | * |
| 10 | * Unless required by applicable law or agreed to in writing, software |
| 11 | * distributed under the License is distributed on an "AS IS" BASIS, |
| 12 | * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 13 | * See the License for the specific language governing permissions and |
| 14 | * limitations under the License |
| 15 | */ |
| 16 | package com.android.settingslib.wifi; |
| 17 | |
Sundeep Ghuman | 2b48990 | 2017-02-22 18:17:29 -0800 | [diff] [blame] | 18 | import static com.google.common.truth.Truth.assertThat; |
| 19 | |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 20 | import static org.junit.Assert.assertEquals; |
Stephen Chen | 21f6868 | 2017-04-04 13:23:31 -0700 | [diff] [blame] | 21 | import static org.junit.Assert.assertFalse; |
Ajay Nadathur | 7d176bc | 2016-10-24 16:55:24 -0700 | [diff] [blame] | 22 | import static org.junit.Assert.assertTrue; |
Sundeep Ghuman | 71f4a82 | 2017-04-18 19:51:46 -0700 | [diff] [blame] | 23 | import static org.junit.Assert.fail; |
Peter Qiu | f562838 | 2017-03-28 13:24:34 -0700 | [diff] [blame] | 24 | import static org.mockito.Mockito.any; |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 25 | import static org.mockito.Mockito.anyInt; |
Sundeep Ghuman | 71f4a82 | 2017-04-18 19:51:46 -0700 | [diff] [blame] | 26 | import static org.mockito.Mockito.atMost; |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 27 | import static org.mockito.Mockito.doAnswer; |
| 28 | import static org.mockito.Mockito.doNothing; |
Peter Qiu | f562838 | 2017-03-28 13:24:34 -0700 | [diff] [blame] | 29 | import static org.mockito.Mockito.doThrow; |
Sundeep Ghuman | ce6bab8 | 2017-04-19 16:21:46 -0700 | [diff] [blame] | 30 | import static org.mockito.Mockito.mock; |
Sundeep Ghuman | d4a79ac | 2017-06-07 18:11:39 -0700 | [diff] [blame] | 31 | import static org.mockito.Mockito.never; |
Sundeep Ghuman | ce6bab8 | 2017-04-19 16:21:46 -0700 | [diff] [blame] | 32 | import static org.mockito.Mockito.times; |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 33 | import static org.mockito.Mockito.verify; |
Sundeep Ghuman | 71f4a82 | 2017-04-18 19:51:46 -0700 | [diff] [blame] | 34 | import static org.mockito.Mockito.verifyNoMoreInteractions; |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 35 | import static org.mockito.Mockito.when; |
| 36 | |
| 37 | import android.content.Context; |
| 38 | import android.content.Intent; |
| 39 | import android.net.ConnectivityManager; |
Sundeep Ghuman | 9c0b97e | 2017-04-13 16:05:46 -0700 | [diff] [blame] | 40 | import android.net.Network; |
Sundeep Ghuman | 2b48990 | 2017-02-22 18:17:29 -0800 | [diff] [blame] | 41 | import android.net.NetworkInfo; |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 42 | import android.net.NetworkKey; |
| 43 | import android.net.NetworkScoreManager; |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 44 | import android.net.RssiCurve; |
Sundeep Ghuman | 9c0b97e | 2017-04-13 16:05:46 -0700 | [diff] [blame] | 45 | import android.net.ScoredNetwork; |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 46 | import android.net.WifiKey; |
Ajay Nadathur | 7d176bc | 2016-10-24 16:55:24 -0700 | [diff] [blame] | 47 | import android.net.wifi.ScanResult; |
| 48 | import android.net.wifi.WifiConfiguration; |
Sundeep Ghuman | 2b48990 | 2017-02-22 18:17:29 -0800 | [diff] [blame] | 49 | import android.net.wifi.WifiInfo; |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 50 | import android.net.wifi.WifiManager; |
| 51 | import android.net.wifi.WifiNetworkScoreCache; |
| 52 | import android.net.wifi.WifiSsid; |
| 53 | import android.os.Bundle; |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 54 | import android.os.Handler; |
| 55 | import android.os.HandlerThread; |
| 56 | import android.os.Looper; |
Sundeep Ghuman | 9c0b97e | 2017-04-13 16:05:46 -0700 | [diff] [blame] | 57 | import android.os.SystemClock; |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 58 | import android.provider.Settings; |
Ajay Nadathur | 7d176bc | 2016-10-24 16:55:24 -0700 | [diff] [blame] | 59 | import android.support.test.InstrumentationRegistry; |
| 60 | import android.support.test.filters.SmallTest; |
| 61 | import android.support.test.runner.AndroidJUnit4; |
Sundeep Ghuman | 266b409 | 2017-06-16 15:12:56 -0700 | [diff] [blame] | 62 | import android.support.test.filters.FlakyTest; |
Ajay Nadathur | 7d176bc | 2016-10-24 16:55:24 -0700 | [diff] [blame] | 63 | |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 64 | import org.junit.After; |
| 65 | import org.junit.Before; |
Ajay Nadathur | 7d176bc | 2016-10-24 16:55:24 -0700 | [diff] [blame] | 66 | import org.junit.Test; |
| 67 | import org.junit.runner.RunWith; |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 68 | import org.mockito.ArgumentCaptor; |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 69 | import org.mockito.Captor; |
Sundeep Ghuman | 9c0b97e | 2017-04-13 16:05:46 -0700 | [diff] [blame] | 70 | import org.mockito.Matchers; |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 71 | import org.mockito.Mock; |
| 72 | import org.mockito.MockitoAnnotations; |
| 73 | import org.mockito.invocation.InvocationOnMock; |
| 74 | import org.mockito.stubbing.Answer; |
| 75 | |
Ajay Nadathur | 7d176bc | 2016-10-24 16:55:24 -0700 | [diff] [blame] | 76 | import java.util.ArrayList; |
| 77 | import java.util.Arrays; |
| 78 | import java.util.BitSet; |
| 79 | import java.util.List; |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 80 | import java.util.concurrent.CountDownLatch; |
| 81 | import java.util.concurrent.TimeUnit; |
Ajay Nadathur | 7d176bc | 2016-10-24 16:55:24 -0700 | [diff] [blame] | 82 | |
Sundeep Ghuman | 2b48990 | 2017-02-22 18:17:29 -0800 | [diff] [blame] | 83 | // TODO(sghuman): Change these to robolectric tests b/35766684. |
| 84 | |
Ajay Nadathur | 7d176bc | 2016-10-24 16:55:24 -0700 | [diff] [blame] | 85 | @SmallTest |
| 86 | @RunWith(AndroidJUnit4.class) |
| 87 | public class WifiTrackerTest { |
| 88 | |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 89 | private static final String TAG = "WifiTrackerTest"; |
Sundeep Ghuman | ac7b436 | 2017-02-08 17:19:27 -0800 | [diff] [blame] | 90 | private static final int LATCH_TIMEOUT = 4000; |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 91 | |
| 92 | private static final String SSID_1 = "ssid1"; |
| 93 | private static final String BSSID_1 = "00:00:00:00:00:00"; |
| 94 | private static final NetworkKey NETWORK_KEY_1 = |
| 95 | new NetworkKey(new WifiKey('"' + SSID_1 + '"', BSSID_1)); |
| 96 | private static final int RSSI_1 = -30; |
| 97 | private static final byte SCORE_1 = 10; |
Sundeep Ghuman | 53200ed | 2017-06-21 16:54:36 -0700 | [diff] [blame] | 98 | private static final int BADGE_1 = AccessPoint.Speed.MODERATE; |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 99 | |
| 100 | private static final String SSID_2 = "ssid2"; |
| 101 | private static final String BSSID_2 = "AA:AA:AA:AA:AA:AA"; |
| 102 | private static final NetworkKey NETWORK_KEY_2 = |
| 103 | new NetworkKey(new WifiKey('"' + SSID_2 + '"', BSSID_2)); |
| 104 | private static final int RSSI_2 = -30; |
| 105 | private static final byte SCORE_2 = 15; |
Sundeep Ghuman | 53200ed | 2017-06-21 16:54:36 -0700 | [diff] [blame] | 106 | private static final int BADGE_2 = AccessPoint.Speed.FAST; |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 107 | |
Sundeep Ghuman | 8c79288 | 2017-04-04 17:23:29 -0700 | [diff] [blame] | 108 | private static final int CONNECTED_NETWORK_ID = 123; |
| 109 | private static final int CONNECTED_RSSI = -50; |
| 110 | private static final WifiInfo CONNECTED_AP_1_INFO = new WifiInfo(); |
| 111 | static { |
| 112 | CONNECTED_AP_1_INFO.setSSID(WifiSsid.createFromAsciiEncoded(SSID_1)); |
| 113 | CONNECTED_AP_1_INFO.setBSSID(BSSID_1); |
| 114 | CONNECTED_AP_1_INFO.setNetworkId(CONNECTED_NETWORK_ID); |
| 115 | CONNECTED_AP_1_INFO.setRssi(CONNECTED_RSSI); |
| 116 | } |
| 117 | |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 118 | @Captor ArgumentCaptor<WifiNetworkScoreCache> mScoreCacheCaptor; |
| 119 | @Mock private ConnectivityManager mockConnectivityManager; |
| 120 | @Mock private NetworkScoreManager mockNetworkScoreManager; |
| 121 | @Mock private RssiCurve mockCurve1; |
| 122 | @Mock private RssiCurve mockCurve2; |
| 123 | @Mock private RssiCurve mockBadgeCurve1; |
| 124 | @Mock private RssiCurve mockBadgeCurve2; |
| 125 | @Mock private WifiManager mockWifiManager; |
| 126 | @Mock private WifiTracker.WifiListener mockWifiListener; |
| 127 | |
| 128 | private final List<NetworkKey> mRequestedKeys = new ArrayList<>(); |
| 129 | |
| 130 | private Context mContext; |
| 131 | private CountDownLatch mAccessPointsChangedLatch; |
| 132 | private CountDownLatch mRequestScoresLatch; |
| 133 | private Handler mScannerHandler; |
| 134 | private HandlerThread mMainThread; |
| 135 | private HandlerThread mWorkerThread; |
Sundeep Ghuman | 71f4a82 | 2017-04-18 19:51:46 -0700 | [diff] [blame] | 136 | private Looper mWorkerLooper; |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 137 | private Looper mMainLooper; |
Sundeep Ghuman | 4205874 | 2017-07-21 18:42:10 -0700 | [diff] [blame] | 138 | |
Sundeep Ghuman | e869d83 | 2017-01-25 16:23:43 -0800 | [diff] [blame] | 139 | private int mOriginalScoringUiSettingValue; |
Sundeep Ghuman | 4205874 | 2017-07-21 18:42:10 -0700 | [diff] [blame] | 140 | private boolean mOriginalStaleScanResultsValue; |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 141 | |
| 142 | @Before |
| 143 | public void setUp() { |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 144 | MockitoAnnotations.initMocks(this); |
| 145 | |
Sundeep Ghuman | 2b48990 | 2017-02-22 18:17:29 -0800 | [diff] [blame] | 146 | mContext = InstrumentationRegistry.getTargetContext(); |
| 147 | |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 148 | mWorkerThread = new HandlerThread("TestHandlerWorkerThread"); |
| 149 | mWorkerThread.start(); |
Sundeep Ghuman | 71f4a82 | 2017-04-18 19:51:46 -0700 | [diff] [blame] | 150 | mWorkerLooper = mWorkerThread.getLooper(); |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 151 | mMainThread = new HandlerThread("TestHandlerThread"); |
| 152 | mMainThread.start(); |
| 153 | mMainLooper = mMainThread.getLooper(); |
| 154 | |
| 155 | // Make sure the scanner doesn't try to run on the testing thread. |
| 156 | HandlerThread scannerThread = new HandlerThread("ScannerWorkerThread"); |
| 157 | scannerThread.start(); |
| 158 | mScannerHandler = new Handler(scannerThread.getLooper()); |
| 159 | |
| 160 | when(mockWifiManager.isWifiEnabled()).thenReturn(true); |
| 161 | when(mockWifiManager.getScanResults()) |
| 162 | .thenReturn(Arrays.asList(buildScanResult1(), buildScanResult2())); |
| 163 | |
| 164 | |
| 165 | when(mockCurve1.lookupScore(RSSI_1)).thenReturn(SCORE_1); |
| 166 | when(mockCurve2.lookupScore(RSSI_2)).thenReturn(SCORE_2); |
| 167 | |
| 168 | when(mockBadgeCurve1.lookupScore(RSSI_1)).thenReturn((byte) BADGE_1); |
| 169 | when(mockBadgeCurve2.lookupScore(RSSI_2)).thenReturn((byte) BADGE_2); |
| 170 | |
| 171 | doNothing() |
| 172 | .when(mockNetworkScoreManager) |
| 173 | .registerNetworkScoreCache( |
| 174 | anyInt(), |
| 175 | mScoreCacheCaptor.capture(), |
| 176 | Matchers.anyInt()); |
| 177 | |
| 178 | // Capture requested keys and count down latch if present |
| 179 | doAnswer( |
| 180 | new Answer<Boolean>() { |
| 181 | @Override |
| 182 | public Boolean answer(InvocationOnMock input) { |
| 183 | if (mRequestScoresLatch != null) { |
| 184 | mRequestScoresLatch.countDown(); |
| 185 | } |
| 186 | NetworkKey[] keys = (NetworkKey[]) input.getArguments()[0]; |
| 187 | for (NetworkKey key : keys) { |
| 188 | mRequestedKeys.add(key); |
| 189 | } |
| 190 | return true; |
| 191 | } |
| 192 | }).when(mockNetworkScoreManager).requestScores(Matchers.<NetworkKey[]>any()); |
| 193 | |
| 194 | doAnswer( |
| 195 | new Answer<Void>() { |
| 196 | @Override |
| 197 | public Void answer (InvocationOnMock invocation) throws Throwable { |
| 198 | if (mAccessPointsChangedLatch != null) { |
| 199 | mAccessPointsChangedLatch.countDown(); |
| 200 | } |
| 201 | |
| 202 | return null; |
| 203 | } |
| 204 | }).when(mockWifiListener).onAccessPointsChanged(); |
| 205 | |
Sundeep Ghuman | e869d83 | 2017-01-25 16:23:43 -0800 | [diff] [blame] | 206 | // Turn on Scoring UI features |
| 207 | mOriginalScoringUiSettingValue = Settings.Global.getInt( |
| 208 | InstrumentationRegistry.getTargetContext().getContentResolver(), |
| 209 | Settings.Global.NETWORK_SCORING_UI_ENABLED, |
| 210 | 0 /* disabled */); |
| 211 | Settings.Global.putInt( |
| 212 | InstrumentationRegistry.getTargetContext().getContentResolver(), |
| 213 | Settings.Global.NETWORK_SCORING_UI_ENABLED, |
| 214 | 1 /* enabled */); |
Sundeep Ghuman | 4205874 | 2017-07-21 18:42:10 -0700 | [diff] [blame] | 215 | |
| 216 | mOriginalStaleScanResultsValue = WifiTracker.sStaleScanResults; |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 217 | } |
| 218 | |
| 219 | @After |
| 220 | public void cleanUp() { |
| 221 | Settings.Global.putInt( |
Sundeep Ghuman | e869d83 | 2017-01-25 16:23:43 -0800 | [diff] [blame] | 222 | InstrumentationRegistry.getTargetContext().getContentResolver(), |
| 223 | Settings.Global.NETWORK_SCORING_UI_ENABLED, |
| 224 | mOriginalScoringUiSettingValue); |
Sundeep Ghuman | 4205874 | 2017-07-21 18:42:10 -0700 | [diff] [blame] | 225 | |
| 226 | WifiTracker.sStaleScanResults = mOriginalStaleScanResultsValue; |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 227 | } |
| 228 | |
| 229 | private static ScanResult buildScanResult1() { |
| 230 | return new ScanResult( |
| 231 | WifiSsid.createFromAsciiEncoded(SSID_1), |
| 232 | BSSID_1, |
| 233 | 0, // hessid |
| 234 | 0, //anqpDomainId |
| 235 | null, // osuProviders |
| 236 | "", // capabilities |
| 237 | RSSI_1, |
| 238 | 0, // frequency |
| 239 | SystemClock.elapsedRealtime() * 1000 /* microsecond timestamp */); |
| 240 | } |
| 241 | |
| 242 | private static ScanResult buildScanResult2() { |
| 243 | return new ScanResult( |
| 244 | WifiSsid.createFromAsciiEncoded(SSID_2), |
| 245 | BSSID_2, |
| 246 | 0, // hessid |
| 247 | 0, //anqpDomainId |
| 248 | null, // osuProviders |
| 249 | "", // capabilities |
| 250 | RSSI_2, |
| 251 | 0, // frequency |
| 252 | SystemClock.elapsedRealtime() * 1000 /* microsecond timestamp */); |
| 253 | } |
| 254 | |
Sundeep Ghuman | 2b48990 | 2017-02-22 18:17:29 -0800 | [diff] [blame] | 255 | private WifiTracker createTrackerWithImmediateBroadcastsAndInjectInitialScanResults( |
| 256 | Intent ... intents) |
| 257 | throws InterruptedException { |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 258 | WifiTracker tracker = createMockedWifiTracker(); |
Sundeep Ghuman | 2b48990 | 2017-02-22 18:17:29 -0800 | [diff] [blame] | 259 | |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 260 | startTracking(tracker); |
Sundeep Ghuman | 2b48990 | 2017-02-22 18:17:29 -0800 | [diff] [blame] | 261 | for (Intent intent : intents) { |
| 262 | tracker.mReceiver.onReceive(mContext, intent); |
| 263 | } |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 264 | |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 265 | sendScanResultsAndProcess(tracker); |
Sundeep Ghuman | e801309 | 2017-06-21 22:35:30 -0700 | [diff] [blame] | 266 | waitForHandlersToProcessCurrentlyEnqueuedMessages(tracker); |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 267 | |
| 268 | return tracker; |
| 269 | } |
| 270 | |
| 271 | private WifiTracker createMockedWifiTracker() { |
| 272 | WifiTracker tracker = |
| 273 | new WifiTracker( |
Sundeep Ghuman | 2b48990 | 2017-02-22 18:17:29 -0800 | [diff] [blame] | 274 | mContext, |
| 275 | mockWifiListener, |
Sundeep Ghuman | 71f4a82 | 2017-04-18 19:51:46 -0700 | [diff] [blame] | 276 | mWorkerLooper, |
Sundeep Ghuman | 2b48990 | 2017-02-22 18:17:29 -0800 | [diff] [blame] | 277 | true, |
| 278 | true, |
| 279 | true, |
| 280 | mockWifiManager, |
| 281 | mockConnectivityManager, |
| 282 | mockNetworkScoreManager, |
| 283 | mMainLooper |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 284 | ); |
| 285 | |
| 286 | return tracker; |
| 287 | } |
| 288 | |
| 289 | private void startTracking(WifiTracker tracker) throws InterruptedException { |
| 290 | CountDownLatch latch = new CountDownLatch(1); |
| 291 | mScannerHandler.post(new Runnable() { |
| 292 | @Override |
| 293 | public void run() { |
| 294 | tracker.startTracking(); |
| 295 | latch.countDown(); |
| 296 | } |
| 297 | }); |
Sundeep Ghuman | ac7b436 | 2017-02-08 17:19:27 -0800 | [diff] [blame] | 298 | assertTrue("Latch timed out", latch.await(LATCH_TIMEOUT, TimeUnit.MILLISECONDS)); |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 299 | } |
| 300 | |
| 301 | private void sendScanResultsAndProcess(WifiTracker tracker) throws InterruptedException { |
| 302 | mAccessPointsChangedLatch = new CountDownLatch(1); |
| 303 | Intent i = new Intent(WifiManager.SCAN_RESULTS_AVAILABLE_ACTION); |
| 304 | tracker.mReceiver.onReceive(mContext, i); |
| 305 | |
Sundeep Ghuman | ac7b436 | 2017-02-08 17:19:27 -0800 | [diff] [blame] | 306 | assertTrue("Latch timed out", |
| 307 | mAccessPointsChangedLatch.await(LATCH_TIMEOUT, TimeUnit.MILLISECONDS)); |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 308 | } |
| 309 | |
| 310 | private void updateScores() { |
| 311 | Bundle attr1 = new Bundle(); |
| 312 | attr1.putParcelable(ScoredNetwork.ATTRIBUTES_KEY_BADGING_CURVE, mockBadgeCurve1); |
| 313 | ScoredNetwork sc1 = |
| 314 | new ScoredNetwork( |
| 315 | NETWORK_KEY_1, |
| 316 | mockCurve1, |
| 317 | false /* meteredHint */, |
| 318 | attr1); |
| 319 | |
| 320 | Bundle attr2 = new Bundle(); |
| 321 | attr2.putParcelable(ScoredNetwork.ATTRIBUTES_KEY_BADGING_CURVE, mockBadgeCurve2); |
| 322 | ScoredNetwork sc2 = |
| 323 | new ScoredNetwork( |
| 324 | NETWORK_KEY_2, |
| 325 | mockCurve2, |
Stephen Chen | 21f6868 | 2017-04-04 13:23:31 -0700 | [diff] [blame] | 326 | true /* meteredHint */, |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 327 | attr2); |
| 328 | |
| 329 | WifiNetworkScoreCache scoreCache = mScoreCacheCaptor.getValue(); |
| 330 | scoreCache.updateScores(Arrays.asList(sc1, sc2)); |
| 331 | } |
| 332 | |
Sundeep Ghuman | 2b48990 | 2017-02-22 18:17:29 -0800 | [diff] [blame] | 333 | private WifiTracker createTrackerWithScanResultsAndAccessPoint1Connected() |
| 334 | throws InterruptedException { |
Sundeep Ghuman | 8c79288 | 2017-04-04 17:23:29 -0700 | [diff] [blame] | 335 | when(mockWifiManager.getConnectionInfo()).thenReturn(CONNECTED_AP_1_INFO); |
Sundeep Ghuman | 2b48990 | 2017-02-22 18:17:29 -0800 | [diff] [blame] | 336 | |
| 337 | WifiConfiguration configuration = new WifiConfiguration(); |
| 338 | configuration.SSID = SSID_1; |
| 339 | configuration.BSSID = BSSID_1; |
Sundeep Ghuman | 8c79288 | 2017-04-04 17:23:29 -0700 | [diff] [blame] | 340 | configuration.networkId = CONNECTED_NETWORK_ID; |
Sundeep Ghuman | 2b48990 | 2017-02-22 18:17:29 -0800 | [diff] [blame] | 341 | when(mockWifiManager.getConfiguredNetworks()).thenReturn(Arrays.asList(configuration)); |
| 342 | |
| 343 | NetworkInfo networkInfo = new NetworkInfo( |
| 344 | ConnectivityManager.TYPE_WIFI, 0, "Type Wifi", "subtype"); |
| 345 | networkInfo.setDetailedState(NetworkInfo.DetailedState.CONNECTED, "connected", "test"); |
| 346 | |
| 347 | Intent intent = new Intent(WifiManager.NETWORK_STATE_CHANGED_ACTION); |
| 348 | intent.putExtra(WifiManager.EXTRA_NETWORK_INFO, networkInfo); |
Sundeep Ghuman | db2eabb | 2017-07-28 14:51:05 -0700 | [diff] [blame] | 349 | WifiTracker tracker = |
| 350 | createTrackerWithImmediateBroadcastsAndInjectInitialScanResults(intent); |
| 351 | assertThat(tracker.isConnected()).isTrue(); |
| 352 | return tracker; |
Sundeep Ghuman | 2b48990 | 2017-02-22 18:17:29 -0800 | [diff] [blame] | 353 | } |
| 354 | |
Sundeep Ghuman | e801309 | 2017-06-21 22:35:30 -0700 | [diff] [blame] | 355 | private void waitForHandlersToProcessCurrentlyEnqueuedMessages(WifiTracker tracker) |
| 356 | throws InterruptedException { |
| 357 | CountDownLatch workerLatch = new CountDownLatch(1); |
| 358 | tracker.mWorkHandler.post(() -> { |
| 359 | workerLatch.countDown(); |
| 360 | }); |
| 361 | assertTrue("Latch timed out while waiting for WorkerHandler", |
| 362 | workerLatch.await(LATCH_TIMEOUT, TimeUnit.MILLISECONDS)); |
| 363 | |
| 364 | CountDownLatch mainLatch = new CountDownLatch(1); |
| 365 | tracker.mMainHandler.post(() -> { |
| 366 | mainLatch.countDown(); |
| 367 | }); |
| 368 | assertTrue("Latch timed out while waiting for MainHandler", |
| 369 | mainLatch.await(LATCH_TIMEOUT, TimeUnit.MILLISECONDS)); |
| 370 | } |
| 371 | |
Ajay Nadathur | 7d176bc | 2016-10-24 16:55:24 -0700 | [diff] [blame] | 372 | @Test |
| 373 | public void testAccessPointListenerSetWhenLookingUpUsingScanResults() { |
| 374 | ScanResult scanResult = new ScanResult(); |
| 375 | scanResult.level = 123; |
| 376 | scanResult.BSSID = "bssid-" + 111; |
| 377 | scanResult.timestamp = SystemClock.elapsedRealtime() * 1000; |
| 378 | scanResult.capabilities = ""; |
| 379 | |
| 380 | WifiTracker tracker = new WifiTracker( |
Sundeep Ghuman | 71f4a82 | 2017-04-18 19:51:46 -0700 | [diff] [blame] | 381 | InstrumentationRegistry.getTargetContext(), null, mWorkerLooper, true, true); |
Ajay Nadathur | 7d176bc | 2016-10-24 16:55:24 -0700 | [diff] [blame] | 382 | |
| 383 | AccessPoint result = tracker.getCachedOrCreate(scanResult, new ArrayList<AccessPoint>()); |
| 384 | assertTrue(result.mAccessPointListener != null); |
| 385 | } |
| 386 | |
| 387 | @Test |
| 388 | public void testAccessPointListenerSetWhenLookingUpUsingWifiConfiguration() { |
| 389 | WifiConfiguration configuration = new WifiConfiguration(); |
| 390 | configuration.SSID = "test123"; |
| 391 | configuration.BSSID="bssid"; |
| 392 | configuration.networkId = 123; |
| 393 | configuration.allowedKeyManagement = new BitSet(); |
| 394 | configuration.allowedKeyManagement.set(WifiConfiguration.KeyMgmt.WPA_PSK); |
| 395 | |
| 396 | WifiTracker tracker = new WifiTracker( |
Sundeep Ghuman | 71f4a82 | 2017-04-18 19:51:46 -0700 | [diff] [blame] | 397 | InstrumentationRegistry.getTargetContext(), null, mWorkerLooper, true, true); |
Ajay Nadathur | 7d176bc | 2016-10-24 16:55:24 -0700 | [diff] [blame] | 398 | |
| 399 | AccessPoint result = tracker.getCachedOrCreate(configuration, new ArrayList<AccessPoint>()); |
| 400 | assertTrue(result.mAccessPointListener != null); |
| 401 | } |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 402 | |
| 403 | @Test |
| 404 | public void startAndStopTrackingShouldRegisterAndUnregisterScoreCache() |
| 405 | throws InterruptedException { |
| 406 | WifiTracker tracker = createMockedWifiTracker(); |
| 407 | |
| 408 | // Test register |
| 409 | startTracking(tracker); |
| 410 | verify(mockNetworkScoreManager) |
| 411 | .registerNetworkScoreCache( |
| 412 | Matchers.anyInt(), |
| 413 | mScoreCacheCaptor.capture(), |
| 414 | Matchers.anyInt()); |
| 415 | |
| 416 | WifiNetworkScoreCache scoreCache = mScoreCacheCaptor.getValue(); |
| 417 | |
Sundeep Ghuman | e869d83 | 2017-01-25 16:23:43 -0800 | [diff] [blame] | 418 | CountDownLatch latch = new CountDownLatch(1); |
| 419 | doAnswer( |
| 420 | (invocation) -> { |
| 421 | latch.countDown(); |
| 422 | return null; |
| 423 | }).when(mockNetworkScoreManager) |
| 424 | .unregisterNetworkScoreCache(NetworkKey.TYPE_WIFI, scoreCache); |
| 425 | |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 426 | // Test unregister |
| 427 | tracker.stopTracking(); |
| 428 | |
Sundeep Ghuman | ac7b436 | 2017-02-08 17:19:27 -0800 | [diff] [blame] | 429 | assertTrue("Latch timed out", latch.await(LATCH_TIMEOUT, TimeUnit.MILLISECONDS)); |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 430 | verify(mockNetworkScoreManager) |
| 431 | .unregisterNetworkScoreCache(NetworkKey.TYPE_WIFI, scoreCache); |
| 432 | } |
| 433 | |
| 434 | @Test |
Amin Shaikh | a80ae0c | 2017-03-23 16:18:30 -0700 | [diff] [blame] | 435 | public void testGetNumSavedNetworks() throws InterruptedException { |
| 436 | WifiConfiguration validConfig = new WifiConfiguration(); |
| 437 | validConfig.SSID = SSID_1; |
| 438 | validConfig.BSSID = BSSID_1; |
| 439 | |
| 440 | WifiConfiguration selfAddedNoAssociation = new WifiConfiguration(); |
Peter Qiu | abea726 | 2017-05-31 10:18:17 -0700 | [diff] [blame] | 441 | selfAddedNoAssociation.ephemeral = true; |
Amin Shaikh | a80ae0c | 2017-03-23 16:18:30 -0700 | [diff] [blame] | 442 | selfAddedNoAssociation.selfAdded = true; |
| 443 | selfAddedNoAssociation.numAssociation = 0; |
| 444 | selfAddedNoAssociation.SSID = SSID_2; |
| 445 | selfAddedNoAssociation.BSSID = BSSID_2; |
| 446 | |
| 447 | when(mockWifiManager.getConfiguredNetworks()) |
| 448 | .thenReturn(Arrays.asList(validConfig, selfAddedNoAssociation)); |
| 449 | |
| 450 | WifiTracker tracker = createTrackerWithImmediateBroadcastsAndInjectInitialScanResults(); |
| 451 | |
| 452 | assertEquals(1, tracker.getNumSavedNetworks()); |
| 453 | } |
| 454 | |
| 455 | @Test |
Sundeep Ghuman | 2b48990 | 2017-02-22 18:17:29 -0800 | [diff] [blame] | 456 | public void startTrackingShouldSetConnectedAccessPointAsActive() throws InterruptedException { |
Sundeep Ghuman | e801309 | 2017-06-21 22:35:30 -0700 | [diff] [blame] | 457 | WifiTracker tracker = createTrackerWithScanResultsAndAccessPoint1Connected(); |
Sundeep Ghuman | 2b48990 | 2017-02-22 18:17:29 -0800 | [diff] [blame] | 458 | |
| 459 | List<AccessPoint> aps = tracker.getAccessPoints(); |
| 460 | |
| 461 | assertThat(aps).hasSize(2); |
| 462 | assertThat(aps.get(0).isActive()).isTrue(); |
| 463 | } |
| 464 | |
| 465 | @Test |
Sundeep Ghuman | 71f4a82 | 2017-04-18 19:51:46 -0700 | [diff] [blame] | 466 | public void startTrackingAfterStopTracking_shouldRequestNewScores() |
| 467 | throws InterruptedException { |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 468 | // Start the tracker and inject the initial scan results and then stop tracking |
Sundeep Ghuman | 2b48990 | 2017-02-22 18:17:29 -0800 | [diff] [blame] | 469 | WifiTracker tracker = createTrackerWithImmediateBroadcastsAndInjectInitialScanResults(); |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 470 | |
| 471 | tracker.stopTracking(); |
| 472 | mRequestedKeys.clear(); |
| 473 | |
Sundeep Ghuman | ac7b436 | 2017-02-08 17:19:27 -0800 | [diff] [blame] | 474 | mRequestScoresLatch = new CountDownLatch(1); |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 475 | startTracking(tracker); |
Sundeep Ghuman | 71f4a82 | 2017-04-18 19:51:46 -0700 | [diff] [blame] | 476 | tracker.forceUpdate(); |
Sundeep Ghuman | ac7b436 | 2017-02-08 17:19:27 -0800 | [diff] [blame] | 477 | assertTrue("Latch timed out", |
| 478 | mRequestScoresLatch.await(LATCH_TIMEOUT, TimeUnit.MILLISECONDS)); |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 479 | |
| 480 | assertTrue(mRequestedKeys.contains(NETWORK_KEY_1)); |
| 481 | assertTrue(mRequestedKeys.contains(NETWORK_KEY_2)); |
| 482 | } |
| 483 | |
| 484 | @Test |
Sundeep Ghuman | 75f9480 | 2017-06-26 16:05:40 -0700 | [diff] [blame] | 485 | public void stopTracking_shouldNotClearExistingScores() |
| 486 | throws InterruptedException { |
| 487 | // Start the tracker and inject the initial scan results and then stop tracking |
| 488 | WifiTracker tracker = createTrackerWithImmediateBroadcastsAndInjectInitialScanResults(); |
| 489 | updateScoresAndWaitForAccessPointsChangedCallback(tracker); |
| 490 | tracker.stopTracking(); |
| 491 | |
| 492 | assertThat(mScoreCacheCaptor.getValue().getScoredNetwork(NETWORK_KEY_1)).isNotNull(); |
| 493 | } |
| 494 | |
| 495 | @Test |
Sundeep Ghuman | 8c79288 | 2017-04-04 17:23:29 -0700 | [diff] [blame] | 496 | public void scoreCacheUpdateScoresShouldTriggerOnAccessPointsChanged() |
| 497 | throws InterruptedException { |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 498 | WifiTracker tracker = createMockedWifiTracker(); |
| 499 | startTracking(tracker); |
| 500 | sendScanResultsAndProcess(tracker); |
| 501 | |
Sundeep Ghuman | e801309 | 2017-06-21 22:35:30 -0700 | [diff] [blame] | 502 | updateScoresAndWaitForAccessPointsChangedCallback(tracker); |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 503 | } |
| 504 | |
Sundeep Ghuman | e801309 | 2017-06-21 22:35:30 -0700 | [diff] [blame] | 505 | private void updateScoresAndWaitForAccessPointsChangedCallback(WifiTracker tracker) |
| 506 | throws InterruptedException { |
Sundeep Ghuman | ac7b436 | 2017-02-08 17:19:27 -0800 | [diff] [blame] | 507 | // Updating scores can happen together or one after the other, so the latch countdown is set |
| 508 | // to 2. |
Sundeep Ghuman | e801309 | 2017-06-21 22:35:30 -0700 | [diff] [blame] | 509 | mAccessPointsChangedLatch = new CountDownLatch(1); |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 510 | updateScores(); |
Sundeep Ghuman | e801309 | 2017-06-21 22:35:30 -0700 | [diff] [blame] | 511 | assertTrue("onAccessPointChanged was not called after updating scores", |
Sundeep Ghuman | ce6bab8 | 2017-04-19 16:21:46 -0700 | [diff] [blame] | 512 | mAccessPointsChangedLatch.await(LATCH_TIMEOUT, TimeUnit.MILLISECONDS)); |
Sundeep Ghuman | e801309 | 2017-06-21 22:35:30 -0700 | [diff] [blame] | 513 | waitForHandlersToProcessCurrentlyEnqueuedMessages(tracker); |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 514 | } |
| 515 | |
Sundeep Ghuman | 266b409 | 2017-06-16 15:12:56 -0700 | [diff] [blame] | 516 | @FlakyTest |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 517 | @Test |
| 518 | public void scoreCacheUpdateScoresShouldChangeSortOrder() throws InterruptedException { |
Sundeep Ghuman | 8c79288 | 2017-04-04 17:23:29 -0700 | [diff] [blame] | 519 | WifiTracker tracker = createTrackerWithImmediateBroadcastsAndInjectInitialScanResults(); |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 520 | List<AccessPoint> aps = tracker.getAccessPoints(); |
| 521 | assertTrue(aps.size() == 2); |
| 522 | assertEquals(aps.get(0).getSsidStr(), SSID_1); |
| 523 | assertEquals(aps.get(1).getSsidStr(), SSID_2); |
| 524 | |
Sundeep Ghuman | e801309 | 2017-06-21 22:35:30 -0700 | [diff] [blame] | 525 | updateScoresAndWaitForAccessPointsChangedCallback(tracker); |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 526 | |
| 527 | aps = tracker.getAccessPoints(); |
| 528 | assertTrue(aps.size() == 2); |
| 529 | assertEquals(aps.get(0).getSsidStr(), SSID_2); |
| 530 | assertEquals(aps.get(1).getSsidStr(), SSID_1); |
Sundeep Ghuman | e869d83 | 2017-01-25 16:23:43 -0800 | [diff] [blame] | 531 | } |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 532 | |
Sundeep Ghuman | e869d83 | 2017-01-25 16:23:43 -0800 | [diff] [blame] | 533 | @Test |
| 534 | public void scoreCacheUpdateScoresShouldNotChangeSortOrderWhenSortingDisabled() |
| 535 | throws InterruptedException { |
| 536 | Settings.Global.putInt( |
| 537 | InstrumentationRegistry.getTargetContext().getContentResolver(), |
| 538 | Settings.Global.NETWORK_SCORING_UI_ENABLED, |
| 539 | 0 /* disabled */); |
| 540 | |
Sundeep Ghuman | 2b48990 | 2017-02-22 18:17:29 -0800 | [diff] [blame] | 541 | WifiTracker tracker = createTrackerWithImmediateBroadcastsAndInjectInitialScanResults(); |
Sundeep Ghuman | e869d83 | 2017-01-25 16:23:43 -0800 | [diff] [blame] | 542 | List<AccessPoint> aps = tracker.getAccessPoints(); |
| 543 | assertTrue(aps.size() == 2); |
| 544 | assertEquals(aps.get(0).getSsidStr(), SSID_1); |
| 545 | assertEquals(aps.get(1).getSsidStr(), SSID_2); |
| 546 | |
Sundeep Ghuman | e801309 | 2017-06-21 22:35:30 -0700 | [diff] [blame] | 547 | updateScoresAndWaitForAccessPointsChangedCallback(tracker); |
Sundeep Ghuman | e869d83 | 2017-01-25 16:23:43 -0800 | [diff] [blame] | 548 | |
| 549 | aps = tracker.getAccessPoints(); |
| 550 | assertTrue(aps.size() == 2); |
| 551 | assertEquals(aps.get(0).getSsidStr(), SSID_1); |
| 552 | assertEquals(aps.get(1).getSsidStr(), SSID_2); |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 553 | } |
| 554 | |
Sundeep Ghuman | 266b409 | 2017-06-16 15:12:56 -0700 | [diff] [blame] | 555 | @FlakyTest |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 556 | @Test |
Sundeep Ghuman | 55adc6b | 2017-06-05 16:46:59 -0700 | [diff] [blame] | 557 | public void scoreCacheUpdateScoresShouldInsertSpeedIntoAccessPoint() |
| 558 | throws InterruptedException { |
Sundeep Ghuman | 2b48990 | 2017-02-22 18:17:29 -0800 | [diff] [blame] | 559 | WifiTracker tracker = createTrackerWithImmediateBroadcastsAndInjectInitialScanResults(); |
Sundeep Ghuman | e801309 | 2017-06-21 22:35:30 -0700 | [diff] [blame] | 560 | updateScoresAndWaitForAccessPointsChangedCallback(tracker); |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 561 | |
| 562 | List<AccessPoint> aps = tracker.getAccessPoints(); |
| 563 | |
| 564 | for (AccessPoint ap : aps) { |
| 565 | if (ap.getSsidStr().equals(SSID_1)) { |
Sundeep Ghuman | 55adc6b | 2017-06-05 16:46:59 -0700 | [diff] [blame] | 566 | assertEquals(BADGE_1, ap.getSpeed()); |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 567 | } else if (ap.getSsidStr().equals(SSID_2)) { |
Sundeep Ghuman | 55adc6b | 2017-06-05 16:46:59 -0700 | [diff] [blame] | 568 | assertEquals(BADGE_2, ap.getSpeed()); |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 569 | } |
| 570 | } |
| 571 | } |
| 572 | |
| 573 | @Test |
Stephen Chen | 21f6868 | 2017-04-04 13:23:31 -0700 | [diff] [blame] | 574 | public void scoreCacheUpdateMeteredShouldUpdateAccessPointMetering() |
| 575 | throws InterruptedException { |
| 576 | WifiTracker tracker = createTrackerWithImmediateBroadcastsAndInjectInitialScanResults(); |
Sundeep Ghuman | e801309 | 2017-06-21 22:35:30 -0700 | [diff] [blame] | 577 | updateScoresAndWaitForAccessPointsChangedCallback(tracker); |
Stephen Chen | 21f6868 | 2017-04-04 13:23:31 -0700 | [diff] [blame] | 578 | |
| 579 | List<AccessPoint> aps = tracker.getAccessPoints(); |
| 580 | |
| 581 | for (AccessPoint ap : aps) { |
| 582 | if (ap.getSsidStr().equals(SSID_1)) { |
| 583 | assertFalse(ap.isMetered()); |
| 584 | } else if (ap.getSsidStr().equals(SSID_2)) { |
| 585 | assertTrue(ap.isMetered()); |
| 586 | } |
| 587 | } |
| 588 | } |
| 589 | |
| 590 | @Test |
Sundeep Ghuman | 55adc6b | 2017-06-05 16:46:59 -0700 | [diff] [blame] | 591 | public void noSpeedsShouldBeInsertedIntoAccessPointWhenScoringUiDisabled() |
Sundeep Ghuman | e869d83 | 2017-01-25 16:23:43 -0800 | [diff] [blame] | 592 | throws InterruptedException { |
| 593 | Settings.Global.putInt( |
| 594 | InstrumentationRegistry.getTargetContext().getContentResolver(), |
| 595 | Settings.Global.NETWORK_SCORING_UI_ENABLED, |
| 596 | 0 /* disabled */); |
| 597 | |
Sundeep Ghuman | 2b48990 | 2017-02-22 18:17:29 -0800 | [diff] [blame] | 598 | WifiTracker tracker = createTrackerWithImmediateBroadcastsAndInjectInitialScanResults(); |
Sundeep Ghuman | e801309 | 2017-06-21 22:35:30 -0700 | [diff] [blame] | 599 | updateScoresAndWaitForAccessPointsChangedCallback(tracker); |
Sundeep Ghuman | e869d83 | 2017-01-25 16:23:43 -0800 | [diff] [blame] | 600 | |
| 601 | List<AccessPoint> aps = tracker.getAccessPoints(); |
| 602 | |
| 603 | for (AccessPoint ap : aps) { |
| 604 | if (ap.getSsidStr().equals(SSID_1)) { |
Sundeep Ghuman | 53200ed | 2017-06-21 16:54:36 -0700 | [diff] [blame] | 605 | assertEquals(AccessPoint.Speed.NONE, ap.getSpeed()); |
Sundeep Ghuman | e869d83 | 2017-01-25 16:23:43 -0800 | [diff] [blame] | 606 | } else if (ap.getSsidStr().equals(SSID_2)) { |
Sundeep Ghuman | 53200ed | 2017-06-21 16:54:36 -0700 | [diff] [blame] | 607 | assertEquals(AccessPoint.Speed.NONE, ap.getSpeed()); |
Sundeep Ghuman | e869d83 | 2017-01-25 16:23:43 -0800 | [diff] [blame] | 608 | } |
| 609 | } |
| 610 | } |
| 611 | |
| 612 | @Test |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 613 | public void scoresShouldBeRequestedForNewScanResultOnly() throws InterruptedException { |
Sundeep Ghuman | ac7b436 | 2017-02-08 17:19:27 -0800 | [diff] [blame] | 614 | // Scores can be requested together or serially depending on how the scan results are |
| 615 | // processed. |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 616 | mRequestScoresLatch = new CountDownLatch(2); |
Sundeep Ghuman | 2b48990 | 2017-02-22 18:17:29 -0800 | [diff] [blame] | 617 | WifiTracker tracker = createTrackerWithImmediateBroadcastsAndInjectInitialScanResults(); |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 618 | mRequestScoresLatch.await(LATCH_TIMEOUT, TimeUnit.MILLISECONDS); |
| 619 | mRequestedKeys.clear(); |
| 620 | |
| 621 | String ssid = "ssid3"; |
| 622 | String bssid = "00:00:00:00:00:00"; |
| 623 | ScanResult newResult = new ScanResult( |
| 624 | WifiSsid.createFromAsciiEncoded(ssid), |
| 625 | bssid, |
| 626 | 0, // hessid |
| 627 | 0, //anqpDomainId |
| 628 | null, // osuProviders |
| 629 | "", // capabilities |
| 630 | RSSI_1, |
| 631 | 0, // frequency |
| 632 | SystemClock.elapsedRealtime() * 1000); |
| 633 | when(mockWifiManager.getScanResults()) |
| 634 | .thenReturn(Arrays.asList(buildScanResult1(), buildScanResult2(), newResult)); |
| 635 | |
| 636 | mRequestScoresLatch = new CountDownLatch(1); |
| 637 | sendScanResultsAndProcess(tracker); |
Sundeep Ghuman | ac7b436 | 2017-02-08 17:19:27 -0800 | [diff] [blame] | 638 | assertTrue(mRequestScoresLatch.await(LATCH_TIMEOUT, TimeUnit.MILLISECONDS)); |
Sundeep Ghuman | 5519b7b | 2016-12-14 17:53:31 -0800 | [diff] [blame] | 639 | |
| 640 | assertEquals(1, mRequestedKeys.size()); |
| 641 | assertTrue(mRequestedKeys.contains(new NetworkKey(new WifiKey('"' + ssid + '"', bssid)))); |
| 642 | } |
Sundeep Ghuman | ac7b436 | 2017-02-08 17:19:27 -0800 | [diff] [blame] | 643 | |
| 644 | @Test |
| 645 | public void scoreCacheAndListenerShouldBeUnregisteredWhenStopTrackingIsCalled() throws Exception |
| 646 | { |
Sundeep Ghuman | 2b48990 | 2017-02-22 18:17:29 -0800 | [diff] [blame] | 647 | WifiTracker tracker = createTrackerWithImmediateBroadcastsAndInjectInitialScanResults(); |
Sundeep Ghuman | ac7b436 | 2017-02-08 17:19:27 -0800 | [diff] [blame] | 648 | WifiNetworkScoreCache cache = mScoreCacheCaptor.getValue(); |
| 649 | |
| 650 | tracker.stopTracking(); |
| 651 | verify(mockNetworkScoreManager).unregisterNetworkScoreCache(NetworkKey.TYPE_WIFI, cache); |
| 652 | |
| 653 | // Verify listener is unregistered so updating a score does not throw an error by posting |
| 654 | // a message to the dead work handler |
| 655 | mWorkerThread.quit(); |
| 656 | updateScores(); |
| 657 | } |
Peter Qiu | f562838 | 2017-03-28 13:24:34 -0700 | [diff] [blame] | 658 | |
| 659 | /** |
| 660 | * Verify that tracking a Passpoint AP on a device with Passpoint disabled doesn't cause |
| 661 | * any crash. |
| 662 | * |
| 663 | * @throws Exception |
| 664 | */ |
| 665 | @Test |
| 666 | public void trackPasspointApWithPasspointDisabled() throws Exception { |
| 667 | WifiTracker tracker = createMockedWifiTracker(); |
| 668 | |
| 669 | // Add a Passpoint AP to the scan results. |
| 670 | List<ScanResult> results = new ArrayList<>(); |
| 671 | ScanResult passpointAp = new ScanResult( |
| 672 | WifiSsid.createFromAsciiEncoded(SSID_1), |
| 673 | BSSID_1, |
| 674 | 0, // hessid |
| 675 | 0, //anqpDomainId |
| 676 | null, // osuProviders |
| 677 | "", // capabilities |
| 678 | RSSI_1, |
| 679 | 0, // frequency |
| 680 | SystemClock.elapsedRealtime() * 1000 /* microsecond timestamp */); |
| 681 | passpointAp.setFlag(ScanResult.FLAG_PASSPOINT_NETWORK); |
| 682 | results.add(passpointAp); |
| 683 | |
| 684 | // Update access point and verify UnsupportedOperationException is being caught for |
| 685 | // call to WifiManager#getMatchingWifiConfig. |
| 686 | when(mockWifiManager.getConfiguredNetworks()) |
| 687 | .thenReturn(new ArrayList<WifiConfiguration>()); |
| 688 | when(mockWifiManager.getScanResults()).thenReturn(results); |
| 689 | doThrow(new UnsupportedOperationException()) |
| 690 | .when(mockWifiManager).getMatchingWifiConfig(any(ScanResult.class)); |
| 691 | tracker.forceUpdate(); |
| 692 | verify(mockWifiManager).getMatchingWifiConfig(any(ScanResult.class)); |
| 693 | } |
Sundeep Ghuman | 8c79288 | 2017-04-04 17:23:29 -0700 | [diff] [blame] | 694 | |
| 695 | @Test |
| 696 | public void rssiChangeBroadcastShouldUpdateConnectedAp() throws Exception { |
| 697 | WifiTracker tracker = createTrackerWithScanResultsAndAccessPoint1Connected(); |
| 698 | assertThat(tracker.getAccessPoints().get(0).isActive()).isTrue(); |
| 699 | |
Sundeep Ghuman | 8c79288 | 2017-04-04 17:23:29 -0700 | [diff] [blame] | 700 | int newRssi = CONNECTED_RSSI + 10; |
| 701 | WifiInfo info = new WifiInfo(CONNECTED_AP_1_INFO); |
| 702 | info.setRssi(newRssi); |
Sundeep Ghuman | 8c79288 | 2017-04-04 17:23:29 -0700 | [diff] [blame] | 703 | |
Sundeep Ghuman | 71f4a82 | 2017-04-18 19:51:46 -0700 | [diff] [blame] | 704 | CountDownLatch latch = new CountDownLatch(1); |
| 705 | |
| 706 | // Once the new info has been fetched, we need to wait for the access points to be copied |
| 707 | doAnswer(invocation -> { |
| 708 | latch.countDown(); |
| 709 | mAccessPointsChangedLatch = new CountDownLatch(1); |
| 710 | return info; |
| 711 | }).when(mockWifiManager).getConnectionInfo(); |
| 712 | |
Sundeep Ghuman | 8c79288 | 2017-04-04 17:23:29 -0700 | [diff] [blame] | 713 | tracker.mReceiver.onReceive(mContext, new Intent(WifiManager.RSSI_CHANGED_ACTION)); |
Sundeep Ghuman | 71f4a82 | 2017-04-18 19:51:46 -0700 | [diff] [blame] | 714 | assertTrue("New connection info never retrieved", |
| 715 | latch.await(LATCH_TIMEOUT, TimeUnit.MILLISECONDS)); |
| 716 | assertTrue("onAccessPointsChanged never called", |
| 717 | mAccessPointsChangedLatch.await(LATCH_TIMEOUT, TimeUnit.MILLISECONDS)); |
Sundeep Ghuman | 8c79288 | 2017-04-04 17:23:29 -0700 | [diff] [blame] | 718 | |
Sundeep Ghuman | 8c79288 | 2017-04-04 17:23:29 -0700 | [diff] [blame] | 719 | assertThat(tracker.getAccessPoints().get(0).getRssi()).isEqualTo(newRssi); |
| 720 | } |
Sundeep Ghuman | 9c0b97e | 2017-04-13 16:05:46 -0700 | [diff] [blame] | 721 | |
| 722 | @Test |
| 723 | public void forceUpdateShouldSynchronouslyFetchLatestInformation() throws Exception { |
Sundeep Ghuman | ce6bab8 | 2017-04-19 16:21:46 -0700 | [diff] [blame] | 724 | Network mockNetwork = mock(Network.class); |
| 725 | when(mockWifiManager.getCurrentNetwork()).thenReturn(mockNetwork); |
Sundeep Ghuman | 71f4a82 | 2017-04-18 19:51:46 -0700 | [diff] [blame] | 726 | |
Sundeep Ghuman | 9c0b97e | 2017-04-13 16:05:46 -0700 | [diff] [blame] | 727 | when(mockWifiManager.getConnectionInfo()).thenReturn(CONNECTED_AP_1_INFO); |
| 728 | |
| 729 | WifiConfiguration configuration = new WifiConfiguration(); |
| 730 | configuration.SSID = SSID_1; |
| 731 | configuration.BSSID = BSSID_1; |
| 732 | configuration.networkId = CONNECTED_NETWORK_ID; |
| 733 | when(mockWifiManager.getConfiguredNetworks()).thenReturn(Arrays.asList(configuration)); |
| 734 | |
| 735 | NetworkInfo networkInfo = new NetworkInfo( |
| 736 | ConnectivityManager.TYPE_WIFI, 0, "Type Wifi", "subtype"); |
| 737 | networkInfo.setDetailedState(NetworkInfo.DetailedState.CONNECTED, "connected", "test"); |
| 738 | when(mockConnectivityManager.getNetworkInfo(any(Network.class))).thenReturn(networkInfo); |
| 739 | |
Sundeep Ghuman | 9c0b97e | 2017-04-13 16:05:46 -0700 | [diff] [blame] | 740 | WifiTracker tracker = createMockedWifiTracker(); |
Sundeep Ghuman | 9c0b97e | 2017-04-13 16:05:46 -0700 | [diff] [blame] | 741 | tracker.forceUpdate(); |
| 742 | |
Sundeep Ghuman | ce6bab8 | 2017-04-19 16:21:46 -0700 | [diff] [blame] | 743 | verify(mockWifiManager).getConnectionInfo(); |
| 744 | verify(mockWifiManager, times(2)).getConfiguredNetworks(); |
| 745 | verify(mockConnectivityManager).getNetworkInfo(any(Network.class)); |
| 746 | |
Sundeep Ghuman | d2b84a8 | 2017-07-05 14:19:30 -0700 | [diff] [blame] | 747 | verify(mockWifiListener, never()).onAccessPointsChanged(); // mStaleAccessPoints is true |
Sundeep Ghuman | 9c0b97e | 2017-04-13 16:05:46 -0700 | [diff] [blame] | 748 | assertThat(tracker.getAccessPoints().size()).isEqualTo(2); |
| 749 | assertThat(tracker.getAccessPoints().get(0).isActive()).isTrue(); |
| 750 | } |
Sundeep Ghuman | 71f4a82 | 2017-04-18 19:51:46 -0700 | [diff] [blame] | 751 | |
| 752 | @Test |
| 753 | public void stopTrackingShouldRemoveWifiListenerCallbacks() throws Exception { |
| 754 | WifiTracker tracker = createMockedWifiTracker(); |
| 755 | startTracking(tracker); |
| 756 | |
| 757 | CountDownLatch latch = new CountDownLatch(1); |
| 758 | CountDownLatch lock = new CountDownLatch(1); |
| 759 | tracker.mMainHandler.post(() -> { |
| 760 | try { |
| 761 | lock.await(); |
| 762 | latch.countDown(); |
| 763 | } catch (InterruptedException e) { |
| 764 | fail("Interrupted Exception while awaiting lock release: " + e); |
| 765 | } |
| 766 | }); |
| 767 | |
| 768 | // Enqueue messages |
| 769 | tracker.mMainHandler.sendEmptyMessage( |
| 770 | WifiTracker.MainHandler.MSG_ACCESS_POINT_CHANGED); |
| 771 | tracker.mMainHandler.sendEmptyMessage( |
| 772 | WifiTracker.MainHandler.MSG_CONNECTED_CHANGED); |
| 773 | tracker.mMainHandler.sendEmptyMessage( |
| 774 | WifiTracker.MainHandler.MSG_WIFI_STATE_CHANGED); |
| 775 | |
| 776 | tracker.stopTracking(); |
| 777 | |
| 778 | verify(mockWifiListener, atMost(1)).onAccessPointsChanged(); |
| 779 | verify(mockWifiListener, atMost(1)).onConnectedChanged(); |
| 780 | verify(mockWifiListener, atMost(1)).onWifiStateChanged(anyInt()); |
| 781 | |
| 782 | lock.countDown(); |
Sundeep Ghuman | ce6bab8 | 2017-04-19 16:21:46 -0700 | [diff] [blame] | 783 | assertTrue("Latch timed out", latch.await(LATCH_TIMEOUT, TimeUnit.MILLISECONDS)); |
Sundeep Ghuman | 71f4a82 | 2017-04-18 19:51:46 -0700 | [diff] [blame] | 784 | |
| 785 | assertThat(tracker.mMainHandler.hasMessages( |
| 786 | WifiTracker.MainHandler.MSG_ACCESS_POINT_CHANGED)).isFalse(); |
| 787 | assertThat(tracker.mMainHandler.hasMessages( |
| 788 | WifiTracker.MainHandler.MSG_CONNECTED_CHANGED)).isFalse(); |
| 789 | assertThat(tracker.mMainHandler.hasMessages( |
| 790 | WifiTracker.MainHandler.MSG_WIFI_STATE_CHANGED)).isFalse(); |
| 791 | |
| 792 | verifyNoMoreInteractions(mockWifiListener); |
| 793 | } |
Sundeep Ghuman | d4a79ac | 2017-06-07 18:11:39 -0700 | [diff] [blame] | 794 | |
| 795 | @Test |
| 796 | public void stopTrackingShouldSetStaleBitWhichPreventsCallbacksUntilNextScanResult() |
| 797 | throws Exception { |
| 798 | WifiTracker tracker = createMockedWifiTracker(); |
| 799 | startTracking(tracker); |
Sundeep Ghuman | e801309 | 2017-06-21 22:35:30 -0700 | [diff] [blame] | 800 | waitForHandlersToProcessCurrentlyEnqueuedMessages(tracker); |
Sundeep Ghuman | d4a79ac | 2017-06-07 18:11:39 -0700 | [diff] [blame] | 801 | |
Sundeep Ghuman | e801309 | 2017-06-21 22:35:30 -0700 | [diff] [blame] | 802 | tracker.stopTracking(); |
| 803 | waitForHandlersToProcessCurrentlyEnqueuedMessages(tracker); |
Sundeep Ghuman | d4a79ac | 2017-06-07 18:11:39 -0700 | [diff] [blame] | 804 | |
| 805 | startTracking(tracker); |
| 806 | |
| 807 | tracker.mReceiver.onReceive(mContext, new Intent(WifiManager.WIFI_STATE_CHANGED_ACTION)); |
| 808 | tracker.mReceiver.onReceive( |
| 809 | mContext, new Intent(WifiManager.CONFIGURED_NETWORKS_CHANGED_ACTION)); |
| 810 | tracker.mReceiver.onReceive( |
| 811 | mContext, new Intent(WifiManager.LINK_CONFIGURATION_CHANGED_ACTION)); |
| 812 | |
Sundeep Ghuman | e801309 | 2017-06-21 22:35:30 -0700 | [diff] [blame] | 813 | waitForHandlersToProcessCurrentlyEnqueuedMessages(tracker); |
Sundeep Ghuman | d4a79ac | 2017-06-07 18:11:39 -0700 | [diff] [blame] | 814 | |
| 815 | verify(mockWifiListener, never()).onAccessPointsChanged(); |
| 816 | |
| 817 | sendScanResultsAndProcess(tracker); // verifies onAccessPointsChanged is invoked |
| 818 | } |
Sundeep Ghuman | e801309 | 2017-06-21 22:35:30 -0700 | [diff] [blame] | 819 | |
| 820 | @Test |
Sundeep Ghuman | d2b84a8 | 2017-07-05 14:19:30 -0700 | [diff] [blame] | 821 | public void startTrackingShouldNotSendAnyCallbacksUntilScanResultsAreProcessed() |
| 822 | throws Exception { |
| 823 | WifiTracker tracker = createMockedWifiTracker(); |
| 824 | startTracking(tracker); |
| 825 | waitForHandlersToProcessCurrentlyEnqueuedMessages(tracker); |
| 826 | |
| 827 | tracker.mReceiver.onReceive(mContext, new Intent(WifiManager.WIFI_STATE_CHANGED_ACTION)); |
| 828 | tracker.mReceiver.onReceive( |
| 829 | mContext, new Intent(WifiManager.CONFIGURED_NETWORKS_CHANGED_ACTION)); |
| 830 | tracker.mReceiver.onReceive( |
| 831 | mContext, new Intent(WifiManager.LINK_CONFIGURATION_CHANGED_ACTION)); |
| 832 | |
| 833 | waitForHandlersToProcessCurrentlyEnqueuedMessages(tracker); |
| 834 | verify(mockWifiListener, never()).onAccessPointsChanged(); |
| 835 | |
| 836 | sendScanResultsAndProcess(tracker); // verifies onAccessPointsChanged is invoked |
| 837 | } |
| 838 | |
| 839 | @Test |
Sundeep Ghuman | e801309 | 2017-06-21 22:35:30 -0700 | [diff] [blame] | 840 | public void disablingWifiShouldClearExistingAccessPoints() throws Exception { |
| 841 | WifiTracker tracker = createTrackerWithScanResultsAndAccessPoint1Connected(); |
| 842 | |
| 843 | when(mockWifiManager.isWifiEnabled()).thenReturn(false); |
| 844 | mAccessPointsChangedLatch = new CountDownLatch(1); |
| 845 | tracker.mReceiver.onReceive(mContext, new Intent(WifiManager.WIFI_STATE_CHANGED_ACTION)); |
| 846 | |
| 847 | mAccessPointsChangedLatch.await(LATCH_TIMEOUT, TimeUnit.MILLISECONDS); |
| 848 | waitForHandlersToProcessCurrentlyEnqueuedMessages(tracker); |
| 849 | |
| 850 | assertThat(tracker.getAccessPoints()).isEmpty(); |
| 851 | } |
Sundeep Ghuman | 4205874 | 2017-07-21 18:42:10 -0700 | [diff] [blame] | 852 | |
| 853 | @Test |
| 854 | public void onConnectedChangedCallback_shouldNotBeInvokedWhenNoStateChange() throws Exception { |
| 855 | WifiTracker tracker = createTrackerWithScanResultsAndAccessPoint1Connected(); |
| 856 | verify(mockWifiListener, times(1)).onConnectedChanged(); |
| 857 | |
| 858 | NetworkInfo networkInfo = new NetworkInfo( |
| 859 | ConnectivityManager.TYPE_WIFI, 0, "Type Wifi", "subtype"); |
| 860 | networkInfo.setDetailedState(NetworkInfo.DetailedState.CONNECTED, "connected", "test"); |
| 861 | |
| 862 | Intent intent = new Intent(WifiManager.NETWORK_STATE_CHANGED_ACTION); |
| 863 | intent.putExtra(WifiManager.EXTRA_NETWORK_INFO, networkInfo); |
| 864 | tracker.mReceiver.onReceive(mContext, intent); |
| 865 | |
Sundeep Ghuman | db2eabb | 2017-07-28 14:51:05 -0700 | [diff] [blame] | 866 | waitForHandlersToProcessCurrentlyEnqueuedMessages(tracker); |
Sundeep Ghuman | 4205874 | 2017-07-21 18:42:10 -0700 | [diff] [blame] | 867 | verify(mockWifiListener, times(1)).onConnectedChanged(); |
| 868 | } |
| 869 | |
| 870 | @Test |
Sundeep Ghuman | db2eabb | 2017-07-28 14:51:05 -0700 | [diff] [blame] | 871 | public void onConnectedChangedCallback_shouldBeInvokedWhenStateChanges() throws Exception { |
Sundeep Ghuman | 4205874 | 2017-07-21 18:42:10 -0700 | [diff] [blame] | 872 | WifiTracker tracker = createTrackerWithScanResultsAndAccessPoint1Connected(); |
| 873 | verify(mockWifiListener, times(1)).onConnectedChanged(); |
| 874 | |
| 875 | NetworkInfo networkInfo = new NetworkInfo( |
| 876 | ConnectivityManager.TYPE_WIFI, 0, "Type Wifi", "subtype"); |
| 877 | networkInfo.setDetailedState( |
Sundeep Ghuman | db2eabb | 2017-07-28 14:51:05 -0700 | [diff] [blame] | 878 | NetworkInfo.DetailedState.DISCONNECTED, "disconnected", "test"); |
Sundeep Ghuman | 4205874 | 2017-07-21 18:42:10 -0700 | [diff] [blame] | 879 | |
| 880 | Intent intent = new Intent(WifiManager.NETWORK_STATE_CHANGED_ACTION); |
| 881 | intent.putExtra(WifiManager.EXTRA_NETWORK_INFO, networkInfo); |
| 882 | tracker.mReceiver.onReceive(mContext, intent); |
| 883 | |
Sundeep Ghuman | db2eabb | 2017-07-28 14:51:05 -0700 | [diff] [blame] | 884 | waitForHandlersToProcessCurrentlyEnqueuedMessages(tracker); |
| 885 | assertThat(tracker.isConnected()).isFalse(); |
Sundeep Ghuman | 4205874 | 2017-07-21 18:42:10 -0700 | [diff] [blame] | 886 | verify(mockWifiListener, times(2)).onConnectedChanged(); |
| 887 | } |
Peter Qiu | abea726 | 2017-05-31 10:18:17 -0700 | [diff] [blame] | 888 | } |