Adding the "Authenticate" step to the "MANUAL_Auth" test case. Also added the username and password…

remoting/test/remote_desktop_browsertest.cc

 // Copyright (c) 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/command_line.h"
 #include "base/path_service.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/extensions/extension_browsertest.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/extensions/unpacked_installer.h"
@@ skipping 17 matching lines @@
 #include "net/dns/mock_host_resolver.h"
 
 using extensions::Extension;
 
 namespace {
 // Command line arguments specific to the chromoting browser tests.
 const char kOverrideUserDataDir[] = "override-user-data-dir";
 const char kNoCleanup[] = "no-cleanup";
 const char kNoInstall[] = "no-install";
 const char kWebAppCrx[] = "webapp-crx";
+const char kUsername[] = "username";
+const char kPassword[] = "password";
 }
 
 namespace remoting {
 
 class RemoteDesktopBrowserTest : public ExtensionBrowserTest {
  public:
   virtual void SetUp() OVERRIDE {
     ParseCommandLine();
     ExtensionBrowserTest::SetUp();
   }
@@ skipping 22 matching lines @@
   void VerifyChromotingLoaded(bool expected);
 
   // Launch the chromoting app.
   void LaunchChromotingApp();
 
   // Verify the test has access to the internet (specifically google.com)
   void VerifyInternetAccess();
 
   void Authorize();
 
+  void Authenticate();
+
   // Whether to perform the cleanup tasks (uninstalling chromoting, etc).
   // This is useful for diagnostic purposes.
   bool NoCleanup() { return no_cleanup_; }
 
   // Whether to install the chromoting extension before running the test cases.
   // This is useful for diagnostic purposes.
   bool NoInstall() { return no_install_; }
 
  private:
   void ParseCommandLine();
@@ skipping 19 matching lines @@
 
   // This test needs to make live DNS requests for access to
   // GAIA and sync server URLs under google.com. We use a scoped version
   // to override the default resolver while the test is active.
   scoped_ptr<net::ScopedDefaultHostResolverProc> mock_host_resolver_override_;
 
   bool no_cleanup_;
   bool no_install_;
   std::string chromoting_id_;
   base::FilePath webapp_crx_;
+  std::string username_;
+  std::string password_;
 };
 
 void RemoteDesktopBrowserTest::ParseCommandLine() {
   CommandLine* command_line = CommandLine::ForCurrentProcess();
 
   // The test framework overrides any command line user-data-dir
   // argument with a /tmp/.org.chromium.Chromium.XXXXXX directory.
   // That happens in the ChromeTestLauncherDelegate, and affects
   // all unit tests (no opt out available). It intentionally erases
   // any --user-data-dir switch if present and appends a new one.
   // Re-override the default data dir if override-user-data-dir
   // is specified.
   if (command_line->HasSwitch(kOverrideUserDataDir)) {
     const base::FilePath& override_user_data_dir =
         command_line->GetSwitchValuePath(kOverrideUserDataDir);
 
     ASSERT_FALSE(override_user_data_dir.empty());
 
     command_line->AppendSwitchPath(switches::kUserDataDir,
                                    override_user_data_dir);
   }
 
+  username_ = command_line->GetSwitchValueNative(kUsername);
+  password_ = command_line->GetSwitchValueNative(kPassword);
+
   no_cleanup_ = command_line->HasSwitch(kNoCleanup);
   no_install_ = command_line->HasSwitch(kNoInstall);
 
   if (!no_install_) {
     webapp_crx_ = command_line->GetSwitchValuePath(kWebAppCrx);
     ASSERT_FALSE(webapp_crx_.empty());
   }
 }
 
 void RemoteDesktopBrowserTest::EnableDNSLookupForThisTest(
@@ skipping 129 matching lines @@
   EXPECT_TRUE(result);
 
   ASSERT_TRUE(content::ExecuteScriptAndExtractBool(
       browser()->tab_strip_model()->GetActiveWebContents(),
       "window.domAutomationController.send("
           "document.getElementById(\"Passwd\") != null);",
       &result));
   EXPECT_TRUE(result);
 }
 
+void RemoteDesktopBrowserTest::Authenticate() {
+  // The chromoting extension should be installed.
+  ASSERT_FALSE(ChromotingID().empty());
+
+  // The active tab should have the "Google Accounts" login page loaded.
+  ASSERT_EQ(GetCurrentURL().host(), "accounts.google.com");
+
+  bool result;
+
+  ASSERT_TRUE(content::ExecuteScriptAndExtractBool(
+      browser()->tab_strip_model()->GetActiveWebContents(),
+      "window.domAutomationController.send("
+          "document.getElementById(\"Email\") != null);",
+      &result));
+  ASSERT_TRUE(result);
+
+  ASSERT_TRUE(content::ExecuteScriptAndExtractBool(
+      browser()->tab_strip_model()->GetActiveWebContents(),
+      "window.domAutomationController.send("
+          "document.getElementById(\"Passwd\") != null);",
+      &result));

[garykac, 2013/08/07 17:52:18]

Have you tried extracting this pattern:

ASSERT_TRUE(content::ExecuteScriptAndExtractBool(
    browser()->tab_strip_model()->GetActiveWebContents(),
    "window.domAutomationController.send("
        "document.getElementById(\"Passwd\") != null);",
    &result));

so you could write simply:

ASSERT_TRUE(VerifyElement("Passwd"));

?

+  ASSERT_TRUE(result);
+
+  content::WindowedNotificationObserver observer(
+      content::NOTIFICATION_LOAD_STOP,
+      content::Source<content::NavigationController>(
+          &browser()->tab_strip_model()->GetActiveWebContents()->
+              GetController()));
+
+  // Now log in using the username and password passed in from the command line.
+  ASSERT_TRUE(content::ExecuteScript(
+      browser()->tab_strip_model()->GetActiveWebContents(),
+      "document.getElementById(\"Email\").value = \"" + username_ + "\";" +
+      "document.getElementById(\"Passwd\").value = \"" + password_ +"\";" +
+      "document.forms[\"gaia_loginform\"].submit();"));
+  observer.Wait();
+
+  EXPECT_EQ(GetCurrentURL().host(), "accounts.google.com");
+
+  // Is there a better way to verify we are on the "Request for Permission"
+  // page?
+  ASSERT_TRUE(content::ExecuteScriptAndExtractBool(
+      browser()->tab_strip_model()->GetActiveWebContents(),
+      "window.domAutomationController.send("
+          "document.getElementById(\"submit_approve_access\") != null);",
+      &result));
+  EXPECT_TRUE(result);
+}
+
 IN_PROC_BROWSER_TEST_F(RemoteDesktopBrowserTest, MANUAL_Launch) {
   VerifyInternetAccess();
 
   if (!NoInstall()) {
     VerifyChromotingLoaded(false);
     InstallChromotingApp();
   }
 
   VerifyChromotingLoaded(true);
 
@@ skipping 18 matching lines @@
     VerifyChromotingLoaded(false);
     InstallChromotingApp();
   }
 
   VerifyChromotingLoaded(true);
 
   LaunchChromotingApp();
 
   Authorize();
 
+  Authenticate();
+
   if (!NoCleanup()) {
     UninstallChromotingApp();
     VerifyChromotingLoaded(false);
   }
 }
 
 }  // namespace remoting