From 13c598050aeb5b2f1293ea08b82eadd347142b91 Mon Sep 17 00:00:00 2001 From: Etan Cohen Date: Wed, 16 May 2018 15:44:42 -0700 Subject: [PATCH] [RTT] Add sanity checks for RTT ranging results Add checks to validate that the reported number of measurements and of successes is reasonable: non-zero for successful results. Bug: 79879905 Test: 'atest WifiRttTest' passes Change-Id: Ie79b10edacb660dd8c5d116539a4ee5cc32a779d --- tests/cts/net/src/android/net/wifi/rtt/cts/WifiRttTest.java | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/tests/cts/net/src/android/net/wifi/rtt/cts/WifiRttTest.java b/tests/cts/net/src/android/net/wifi/rtt/cts/WifiRttTest.java index 5995bbd32f..1171a6b7de 100644 --- a/tests/cts/net/src/android/net/wifi/rtt/cts/WifiRttTest.java +++ b/tests/cts/net/src/android/net/wifi/rtt/cts/WifiRttTest.java @@ -165,6 +165,10 @@ public class WifiRttTest extends TestBase { (distanceMax - distanceAvg) <= MAX_VARIATION_FROM_AVERAGE_DISTANCE_MM); assertTrue("Wi-Fi RTT: Variation (min direction) exceeds threshold", (distanceAvg - distanceMin) <= MAX_VARIATION_FROM_AVERAGE_DISTANCE_MM); + for (int i = 0; i < numGoodResults; ++i) { + assertNotSame("Number of attempted measurements is 0", 0, numAttempted[i]); + assertNotSame("Number of successful measurements is 0", 0, numSuccessful[i]); + } } }