Merge "[WifiScoreReport] Don't report score if RSSI is invalid" into pi-dev

1 files changed, 15 insertions, 0 deletions

diff --git a/tests/wifitests/src/com/android/server/wifi/WifiScoreReportTest.java b/tests/wifitests/src/com/android/server/wifi/WifiScoreReportTest.java
index 38e45d306..3e9e9217c 100644
--- a/tests/wifitests/src/com/android/server/wifi/WifiScoreReportTest.java
+++ b/tests/wifitests/src/com/android/server/wifi/WifiScoreReportTest.java
@@ -25,6 +25,7 @@ import static org.mockito.Mockito.anyInt;
 import static org.mockito.Mockito.atLeast;
 import static org.mockito.Mockito.atMost;
 import static org.mockito.Mockito.doAnswer;
+import static org.mockito.Mockito.never;
 import static org.mockito.Mockito.times;
 import static org.mockito.Mockito.verify;
 import static org.mockito.Mockito.when;
@@ -160,6 +161,20 @@ public class WifiScoreReportTest {
     }
 
     /**
+     * Test for no score report if rssi is invalid
+     *
+     * The score should be sent to neither the NetworkAgent nor the
+     * WifiMetrics
+     */
+    @Test
+    public void calculateAndReportScoreDoesNotReportWhenRssiIsNotValid() throws Exception {
+        mWifiInfo.setRssi(WifiInfo.INVALID_RSSI);
+        mWifiScoreReport.calculateAndReportScore(mWifiInfo, mNetworkAgent, mWifiMetrics);
+        verify(mNetworkAgent, never()).sendNetworkScore(anyInt());
+        verify(mWifiMetrics, never()).incrementWifiScoreCount(anyInt());
+    }
+
+    /**
      * Test for operation with null NetworkAgent
      *
      * Expect to not die, and to calculate the score and report to metrics.