[chromedriver] Add an error autoreporting feature that automatically raises errors from browser logs

chrome/test/chromedriver/session_commands_unittest.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 <string>
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/file_util.h"
 #include "base/files/file_path.h"
@@ skipping 93 matching lines @@
 };
 
 }  // namespace
 
 TEST(SessionCommandsTest, QuitFails) {
   Session session("id", scoped_ptr<Chrome>(new FailsToQuitChrome()));
   base::DictionaryValue params;
   scoped_ptr<base::Value> value;
   ASSERT_EQ(kUnknownError, ExecuteQuit(false, &session, params, &value).code());
 }
+
+TEST(SessionCommandsTest, AutoReporting) {
+  DetachChrome* chrome = new DetachChrome();
+  Session session("id", scoped_ptr<Chrome>(chrome));
+  base::DictionaryValue params;
+  scoped_ptr<base::Value> value;
+  StatusCode status;

[chrisgao (Use stgao instead), 2014/01/09 22:11:55]

Rename to |status_code|?

[samuong, 2014/01/16 00:29:28]

Done.

+  bool enabled;
+
+  // autoreporting should be disabled by default
+  status = ExecuteIsAutoReportingEnabled(&session, params, &value).code();
+  ASSERT_EQ(kOk, status);
+  ASSERT_FALSE(session.auto_reporting_enabled);
+  value.get()->GetAsBoolean(&enabled);

[chrisgao (Use stgao instead), 2014/01/09 22:11:55]

ASSERT_TRUE?

[samuong, 2014/01/16 00:29:28]

Done.

+  ASSERT_FALSE(enabled);
+
+  // an error should be given if the |enable| parameter is not set
+  status = ExecuteSetAutoReportingEnabled(&session, params, &value).code();
+  ASSERT_EQ(kUnknownError, status);
+
+  // check that autoreporting can be enabled successfully
+  params.SetBoolean("enable", true);
+  status = ExecuteSetAutoReportingEnabled(&session, params, &value).code();
+  ASSERT_EQ(kOk, status);
+  ASSERT_TRUE(session.auto_reporting_enabled);
+  value.get()->GetAsBoolean(&enabled);

[chrisgao (Use stgao instead), 2014/01/09 22:11:55]

ASSERT_TRUE?

[samuong, 2014/01/16 00:29:28]

Done.

+  ASSERT_TRUE(enabled);
+
+  // check that autoreporting can be disabled successfully
+  params.SetBoolean("enable", false);
+  status = ExecuteSetAutoReportingEnabled(&session, params, &value).code();
+  ASSERT_EQ(kOk, status);
+  ASSERT_FALSE(session.auto_reporting_enabled);
+  value.get()->GetAsBoolean(&enabled);

[chrisgao (Use stgao instead), 2014/01/09 22:11:55]

ASSERT_TRUE?

[samuong, 2014/01/16 00:29:28]

Done.

+  ASSERT_FALSE(enabled);
+}