Test that the right events are sent to the Bluetooth chooser.

LayoutTests/bluetooth/requestDevice.html

Index: LayoutTests/bluetooth/requestDevice.html
diff --git a/LayoutTests/bluetooth/requestDevice.html b/LayoutTests/bluetooth/requestDevice.html
index 834bc70756f228076fc087c42c09e290d99cef8e..34add25206efb0e3d81c218f118a806837d1696b 100644
--- a/LayoutTests/bluetooth/requestDevice.html
+++ b/LayoutTests/bluetooth/requestDevice.html
@@ -21,12 +21,18 @@ promise_test(() => {
                                 new TypeError());
 }, 'RequestDeviceOptions requires a |filters| member.');
 
-// TODO(jyasskin): Add a test that the chooser is informed of a failed discovery
-// session.
 promise_test(() => {
   testRunner.setBluetoothMockDataSet('FailStartDiscoveryAdapter');
+  testRunner.setBluetoothManualChooser();
+  let requestDevicePromise =
+      requestDeviceWithKeyDown({filters: [{services: ['generic_access']}]});
+  assert_array_equals(testRunner.getBluetoothManualChooserEvents(),
+                      ['chooser-opened(file:///)',
+                       'discovering',
+                       'discovery-failed-to-start']);
+  testRunner.sendBluetoothManualChooserEvent('cancelled', '');
   return assert_promise_rejects_with_message(
-    requestDeviceWithKeyDown({filters: [{services: ['generic_access']}]}),
+    requestDevicePromise,
     {name: 'NotFoundError', message: 'User cancelled the requestDevice() chooser.'},
     'The adapter failed to start a discovery session.');
 }, 'Discovery session fails to start.');
@@ -39,18 +45,21 @@ promise_test(() => {
     'Bluetooth adapter is not present.');
 }, 'Reject with NotFoundError if the adapter is not present.');
 
-// TODO(jyasskin): Add a test that the chooser is informed of a disabled
-// Bluetooth adapter.
 promise_test(() => {
   testRunner.setBluetoothMockDataSet('NotPoweredAdapter');
+  testRunner.setBluetoothManualChooser();
+  let requestDevicePromise =
+      requestDeviceWithKeyDown({filters: [{services: ['generic_access']}]});
+  assert_array_equals(testRunner.getBluetoothManualChooserEvents(),
+                      ['chooser-opened(file:///)',
+                       'adapter-disabled']);
+  testRunner.sendBluetoothManualChooserEvent('cancelled', '');
   return assert_promise_rejects_with_message(
-    requestDeviceWithKeyDown({filters: [{services: ['generic_access']}]}),
+    requestDevicePromise,
     {name: 'NotFoundError', message: 'User cancelled the requestDevice() chooser.'},
     'Bluetooth adapter is not powered.');
 }, 'Reject with NotFoundError if the adapter is off.');
 
-// TODO(jyasskin): Add a test that the chooser gets a full list of found
-// devices.
 promise_test(() => {
   testRunner.setBluetoothMockDataSet('EmptyAdapter');
   return assert_promise_rejects_with_message(
@@ -123,6 +132,33 @@ promise_test(() => {
 
 promise_test(() => {
   testRunner.setBluetoothMockDataSet('GlucoseHeartRateAdapter');
+  testRunner.setBluetoothManualChooser();
+  let requestDevicePromise = requestDeviceWithKeyDown({
+    filters: [{services: ['glucose']},
+              {services: ['heart_rate']}]
+  });
+  let events = testRunner.getBluetoothManualChooserEvents();
+  while (events.length < 5) {

[ortuno, 2015/09/18 19:56:20]

You do something similar in your other test. Have you noticed any pattern that
we can abstract? Something like:

let tester = new EventTester(testRunner);

tester.assert_events([
  {event: 'chooser-opened', domain: 'file:///'},
  {event: 'devices', devices: ['Heart Rate Device', 'Glucose Device']},
  {event: 'discovering'},
  {event: 'discovery-idle'}
]};

tester.getDevice('Glucose Device');

Or something like that. Just thinking out loud.

+    events.push(...testRunner.getBluetoothManualChooserEvents());
+  }
+  assert_equals(events.length, 5, events);
+  assert_equals(events[0], 'chooser-opened(file:///)', 'events[0]');
+  let idsByName = new AddDeviceEventSet();
+  for (let addedDevice of [events[1], events[2]]) {
+    idsByName.assert_add_device_event(addedDevice);
+  }
+  assert_true(idsByName.has('Heart Rate Device'));
+  assert_true(idsByName.has('Glucose Device'));
+  assert_equals(events[3], 'discovering');
+  assert_equals(events[4], 'discovery-idle');
+  testRunner.sendBluetoothManualChooserEvent('selected',
+                                             idsByName.get('Glucose Device'));
+  return requestDevicePromise
+    .then(device => assert_equals(device.name, 'Glucose Device'));
+}, 'The chooser includes all devices.');
+
+promise_test(() => {
+  testRunner.setBluetoothMockDataSet('GlucoseHeartRateAdapter');
   return requestDeviceWithKeyDown({filters: [{services: ['glucose']}]})
     .then(device => assert_equals(device.name, 'Glucose Device'));
 }, 'Simple filter selects matching device.');