bluetooth: mac: add connected LE devices to chooser

device/bluetooth/bluetooth_adapter_unittest.cc

 // Copyright 2014 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 "device/bluetooth/bluetooth_adapter.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <memory>
@@ skipping 847 matching lines @@
   device->CreateGattConnection(GetGattConnectionCallback(Call::EXPECTED),
                                GetConnectErrorCallback(Call::NOT_EXPECTED));
   SimulateGattConnection(device);
   EXPECT_EQ(1u, adapter_->GetDevices().size());
   RemoveTimedOutDevices();
   EXPECT_EQ(0, observer.device_removed_count());
   EXPECT_EQ(1u, adapter_->GetDevices().size());
 }
 #endif  // defined(OS_ANDROID) || defined(OS_MACOSX)
 
+#if defined(OS_MACOSX)
+// Simulate two devices being connected before starting discovery session with
+// no service filter.
+TEST_F(BluetoothTest, DiscoverConnectedLowEnergyDeviceWithNoFilter) {
+  if (!PlatformSupportsLowEnergy()) {
+    LOG(WARNING) << "Low Energy Bluetooth unavailable, skipping unit test.";
+    return;
+  }
+  InitWithFakeAdapter();
+  TestBluetoothAdapterObserver observer(adapter_);
+
+  BluetoothDevice::UUIDSet uuidSet1;
+  uuidSet1.insert(device::BluetoothUUID("1800"));
+  SimulateConnectedLowEnergyDevice(1, uuidSet1);
+  BluetoothDevice::UUIDSet uuidSet2;
+  uuidSet2.insert(device::BluetoothUUID("1800"));
+  uuidSet2.insert(device::BluetoothUUID("1001"));
+  SimulateConnectedLowEnergyDevice(2, uuidSet2);
+  BluetoothDiscoveryFilter discovery_filter(BLUETOOTH_TRANSPORT_LE);
+  adapter_->RetrieveGattConnectedDevicesWithDiscoveryFilter(&discovery_filter);
+
+  EXPECT_EQ(2, observer.device_added_count());
+  EXPECT_EQ(2u, adapter_->GetDevices().size());

[ortuno, 2016/10/31 04:30:51]

Also check that the return value is correct:

std::unordered_map<BluetoothDevice*, BluetoothDevice::UUIDSet> result =
adapter_->RetrieveGattConnectedDevicesWithDiscoveryFilter(...);

EXPECT_EQ(2u, result.size());
for (const auto& pair : result) {
  EXPECT_TRUE(adapter->GetDevice(pair->first->GetDeviceAddress()));
  EXPECT_TRUE(pair->second.empty());
}

[jlebel, 2016/11/07 01:43:18]

Done.

+}
+
+// Simulate two devices being connected before starting discovery session with
+// one service filter.
+TEST_F(BluetoothTest, DiscoverConnectedLowEnergyDeviceWithFilter) {
+  if (!PlatformSupportsLowEnergy()) {
+    LOG(WARNING) << "Low Energy Bluetooth unavailable, skipping unit test.";
+    return;
+  }
+  InitWithFakeAdapter();
+  TestBluetoothAdapterObserver observer(adapter_);
+
+  BluetoothDevice::UUIDSet uuidSet1;
+  uuidSet1.insert(device::BluetoothUUID("1800"));
+  SimulateConnectedLowEnergyDevice(1, uuidSet1);
+  BluetoothDevice::UUIDSet uuidSet2;
+  uuidSet2.insert(device::BluetoothUUID("1800"));
+  uuidSet2.insert(device::BluetoothUUID("1001"));
+  SimulateConnectedLowEnergyDevice(2, uuidSet2);
+  BluetoothDiscoveryFilter discovery_filter(BLUETOOTH_TRANSPORT_LE);
+  discovery_filter.AddUUID(device::BluetoothUUID("1001"));
+  adapter_->RetrieveGattConnectedDevicesWithDiscoveryFilter(&discovery_filter);
+
+  EXPECT_EQ(1, observer.device_added_count());
+  EXPECT_EQ(1u, adapter_->GetDevices().size());

[ortuno, 2016/10/31 04:30:51]

Same here, please test the return value is correct:

EXPECT_EQ(1u, result.size());
for (const auto& pair : result) {
  EXPECT_EQ("...", pair->first->GetDeviceAddress());
  EXPECT_TRUE(adapter->GetDevice(pair->first->GetDeviceAddress());
 
EXPECT_EQ(BluetoothDevice::UUIDSet({device::BluetoothUUID(kTestHeartRateService)}),
pair->second);
}

[jlebel, 2016/11/07 01:43:18]

Done.

+}
+
+// Simulate two devices being connected before starting discovery session with
+// one service filter that doesn't match.
+TEST_F(BluetoothTest, DiscoverConnectedLowEnergyDeviceWithWrongFilter) {
+  if (!PlatformSupportsLowEnergy()) {
+    LOG(WARNING) << "Low Energy Bluetooth unavailable, skipping unit test.";
+    return;
+  }
+  InitWithFakeAdapter();
+  TestBluetoothAdapterObserver observer(adapter_);
+
+  BluetoothDevice::UUIDSet uuidSet1;
+  uuidSet1.insert(device::BluetoothUUID("1800"));
+  SimulateConnectedLowEnergyDevice(1, uuidSet1);
+  BluetoothDevice::UUIDSet uuidSet2;
+  uuidSet2.insert(device::BluetoothUUID("1800"));
+  uuidSet2.insert(device::BluetoothUUID("1001"));
+  SimulateConnectedLowEnergyDevice(2, uuidSet2);
+  BluetoothDiscoveryFilter discovery_filter(BLUETOOTH_TRANSPORT_LE);
+  discovery_filter.AddUUID(device::BluetoothUUID("1002"));
+  adapter_->RetrieveGattConnectedDevicesWithDiscoveryFilter(&discovery_filter);

[ortuno, 2016/10/31 04:30:51]

Add:

EXPECT_TRUE(result.empty());

[jlebel, 2016/11/07 01:43:18]

Done.

+
+  EXPECT_EQ(0, observer.device_added_count());
+  EXPECT_EQ(0u, adapter_->GetDevices().size());
+}
+#endif  // defined(OS_MACOSX)

[ortuno, 2016/10/31 04:30:51]

Please add the following tests:

Filter with more than one UUID -> Test 1. A device that has only one of the
UUIDs is found and 2. that a device with both of the uuids is found and the
result has both the UUIDs.
RetrieveGattConnectedDevicesWithDiscoveryFilter called twice -> Tests we don't
create a new device.

[jlebel, 2016/11/07 01:43:18]

Done.

+
 }  // namespace device