bluetooth: Basic browser tests for chrome://bluetooth-internals.

chrome/test/data/webui/bluetooth_internals_browsertest.js

+// Copyright 2016 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.
+
+/**
+ * @fileoverview Tests for chrome://bluetooth-internals
+ */
+
+/** @const {string} Path to source root. */
+var ROOT_PATH = '../../../../';
+
+/**
+ * Test fixture for BluetoothInternals WebUI testing.
+ * @constructor
+ * @extends testing.Test
+ */
+function BluetoothInternalsTest() {
+  this.adapterFactory = null;
+  this.setupResolver = new PromiseResolver();
+}
+
+BluetoothInternalsTest.prototype = {
+  __proto__: testing.Test.prototype,
+
+  /** @override */
+  browsePreload: 'chrome://bluetooth-internals',
+
+  /** @override */
+  isAsync: true,
+
+  /** @override */
+  runAccessibilityChecks: false,
+
+  /** @override */
+  extraLibraries: [
+    ROOT_PATH + 'third_party/mocha/mocha.js',
+    ROOT_PATH + 'chrome/test/data/webui/mocha_adapter.js',
+    ROOT_PATH + 'ui/webui/resources/js/promise_resolver.js',
+    ROOT_PATH + 'ui/webui/resources/js/cr.js',
+    ROOT_PATH + 'ui/webui/resources/js/util.js',
+    ROOT_PATH + 'chrome/test/data/webui/settings/test_browser_proxy.js',
+  ],
+
+  preLoad: function() {
+    // A function that is called from chrome://bluetooth-internals to allow this
+    // test to replace the real Mojo browser proxy with a fake one, before any
+    // other code runs.
+    window.setupFn = function() {
+      console.log('Importing modules');
+      return importModules([
+          'content/public/renderer/frame_interfaces',

[dpapad, 2016/11/02 23:19:14]

Nit: There is 2 more spaces of indentation than needed.

[mbrunson, 2016/11/03 20:59:05]

Done.

+          'device/bluetooth/public/interfaces/adapter.mojom',
+          'device/bluetooth/public/interfaces/device.mojom',
+          'mojo/public/js/bindings',
+          'mojo/public/js/connection',
+        ]).then(function([frameInterfaces, adapter, device, bindings,
+            connection]) {
+          console.log('Modules imported');

[dpapad, 2016/11/02 23:19:15]

Can you remove this.

[mbrunson, 2016/11/03 20:59:05]

Done.

+
+          /**
+           * A test adapter factory proxy for the chrome://bluetooth-internals
+           * page.
+           *
+           * @constructor
+           * @extends {TestBrowserProxyBase}
+           */
+          var TestAdapterFactoryProxy = function() {
+            settings.TestBrowserProxy.call(this, [
+              'getAdapter',
+            ]);
+
+            this.adapter = new TestAdapter();
+            this.adapterHandle_ = connection.bindStubDerivedImpl(this.adapter);
+          };
+
+          TestAdapterFactoryProxy.prototype = {
+            __proto__: settings.TestBrowserProxy.prototype,
+            getAdapter: function() {
+              this.methodCalled('getAdapter');
+
+              // Create message pipe bound to TestAdapter.
+              return Promise.resolve({
+                adapter: this.adapterHandle_,
+              });
+            }
+          };
+
+          /**
+           * A test adapter for the chrome://bluetooth-internals page.
+           * Must be used to create message pipe handle from test code.
+           *
+           * @constructor
+           * @extends {adapter.Adapter.stubClass}
+           */
+          var TestAdapter = function() {
+            this.proxy = new TestAdapterProxy();
+          };
+
+          TestAdapter.prototype = {
+            __proto__: adapter.Adapter.stubClass.prototype,
+            getInfo: function() { return this.proxy.getInfo(); },
+            getDevices: function() { return this.proxy.getDevices(); },
+            setClient: function(client) { return this.proxy.setClient(client); }
+          };
+
+          /**
+           * A test adapter proxy for the chrome://bluetooth-internals page.
+           *
+           * @constructor
+           * @extends {TestBrowserProxyBase}
+           */
+          var TestAdapterProxy = function() {
+            settings.TestBrowserProxy.call(this, [
+              'getInfo',
+              'getDevices',
+              'setClient',
+            ]);
+
+            this.client = null;
+
+            this.adapterInfo_ = null;
+            this.devices_ = [];
+          };
+
+          TestAdapterProxy.prototype = {
+            __proto__: settings.TestBrowserProxy.prototype,
+            getInfo: function() {
+              this.methodCalled('getInfo');
+              return Promise.resolve({ info: this.adapterInfo_ });

[dpapad, 2016/11/02 23:19:15]

No need for spaces {info: this.adapterInfo_}

[mbrunson, 2016/11/03 20:55:10]

Done.

+            },

[dpapad, 2016/11/02 23:19:15]

\n between functions

[mbrunson, 2016/11/03 20:55:10]

Done.

+            getDevices: function() {
+              this.methodCalled('getDevices');
+              return Promise.resolve({ devices: this.devices_ });
+            },
+            setClient: function(client) {
+              this.methodCalled('setClient', client);
+              this.client = connection.bindHandleToProxy(client,
+                adapter.AdapterClient);

[dpapad, 2016/11/02 23:19:15]

Indent is off. Also try to keep all arguments on the same line if possible (per
styleguide).

this.client = connection.bindHandleToProxy(
    client, adapter.AdapterClient);

[mbrunson, 2016/11/03 20:55:10]

Done.

+            },
+            setTestAdapter: function(adapterInfo) {
+              this.adapterInfo_ = adapterInfo;
+            },
+            setTestDevices: function(devices) {
+              this.devices_ = devices;
+            }
+          };
+
+
+          frameInterfaces.addInterfaceOverrideForTesting(
+            adapter.AdapterFactory.name,
+            function(handle) {
+              var stub = connection.bindHandleToStub(
+                handle, adapter.AdapterFactory);

[dpapad, 2016/11/02 23:19:15]

Indent off by 2.

[mbrunson, 2016/11/03 20:55:10]

Done.

+              this.adapterFactory = new TestAdapterFactoryProxy();
+              bindings.StubBindings(stub).delegate = this.adapterFactory;
+
+              this.setupResolver.resolve();
+            }.bind(this));
+

[dpapad, 2016/11/02 23:19:15]

Nit: Remove \n

[mbrunson, 2016/11/03 20:55:10]

Done.

+        }.bind(this));
+    }.bind(this);
+  },
+};
+
+TEST_F('BluetoothInternalsTest', 'Startup_BluetoothInternals', function() {
+  var fakeAdapterInfo = {
+    address: '02:1C:7E:6A:11:5A',
+    discoverable: false,
+    discovering: false,
+    initialized: true,
+    name: 'computer.example.com-0',
+    powered: true,
+    present: true,
+  };
+
+  var fakeDeviceInfo1 = {
+    address: "AA:AA:84:96:92:84",
+    name: "AAA",
+    name_for_display: "AAA",
+    rssi: {value: -40},
+    services: []
+  };
+
+  var fakeDeviceInfo2 = {
+    address: "BB:BB:84:96:92:84",
+    name: "BBB",
+    name_for_display: "BBB",
+    rssi: null,
+    services: []
+  };
+
+  var fakeDeviceInfo3 = {
+    address: "CC:CC:84:96:92:84",
+    name: "CCC",
+    name_for_display: "CCC",
+  };
+
+  var adapterFactory = null;
+
+  // Before tests are run, make sure setup completes.
+  var setupPromise = this.setupResolver.promise.then(function() {
+    adapterFactory = this.adapterFactory;
+  }.bind(this));
+
+
+  suite('BluetoothInternalsUITest', function() {
+    var EXPECTED_DEVICES = 2;
+
+    suiteSetup(function() {
+      return setupPromise.then(function() {
+        adapterFactory.adapter.proxy.setTestDevices([
+          fakeDeviceInfo1,
+          fakeDeviceInfo2
+        ]);
+        adapterFactory.adapter.proxy.setTestAdapter(fakeAdapterInfo);
+
+        return Promise.all([
+          adapterFactory.whenCalled('getAdapter'),
+          adapterFactory.adapter.proxy.whenCalled('getInfo'),
+          adapterFactory.adapter.proxy.whenCalled('getDevices'),
+        ]);
+      });
+    });
+
+    setup(function() {
+      var devices = new device_collection.DeviceCollection([]);
+      adapterClient.devices = devices;
+      deviceTable.setDevices(devices);
+      adapterClient.deviceAdded(fakeDeviceInfo1);
+      adapterClient.deviceAdded(fakeDeviceInfo2);
+    });
+
+    teardown(function() {
+      adapterFactory.reset();
+    });
+
+    /**
+     * Updates device info and verifies the contents of the device table.
+     * @param {!device_collection.DeviceInfo} deviceInfo
+     */
+    function changeDevice(deviceInfo) {
+      var deviceRow = document.querySelector('#' + escapeDeviceAddress(
+          deviceInfo.address));
+      var nameForDisplayColumn = deviceRow.children[0];
+      var addressColumn = deviceRow.children[1];
+      var rssiColumn = deviceRow.children[2];
+      var servicesColumn = deviceRow.children[3];
+
+      expectTrue(!!nameForDisplayColumn);
+      expectTrue(!!addressColumn);
+      expectTrue(!!rssiColumn);
+      expectTrue(!!servicesColumn);
+
+      adapterClient.deviceChanged(deviceInfo);
+
+      expectEquals(deviceInfo.name_for_display,
+                   nameForDisplayColumn.textContent);
+      expectEquals(deviceInfo.address, addressColumn.textContent);
+
+      expectEquals(String(deviceInfo.rssi.value), rssiColumn.textContent);
+
+      if (deviceInfo.services) {
+        expectEquals(String(deviceInfo.services.length),
+                     servicesColumn.textContent);
+      } else {
+        expectEquals('Unknown', servicesColumn.textContent);
+      }
+    }
+
+    /**
+     * Escapes colons in a device address for CSS formatting.
+     * @param {!string} address

[dpapad, 2016/11/02 23:19:15]

"!" is implied for number,boolean,string, so not necessary (here and elsewhere).

[mbrunson, 2016/11/03 20:55:10]

Done.

+     */
+    function escapeDeviceAddress(address) {
+      return address.replace(/:/g, '\\:');
+    }
+
+    /**
+     * Expects whether device with |address| is removed.
+     * @param {!string} address
+     * @param {!boolean} expectRemoved
+     */
+    function expectDeviceRemoved(address, expectRemoved) {
+      var removedRow = document.querySelector('#' + escapeDeviceAddress(
+          address));
+
+      expectEquals(expectRemoved, removedRow.classList.contains('removed'));
+    }
+
+    /**
+     * Tests whether a device is added successfully and not duplicated.
+     */
+    test('DeviceAdded', function() {
+      var devices = document.querySelectorAll('#device-table tbody tr');
+      expectEquals(EXPECTED_DEVICES, devices.length);
+
+      adapterClient.deviceAdded(fakeDeviceInfo3);
+
+      // Same device shouldn't appear twice.
+      adapterClient.deviceAdded(fakeDeviceInfo3);
+
+      devices = document.querySelectorAll('#device-table tbody tr');
+      expectEquals(EXPECTED_DEVICES + 1, devices.length);
+    });
+
+    /**
+     * Tests whether a device is marked properly as removed.
+     */
+    test('DeviceSetToRemoved', function() {
+      var devices = document.querySelectorAll('#device-table tbody tr');
+      expectEquals(EXPECTED_DEVICES, devices.length);
+      adapterClient.deviceRemoved(fakeDeviceInfo2);
+
+      // The number of rows shouldn't change.
+      devices = document.querySelectorAll('#device-table tbody tr');
+      expectEquals(EXPECTED_DEVICES, devices.length);
+
+      expectDeviceRemoved(fakeDeviceInfo2.address, true);
+    });
+
+    /**
+     * Tests whether a changed device updates the device table properly.
+     */
+    test('DeviceChanged', function() {
+      var devices = document.querySelectorAll('#device-table tbody tr');
+      expectEquals(EXPECTED_DEVICES, devices.length);
+
+      var newDeviceInfo = Object.assign({}, fakeDeviceInfo1);
+      newDeviceInfo.name_for_display = 'DDDD';
+      newDeviceInfo.rssi = { value: -20 };
+      newDeviceInfo.services = ['service1', 'service2', 'service3'];
+
+      changeDevice(newDeviceInfo);
+    });
+
+    /**
+     * Tests the entire device cycle: added -> updated -> removed -> re-added.
+     */
+    test('DeviceUpdateCycle', function() {
+      var devices = document.querySelectorAll('#device-table tbody tr');
+      expectEquals(EXPECTED_DEVICES, devices.length);
+
+      adapterClient.deviceAdded(fakeDeviceInfo3);
+
+      var newDeviceInfo = Object.assign({}, fakeDeviceInfo3);
+      newDeviceInfo.name_for_display = 'DDDD';
+      newDeviceInfo.rssi = { value: -20 };
+      newDeviceInfo.services = ['service1', 'service2', 'service3'];
+
+      changeDevice(newDeviceInfo);
+      changeDevice(fakeDeviceInfo3);
+
+      adapterClient.deviceRemoved(fakeDeviceInfo3);
+      expectDeviceRemoved(fakeDeviceInfo3.address, true);
+
+      adapterClient.deviceAdded(fakeDeviceInfo3);
+      expectDeviceRemoved(fakeDeviceInfo3.address, false);
+    });
+  });
+
+  // Run all registered tests.
+  mocha.run();
+});