Adding the first set of remote test cases and associated framework.

chrome/test/remoting/me2me_browsertest.cc

 // Copyright 2013 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.
 
 #include "base/files/file_path.h"
 #include "base/files/file_util.h"
 #include "chrome/browser/ui/browser_window.h"
+#include "chrome/test/remoting/key_code_test_map.h"
 #include "chrome/test/remoting/remote_desktop_browsertest.h"
+#include "chrome/test/remoting/remote_test_helper.h"
 #include "chrome/test/remoting/waiter.h"
 #include "extensions/browser/app_window/app_window.h"
 
 namespace remoting {
 
 class Me2MeBrowserTest : public RemoteDesktopBrowserTest {
  protected:
-  void TestKeyboardInput();
-  void TestMouseInput();
+  void TestKeypressInput(ui::KeyboardCode, const char*);
 
   void ConnectPinlessAndCleanupPairings(bool cleanup_all);
   bool IsPairingSpinnerHidden();
+  void SetupForRemoteHostTest();
 
   void RestoreApp();
   void MinimizeApp();
 };
 
 IN_PROC_BROWSER_TEST_F(Me2MeBrowserTest,
                        MANUAL_Me2Me_Connect_Local_Host) {
   SetUpTestForMe2Me();
 
   ConnectToLocalHost(false);
@@ skipping 20 matching lines @@
 
   // TODO(weitaosu): Find a way to verify keyboard input injection.
   // We cannot use TestKeyboardInput because it assumes
   // that the client and the host are on the same machine.
 
   DisconnectMe2Me();
   Cleanup();
 }
 
 IN_PROC_BROWSER_TEST_F(Me2MeBrowserTest,
+                       MANUAL_Me2Me_Remote_Host_Keypress) {
+  SetupForRemoteHostTest();
+
+  // Test all key characters
+  int length = sizeof(test_key_map)/sizeof(KeyCodeTestMap);
+  for (int i = 0; i < length; i++) {
+    KeyCodeTestMap key = test_key_map[i];
+    TestKeypressInput(key.vkey_code, key.code);
+  }
+  DisconnectMe2Me();
+  Cleanup();
+}
+
+IN_PROC_BROWSER_TEST_F(Me2MeBrowserTest,
+                       MANUAL_Me2Me_Remote_Host_Digitpress) {
+  SetupForRemoteHostTest();
+
+  // Test all digit characters
+  int length = sizeof(test_digit_map)/sizeof(KeyCodeTestMap);
+  for (int i = 0; i < length; i++) {
+    KeyCodeTestMap key = test_digit_map[i];
+    TestKeypressInput(key.vkey_code, key.code);
+  }
+  DisconnectMe2Me();
+  Cleanup();
+}
+
+IN_PROC_BROWSER_TEST_F(Me2MeBrowserTest,
+                       MANUAL_Me2Me_Remote_Host_Specialpress) {
+  SetupForRemoteHostTest();
+
+  // Test all special characters
+  int length = sizeof(test_special_map)/sizeof(KeyCodeTestMap);
+  for (int i = 0; i < length; i++) {
+    KeyCodeTestMap key = test_special_map[i];
+    TestKeypressInput(key.vkey_code, key.code);
+  }
+  DisconnectMe2Me();
+  Cleanup();
+}
+
+IN_PROC_BROWSER_TEST_F(Me2MeBrowserTest,
+                       MANUAL_Me2Me_Remote_Host_Numpadpress) {
+  SetupForRemoteHostTest();
+
+  // Test all numpad characters
+  int length = sizeof(test_numpad_map)/sizeof(KeyCodeTestMap);
+  for (int i = 0; i < length; i++) {
+    KeyCodeTestMap key = test_numpad_map[i];
+    TestKeypressInput(key.vkey_code, key.code);
+  }
+  DisconnectMe2Me();
+  Cleanup();
+}
+
+IN_PROC_BROWSER_TEST_F(Me2MeBrowserTest,

[Jamie, 2015/01/06 19:31:38]

Is it possible to add a test for modifier keys as well? Based on past bug
reports, those are the most likely to fail. It can be a follow-up CL.

[Mike Meade, 2015/01/08 18:58:42]

I have a few special keys in there now. We can detect the keypress since I used
the onkeydown function, but many of them have side effects in the browser (like
moving focus) that I haven't figured out a good way to get around. I can go back
and see what others can be added and what the roadblocks are for the ones that
can't be easily added.

I have a feeling we would need a system level event listener to get these
without making a test hacky by trying to guess how to get focus back to the
page. The good thing is that it would be very easy to reuse the remote test
server code with something like this, so that the whole thing doesn't need to be
re-engineered.

                        MANUAL_Me2Me_Connect_Pinless) {
   SetUpTestForMe2Me();
 
   ASSERT_FALSE(HtmlElementVisible("paired-client-manager-message"))
       << "The host must have no pairings before running the pinless test.";
 
   // Test that cleanup works with either the Delete or Delete all buttons.
   ConnectPinlessAndCleanupPairings(false);
   ConnectPinlessAndCleanupPairings(true);
 
@@ skipping 18 matching lines @@
   EXPECT_TRUE(RemoteDesktopBrowserTest::IsSessionConnected());
 
   // Maximize so we can disconnect and teardown.
   RestoreApp();
 
   // Cleanup
   DisconnectMe2Me();
   Cleanup();
 }
 
+void Me2MeBrowserTest::SetupForRemoteHostTest() {
+  VerifyInternetAccess();
+  OpenClientBrowserPage();
+  Install();
+  LaunchChromotingApp(false);
 
-void Me2MeBrowserTest::TestKeyboardInput() {
-  // We will assume here that the browser window is already open on the host
-  // and in focus.
-  // Press tab to put focus on the textbox.
-  SimulateKeyPressWithCode(ui::VKEY_TAB, "Tab", false, false, false, false);
+  // Authorize, Authenticate, and Approve.
+  Auth();
+  ExpandMe2Me();
+  ConnectToRemoteHost(remote_host_name(), false);
 
-  // Write some text in the box and press enter
-  std::string text = "Abigail";
-  SimulateStringInput(text);
-  SimulateKeyPressWithCode(
-      ui::VKEY_RETURN, "Enter", false, false, false, false);
-
-  // Wait until the client tab sets the right variables
-  ConditionalTimeoutWaiter waiter(
-          base::TimeDelta::FromSeconds(10),
-          base::TimeDelta::FromMilliseconds(500),
-          base::Bind(&RemoteDesktopBrowserTest::IsHostActionComplete,
-                     client_web_content(),
-                     "testResult.keypressSucceeded"));
-  EXPECT_TRUE(waiter.Wait());
-
-  // Check that the text we got is correct
-  EXPECT_TRUE(ExecuteScriptAndExtractBool(
-      client_web_content(),
-      "testResult.keypressText == '" + text + "'"));
+  // Wake up the machine if it's sleeping
+  SimulateKeyPressWithCode(ui::VKEY_RETURN, "Enter");
 }
 
-void Me2MeBrowserTest::TestMouseInput() {
-  SimulateMouseLeftClickAt(10, 50);
-  // TODO: Verify programatically the mouse events are received by the host.
-  // This will be tricky as it depends on the host OS, window manager, desktop
-  // layout, and screen resolution. Until then we need to visually verify that
-  // "Dash Home" is clicked on a Unity window manager.
-  ASSERT_TRUE(TimeoutWaiter(base::TimeDelta::FromSeconds(5)).Wait());
+void Me2MeBrowserTest::TestKeypressInput(
+    ui::KeyboardCode keyCode,
+    const char* code) {
+  remote_test_helper()->ClearLastEvent();
+  VLOG(1) << "Pressing " << code;
+  SimulateKeyPressWithCode(keyCode, code);
+  Event event;
+  remote_test_helper()->GetLastEvent(&event);
+  ASSERT_EQ(Action::KEYDOWN, event.action);
+  ASSERT_EQ(keyCode, event.value);
 }
 
 void Me2MeBrowserTest::ConnectPinlessAndCleanupPairings(bool cleanup_all) {
   // First connection: verify that a PIN is requested, and request pairing.
   ConnectToLocalHost(true);
   DisconnectMe2Me();
 
   // TODO(jamiewalch): This reload is only needed because there's a bug in the
   // web-app whereby it doesn't refresh its pairing state correctly.
   // http://crbug.com/311290
@@ skipping 49 matching lines @@
 void Me2MeBrowserTest::RestoreApp() {
   extensions::AppWindow* appWindow = GetFirstAppWindow();
   if (appWindow) {
     appWindow->Restore();
   } else {
     browser()->window()->Restore();
   }
 }
 
 }  // namespace remoting