Added HostListApl implementation to connect to legacy directory and GCD.

remoting/webapp/crd/js/mock_host_list_api.js

 // 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.
 
 /**
  * @fileoverview
  * Mock implementation of remoting.HostList
  */
 
 'use strict';
@@ skipping 14 matching lines @@
   this.authCodeFromRegister = null;
 
   /**
    * The email value for the |register| method to return, or null if
    * it should fail.
    * @type {?string}
    */
   this.emailFromRegister = null;
 
   /**
-   * This host ID to return from register(), or null if it should fail.
+   * The host ID to return from register(), or null if it should fail.
    * @type {?string}
    */
   this.hostIdFromRegister = null;
 
-  /** @type {Array<remoting.Host>} */
-  this.hosts = [
-    {
-      'hostName': 'Online host',
-      'hostId': 'online-host-id',
-      'status': 'ONLINE',
-      'jabberId': 'online-jid',
-      'publicKey': 'online-public-key',
-      'tokenUrlPatterns': [],
-      'updatedTime': new Date().toISOString()
-    },
-    {
-      'hostName': 'Offline host',
-      'hostId': 'offline-host-id',
-      'status': 'OFFLINE',
-      'jabberId': 'offline-jid',
-      'publicKey': 'offline-public-key',
-      'tokenUrlPatterns': [],
-      'updatedTime': new Date(1970, 1, 1).toISOString()
-    }
-  ];
+  /**
+   * The host ID to return from getHostIdFromConfig(), or null if it
+   * should fail.
+   * @type {?string}
+   */
+  this.hostIdFromConfig = null;
+
+  /** @type {!Array<!remoting.Host>} */
+  this.hosts = [];
+};
+
+/**
+ * Creates and adds a new mock host.
+ *
+ * @param {string} hostId The ID of the new host to add.
+ * @return {!remoting.Host} the new mock host
+ */
+remoting.MockHostListApi.prototype.addMockHost = function(hostId) {
+  var newHost = new remoting.Host(hostId);
+  this.hosts.push(newHost);
+  return newHost;
 };
 
 /** @override */
 remoting.MockHostListApi.prototype.register = function(
     hostName, publicKey, hostClientId) {
   if (this.authCodeFromRegister === null || this.emailFromRegister === null) {
     return Promise.reject(
         new remoting.Error(
             remoting.Error.Tag.REGISTRATION_FAILED,
             'MockHostListApi.register'));
   } else {
     return Promise.resolve({
       authCode: this.authCodeFromRegister,
       email: this.emailFromRegister,
       hostId: this.hostIdFromRegister
     });
   }
 };
 
 /** @override */
 remoting.MockHostListApi.prototype.get = function() {
-  var that = this;
-  return new Promise(function(resolve, reject) {
-    remoting.mockIdentity.validateTokenAndCall(
-        resolve, remoting.Error.handler(reject), [that.hosts]);

[John Williams, 2015/06/29 19:45:06]

Many of the changes in this file involve removing calls to
remoting.mockIdentity.validateTokenAndCall.  I made this change partly because
it seemed like the easiest way to make my own unit tests work, and partly
because calling a mock API this way, as part of another mock, seems like an
anti-pattern to me.  None of the existing unit tests were affected.

-  });
+  return Promise.resolve(this.hosts);
 };
 
 /**
  * @override
  * @param {string} hostId
  * @param {string} hostName
  * @param {string} hostPublicKey
  */
 remoting.MockHostListApi.prototype.put =
     function(hostId, hostName, hostPublicKey) {
   /** @type {remoting.MockHostListApi} */
   var that = this;
   return new Promise(function(resolve, reject) {
-    var onTokenValid = function() {
-      for (var i = 0; i < that.hosts.length; ++i) {
-        /** type {remoting.Host} */
-        var host = that.hosts[i];
-        if (host.hostId == hostId) {
-          host.hostName = hostName;
-          host.hostPublicKey = hostPublicKey;
-          resolve(undefined);
-          return;
-        }
+    for (var i = 0; i < that.hosts.length; ++i) {
+      /** type {remoting.Host} */
+      var host = that.hosts[i];
+      if (host.hostId == hostId) {
+        host.hostName = hostName;
+        host.hostPublicKey = hostPublicKey;
+        resolve(undefined);
+        return;
       }
-      console.error('PUT request for unknown host: ' + hostId +
-                    ' (' + hostName + ')');
-      reject(remoting.Error.unexpected());
-    };
-    remoting.mockIdentity.validateTokenAndCall(onTokenValid, reject, []);
+    }
+    console.error('PUT request for unknown host: ' + hostId +
+                  ' (' + hostName + ')');
+    reject(remoting.Error.unexpected());
   });
 };
 
 /**
  * @override
  * @param {string} hostId
  */
 remoting.MockHostListApi.prototype.remove = function(hostId) {
   /** @type {remoting.MockHostListApi} */
   var that = this;
   return new Promise(function(resolve, reject) {
-    var onTokenValid = function() {
-      for (var i = 0; i < that.hosts.length; ++i) {
-        var host = that.hosts[i];
-        if (host.hostId == hostId) {
-          that.hosts.splice(i, 1);
-          resolve(undefined);
-          return;
-        }
+    for (var i = 0; i < that.hosts.length; ++i) {
+      var host = that.hosts[i];
+      if (host.hostId == hostId) {
+        that.hosts.splice(i, 1);
+        resolve(undefined);
+        return;
       }
-      console.error('DELETE request for unknown host: ' + hostId);
-      reject(remoting.Error.unexpected());
-    };
-    remoting.mockIdentity.validateTokenAndCall(onTokenValid, reject, []);
+    }
+    console.error('DELETE request for unknown host: ' + hostId);
+    reject(remoting.Error.unexpected());
   });
 };
 
 /** @override */
 remoting.MockHostListApi.prototype.getSupportHost = function(supportId) {
-  var that = this;
-  return new Promise(function(resolve, reject) {
-    remoting.mockIdentity.validateTokenAndCall(
-        resolve, remoting.Error.handler(reject), [that.hosts[0]]);
-  });
+  return Promise.resolve(this.hosts[0]);
+};
 
+/** @override */
+remoting.MockHostListApi.prototype.getHostIdFromConfig = function(config) {
+  if (this.hostIdFromConfig == null) {
+    throw new Error('Must set hostIdFromConfig');
+  }
+  return this.hostIdFromConfig;
 };
 
 /**
  * @param {boolean} active
  */
 remoting.MockHostListApi.setActive = function(active) {
   remoting.HostListApi.setInstance(
       active ? new remoting.MockHostListApi() : null);
 };