bluetooth: Implement simulateGATTConnectionResponse()

third_party/WebKit/LayoutTests/bluetooth/server/connect/connection-succeeds.html

Index: third_party/WebKit/LayoutTests/bluetooth/server/connect/connection-succeeds.html
diff --git a/third_party/WebKit/LayoutTests/bluetooth/server/connect/connection-succeeds.html b/third_party/WebKit/LayoutTests/bluetooth/server/connect/connection-succeeds.html
index f0d82e1f3f5538e4b4fb11f99a9b2d60547b9b34..5517dc03184732a0adf878a55d2bb7af95a3007b 100644
--- a/third_party/WebKit/LayoutTests/bluetooth/server/connect/connection-succeeds.html
+++ b/third_party/WebKit/LayoutTests/bluetooth/server/connect/connection-succeeds.html
@@ -2,13 +2,20 @@
 <script src="../../../resources/testharness.js"></script>
 <script src="../../../resources/testharnessreport.js"></script>
 <script src="../../../resources/bluetooth/bluetooth-helpers.js"></script>
+<script src="../../../resources/bluetooth/web-bluetooth-test.js"></script>
+<script src="../../../resources/mojo-helpers.js"></script>
 <script>
 'use strict';
 promise_test(() => {
-  return setBluetoothFakeAdapter('HeartRateAdapter')
-    .then(() => requestDeviceWithKeyDown({
-      filters: [{services: ['heart_rate']}]}))
-    .then(device => device.gatt.connect())
-    .then(gattServer => assert_true(gattServer.connected));
+  return setUpHealthThermometerAndHeartRateDevices()

[scheib, 2017/05/12 05:43:55]

a) We should document what is returned by setUp....Devices methods. If we use
this one we should capture the result as something specific
'thermometer_peripheral' since we know which one it is (I think).

b) In this test, we don't need multiple devices, and I think we should simplify
it to the generic flow you used elsewhere 
  setUpPreconnectedDevice({}).then(device => 
  requestDevice ... acceptAllDevices

[ortuno, 2017/05/15 07:01:31]

Makes sense. Added docs.
I introduced getDiscoveredHealthThermometerDevice(). 

My current plan is to have the following functions:

getHealthThermometerDevice();

getConnectedHealthThermometerDevice();

getDiscoveredHealthThermoemeterDevice();

getHealthThermometerDevice() will be similar to 'HeartRateAdapter'. It would
return a connected BluetoothDevice where all attributes have been discovered.
The other functions are for tests that require the device in an intermediate
step e.g. getDiscoveredHealthThermometerDevice() returns a non-connected device,
getConnectedHealthThermometerDevice() would return a connected device whose
services haven't been discovered yet, etc. As I implement more features, I'll
add them to the device returned by getHealthThermometerDevice() e.g. the next
patch will add services.

+    .then(([fake_peripheral]) => {
+      return requestDeviceWithKeyDown({
+        filters: [{services: ['health_thermometer']}]})
+        .then(device => Promise.all([
+          device.gatt.connect(),
+          fake_peripheral.simulateGATTConnectionResponse({code: HCI_SUCCESS})
+        ]))
+        .then(([gattServer]) => assert_true(gattServer.connected));
+    });
 }, 'Device will connect');
 </script>