bluetooth: Return int8_t and use -128 for unknown tx power.

components/proximity_auth/proximity_monitor_impl_unittest.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/proximity_auth/proximity_monitor_impl.h"
 
 #include <memory>
 #include <utility>
 
 #include "base/macros.h"
 #include "base/memory/ptr_util.h"
 #include "base/memory/ref_counted.h"
+#include "base/optional.h"
 #include "base/test/histogram_tester.h"
 #include "base/test/simple_test_tick_clock.h"
 #include "base/test/test_simple_task_runner.h"
 #include "base/thread_task_runner_handle.h"
 #include "base/time/time.h"
 #include "components/proximity_auth/logging/logging.h"
 #include "components/proximity_auth/proximity_monitor_observer.h"
 #include "components/proximity_auth/remote_device.h"
 #include "device/bluetooth/bluetooth_adapter_factory.h"
 #include "device/bluetooth/test/mock_bluetooth_adapter.h"
@@ skipping 206 matching lines @@
   EXPECT_TRUE(monitor_.IsUnlockAllowed());
   EXPECT_TRUE(monitor_.IsInRssiRange());
 }
 
 TEST_F(ProximityAuthProximityMonitorImplTest,
        ProximityState_CheckRssi_UnknownRssi) {
   monitor_.SetStrategy(ProximityMonitor::Strategy::CHECK_RSSI);
   monitor_.Start();
 
   ProvideConnectionInfo({0, 0, 4});
-  ProvideConnectionInfo({BluetoothDevice::kUnknownPower, 0, 4});
+  ProvideConnectionInfo({base::nullopt, 0, 4});
 
   EXPECT_FALSE(monitor_.IsUnlockAllowed());
   EXPECT_FALSE(monitor_.IsInRssiRange());
 }
 
 TEST_F(ProximityAuthProximityMonitorImplTest,
        ProximityState_CheckRssi_UnknownTxPower) {
   monitor_.SetStrategy(ProximityMonitor::Strategy::CHECK_RSSI);
   monitor_.Start();
 
   ProvideConnectionInfo({0, 0, 4});
-  ProvideConnectionInfo({0, BluetoothDevice::kUnknownPower, 4});
+  ProvideConnectionInfo({0, base::nullopt, 4});
 
   EXPECT_FALSE(monitor_.IsUnlockAllowed());
   EXPECT_FALSE(monitor_.IsInRssiRange());
 }
 
 TEST_F(ProximityAuthProximityMonitorImplTest,
        ProximityState_CheckRssi_UnknownMaxTxPower) {
   monitor_.SetStrategy(ProximityMonitor::Strategy::CHECK_RSSI);
   monitor_.Start();
 
   ProvideConnectionInfo({0, 0, 4});
-  ProvideConnectionInfo({0, 0, BluetoothDevice::kUnknownPower});
+  ProvideConnectionInfo({0, 0, base::nullopt});
 
   EXPECT_FALSE(monitor_.IsUnlockAllowed());
   EXPECT_FALSE(monitor_.IsInRssiRange());
 }
 
 TEST_F(ProximityAuthProximityMonitorImplTest,
        ProximityState_CheckTxPower_RssiIndicatesProximity_TxPowerDoesNot) {
   monitor_.SetStrategy(ProximityMonitor::Strategy::CHECK_TRANSMIT_POWER);
   monitor_.Start();
 
@@ skipping 35 matching lines @@
   EXPECT_TRUE(monitor_.IsUnlockAllowed());
   EXPECT_TRUE(monitor_.IsInRssiRange());
 }
 
 TEST_F(ProximityAuthProximityMonitorImplTest,
        ProximityState_CheckTxPower_UnknownRssi) {
   monitor_.SetStrategy(ProximityMonitor::Strategy::CHECK_TRANSMIT_POWER);
   monitor_.Start();
 
   ProvideConnectionInfo({0, 0, 4});
-  ProvideConnectionInfo({BluetoothDevice::kUnknownPower, 0, 4});
+  ProvideConnectionInfo({base::nullopt, 0, 4});
 
   EXPECT_FALSE(monitor_.IsUnlockAllowed());
   EXPECT_FALSE(monitor_.IsInRssiRange());
 }
 
 TEST_F(ProximityAuthProximityMonitorImplTest,
        ProximityState_CheckTxPower_UnknownTxPower) {
   monitor_.SetStrategy(ProximityMonitor::Strategy::CHECK_TRANSMIT_POWER);
   monitor_.Start();
 
   ProvideConnectionInfo({0, 0, 4});
-  ProvideConnectionInfo({0, BluetoothDevice::kUnknownPower, 4});
+  ProvideConnectionInfo({0, base::nullopt, 4});
 
   EXPECT_FALSE(monitor_.IsUnlockAllowed());
   EXPECT_FALSE(monitor_.IsInRssiRange());
 }
 
 TEST_F(ProximityAuthProximityMonitorImplTest,
        ProximityState_CheckTxPower_UnknownMaxTxPower) {
   monitor_.SetStrategy(ProximityMonitor::Strategy::CHECK_TRANSMIT_POWER);
   monitor_.Start();
 
   ProvideConnectionInfo({0, 0, 4});
-  ProvideConnectionInfo({0, 0, BluetoothDevice::kUnknownPower});
+  ProvideConnectionInfo({0, 0, base::nullopt});
 
   EXPECT_FALSE(monitor_.IsUnlockAllowed());
   EXPECT_FALSE(monitor_.IsInRssiRange());
 }
 
 TEST_F(ProximityAuthProximityMonitorImplTest, ProximityState_StartThenStop) {
   monitor_.SetStrategy(ProximityMonitor::Strategy::CHECK_RSSI);
 
   monitor_.Start();
   ProvideConnectionInfo({0, 0, 4});
@@ skipping 163 matching lines @@
   histogram_tester.ExpectUniqueSample(
       "EasyUnlock.AuthProximity.TransmitPowerDelta", -1, 1);
   histogram_tester.ExpectUniqueSample(
       "EasyUnlock.AuthProximity.TimeSinceLastZeroRssi", 304, 1);
   histogram_tester.ExpectUniqueSample(
       "EasyUnlock.AuthProximity.RemoteDeviceModelHash",
       1881443083 /* hash of "LGE Nexus 5" */, 1);
 }
 
 TEST_F(ProximityAuthProximityMonitorImplTest,
-       RecordProximityMetricsOnAuthSuccess_ClampedValues) {
+       RecordProximityMetricsOnAuthSuccess_ClampedValues_LowerThreshold) {
+  // Tests that we clamp values lower than -100 before histograming.
+
+  int8_t rssi = -128;
+  int8_t tx_power = -128;
+  int8_t max_tx_power = 0;
+  // The resulting TransmitPowerDelta that we try to record is -128.
+
   monitor_.Start();
-  ProvideConnectionInfo({-99999, 99999, 12345});
+  ProvideConnectionInfo({rssi, tx_power, max_tx_power});
 
   base::HistogramTester histogram_tester;
   monitor_.RecordProximityMetricsOnAuthSuccess();
   histogram_tester.ExpectUniqueSample("EasyUnlock.AuthProximity.RollingRssi",
                                       -100, 1);
   histogram_tester.ExpectUniqueSample(
+      "EasyUnlock.AuthProximity.TransmitPowerDelta", -100, 1);
+}
+
+TEST_F(ProximityAuthProximityMonitorImplTest,
+       RecordProximityMetricsOnAuthSuccess_ClampedValues_UpperThreshold) {
+  // Tests that we clamp values larger than 50 before histograming.
+
+  int8_t rssi = 126;
+  int8_t tx_power = 126;
+  int8_t max_tx_power = 0;
+  // The resulting TransmitPowerDelta that we try to record is 126.
+
+  monitor_.Start();
+  ProvideConnectionInfo({rssi, tx_power, max_tx_power});
+
+  base::HistogramTester histogram_tester;
+  monitor_.RecordProximityMetricsOnAuthSuccess();
+  histogram_tester.ExpectUniqueSample("EasyUnlock.AuthProximity.RollingRssi",
+                                      50, 1);
+  histogram_tester.ExpectUniqueSample(
       "EasyUnlock.AuthProximity.TransmitPowerDelta", 50, 1);
 }
 
 TEST_F(ProximityAuthProximityMonitorImplTest,
        RecordProximityMetricsOnAuthSuccess_UnknownValues) {
   // Note: A device without a recorded name will have its Bluetooth address as
   // its name.
   RemoteDevice unnamed_remote_device(
       kRemoteDeviceUserId, kBluetoothAddress, kRemoteDevicePublicKey,
       RemoteDevice::BLUETOOTH_CLASSIC, kBluetoothAddress,
       kPersistentSymmetricKey, std::string());
 
   std::unique_ptr<base::TickClock> clock(new base::SimpleTestTickClock());
   ProximityMonitorImpl monitor(unnamed_remote_device, std::move(clock));
   monitor.AddObserver(&observer_);
   monitor.Start();
-  ProvideConnectionInfo({127, 127, 127});
+  ProvideConnectionInfo({base::nullopt, base::nullopt, base::nullopt});
 
   base::HistogramTester histogram_tester;
   monitor.RecordProximityMetricsOnAuthSuccess();
   histogram_tester.ExpectUniqueSample("EasyUnlock.AuthProximity.RollingRssi",
                                       127, 1);
   histogram_tester.ExpectUniqueSample(
       "EasyUnlock.AuthProximity.TransmitPowerDelta", 127, 1);
   histogram_tester.ExpectUniqueSample(
       "EasyUnlock.AuthProximity.TimeSinceLastZeroRssi",
       base::TimeDelta::FromSeconds(10).InMilliseconds(), 1);
   histogram_tester.ExpectUniqueSample(
       "EasyUnlock.AuthProximity.RemoteDeviceModelHash",
       -1808066424 /* hash of "Unknown" */, 1);
 }
 
 }  // namespace proximity_auth