bluetooth: android: Create BluetoothRemoteGattServiceAndroid objects.

device/bluetooth/bluetooth_device_unittest.cc

Index: device/bluetooth/bluetooth_device_unittest.cc
diff --git a/device/bluetooth/bluetooth_device_unittest.cc b/device/bluetooth/bluetooth_device_unittest.cc
index 2aa9688e0168f6370bd2457b8d04d9ec16751764..7a43df6d93fde9bd91f77a4f6f54405d93662c8b 100644
--- a/device/bluetooth/bluetooth_device_unittest.cc
+++ b/device/bluetooth/bluetooth_device_unittest.cc
@@ -425,4 +425,38 @@ TEST_F(BluetoothTest, BluetoothGattConnection_ErrorAfterConnection) {
 }
 #endif  // defined(OS_ANDROID)
 
+#if defined(OS_ANDROID)
+TEST_F(BluetoothTest, SimulateGattServicesDiscovered) {
+  InitWithFakeAdapter();
+  StartDiscoverySession();
+  BluetoothDevice* device = DiscoverLowEnergyDevice(3);
+  device->CreateGattConnection(GetGattConnectionCallback(),
+                               GetConnectErrorCallback());
+  ResetEventCounts();

[ortuno, 2015/10/08 22:53:27]

Any reason why you have this here instead of in a SetUp or TearDown function?

https://code.google.com/p/chromium/codesearch#chromium/src/testing/gtest/incl...

[scheib, 2015/10/08 23:52:19]

Placing it immediately before and the EXPECT after the method under test ensures
that the event is generated as a side effect of that method.

+  SimulateGattConnection(device);
+  EXPECT_EQ(1, gatt_discovery_attempts_);
+
+  // TODO(scheib): Add more control over how many services are created and
+  // their properties.

[ortuno, 2015/10/08 22:53:27]

Link to issue?

[scheib, 2015/10/08 23:52:20]

Done.

+  SimulateGattServicesDiscovered(device);
+  EXPECT_EQ(2u, device->GetGattServices().size());
+}
+#endif  // defined(OS_ANDROID)
+
+#if defined(OS_ANDROID)
+TEST_F(BluetoothTest, SimulateGattServicesDiscoveryError) {
+  InitWithFakeAdapter();
+  StartDiscoverySession();
+  BluetoothDevice* device = DiscoverLowEnergyDevice(3);
+  device->CreateGattConnection(GetGattConnectionCallback(),
+                               GetConnectErrorCallback());
+  ResetEventCounts();
+  SimulateGattConnection(device);
+  EXPECT_EQ(1, gatt_discovery_attempts_);
+
+  SimulateGattServicesDiscoveryError(device);
+  EXPECT_EQ(0u, device->GetGattServices().size());
+}
+#endif  // defined(OS_ANDROID)
+
 }  // namespace device