Report the Auth method for me2me connections.

remoting/webapp/crd/js/me2me_telemetry_integration_test.js

 // Copyright 2015 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.
 
 
 /** @suppress {duplicate} */
 var remoting = remoting || {};
 
 (function() {
 
@@ skipping 266 matching lines @@
   });
 
   var count = 0;
   function onPluginCreated(/** remoting.MockClientPlugin */ plugin) {
     count++;
     if (count == 1) {
       plugin.mock$returnErrorOnConnect(
           remoting.ClientSession.ConnectionError.HOST_IS_OFFLINE);
     } else if (count == 2) {
       plugin.mock$useDefaultBehavior(
-          remoting.MockClientPlugin.AuthMethod.PIN);
+          remoting.ChromotingEvent.AuthMethod.PIN);
     }
   }
 
   /**
    * @param {remoting.MockClientPlugin} plugin
    * @param {remoting.ClientSession.State} state
    */
   function onStatusChanged(plugin, state) {
     if (state == remoting.ClientSession.State.CONNECTED) {
       testDriver.me2meActivity().stop();
@@ skipping 101 matching lines @@
       }
     }
   }
 
   testDriver.mockConnection().pluginFactory().mock$setPluginStatusChanged(
       onStatusChanged);
 
   return testDriver.startTest();
 });
 
+
+/**
+ * @param {remoting.Me2MeTestDriver} testDriver
+ * @param {remoting.ChromotingEvent.AuthMethod} authMethod
+ * @return {Promise}
+ */
+function createAuthMethodTest(testDriver, authMethod) {
+  var ChromotingEvent = remoting.ChromotingEvent;
+  expectSequence(
+    testDriver,
+    {
+      session_entry_point: ChromotingEvent.SessionEntryPoint.CONNECT_BUTTON,
+      role: ChromotingEvent.Role.CLIENT,
+      mode: ChromotingEvent.Mode.ME2ME,
+    },
+    [
+      ChromotingEvent.SessionState.STARTED,
+      ChromotingEvent.SessionState.SIGNALING,
+      ChromotingEvent.SessionState.CREATING_PLUGIN,
+      ChromotingEvent.SessionState.CONNECTING,
+    ]);
+  expectSequence(
+    testDriver,
+    {
+      session_entry_point: ChromotingEvent.SessionEntryPoint.CONNECT_BUTTON,
+      role: ChromotingEvent.Role.CLIENT,
+      mode: ChromotingEvent.Mode.ME2ME,
+      auth_method: authMethod
+    },
+    [
+      ChromotingEvent.SessionState.AUTHENTICATED,
+      ChromotingEvent.SessionState.CONNECTED,
+      ChromotingEvent.SessionState.CLOSED
+    ]);
+
+  /**
+   * @param {remoting.MockClientPlugin} plugin
+   * @param {remoting.ClientSession.State} state
+   */
+  function onStatusChanged(plugin, state) {
+    if (state == remoting.ClientSession.State.CONNECTED) {
+      testDriver.me2meActivity().stop();
+      testDriver.endTest();
+    }
+  }
+  function onPluginCreated(/** remoting.MockClientPlugin */ plugin) {
+    plugin.mock$useDefaultBehavior(authMethod);
+  }
+
+  testDriver.mockConnection().pluginFactory().mock$setPluginCreated(
+      onPluginCreated);
+  testDriver.mockConnection().pluginFactory().mock$setPluginStatusChanged(
+      onStatusChanged);
+
+  return testDriver.startTest();
+}
+
+QUnit.test('Auth method - THIRD_PARTY (connected)', function() {
+  return createAuthMethodTest(testDriver,
+                              remoting.ChromotingEvent.AuthMethod.THIRD_PARTY);
+});
+
+QUnit.test('Auth method - PINLESS (connected)', function() {
+  return createAuthMethodTest(testDriver,
+                              remoting.ChromotingEvent.AuthMethod.PINLESS);
+});
+
+QUnit.test('Auth method - PIN (connected)', function() {
+  return createAuthMethodTest(testDriver,
+                              remoting.ChromotingEvent.AuthMethod.PIN);
+});
+

[Jamie, 2015/10/09 01:17:21]

Add a test that the auth method is set for failed connections too?

[kelvinp, 2015/10/09 18:00:34]

Done.

 })();