Next phase for chrome.networkingPrivate interface.

chrome/browser/chromeos/extensions/networking_private_apitest.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 <stdio.h>
 
 #include "base/stl_util.h"
 #include "chrome/browser/extensions/extension_apitest.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/test/base/ui_test_utils.h"
 
 namespace chromeos {
 
 class ExtensionNetworkingPrivateApiTest : public ExtensionApiTest {
  public:
   // Whitelist the extension ID of the test extension.
   virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
     ExtensionApiTest::SetUpCommandLine(command_line);
     command_line->AppendSwitchASCII(
         switches::kWhitelistedExtensionID, "epcifkihnkjgphfkloaaleeakhpmgdmn");
  }
+
+  void RunNetworkingSubtest(const std::string& subtest) {
+    // Make the first character lowercase so we can match naming conventions.
+    EXPECT_TRUE(RunExtensionSubtest(
+        "networking", "main.html?" + subtest,
+        kFlagEnableFileAccess | kFlagLoadAsComponent)) << message_;
+  }
 };
 
-IN_PROC_BROWSER_TEST_F(ExtensionNetworkingPrivateApiTest, BasicFunctions) {
-  ASSERT_TRUE(RunComponentExtensionTest("networking")) << message_;
+// Place each subtest into a separate browser test so that the stub networking
+// library state is reset for each subtest run. This way they won't affect each
+// other.
+
+IN_PROC_BROWSER_TEST_F(ExtensionNetworkingPrivateApiTest, StartConnect) {
+  RunNetworkingSubtest("startConnect");

[not at google - send to devlin, 2013/02/12 23:36:09]

The way you have it failures won't be on the line that they happen, they'll
always be on line 25. I guess here it doesn't matter so much since each test is
only 1 line so you'll be able to tell... but
EXPECT_TRUE(RunNetworkingSubtest(..)) would generally be better (particularly if
any of these tests get more complicated).

[Greg Spencer (Chromium), 2013/02/12 23:57:42]

Good point.  Fixed.

+}
+
+IN_PROC_BROWSER_TEST_F(ExtensionNetworkingPrivateApiTest, StartDisconnect) {
+  RunNetworkingSubtest("startDisconnect");
+}
+
+IN_PROC_BROWSER_TEST_F(ExtensionNetworkingPrivateApiTest,
+                       StartConnectNonexistent) {
+  RunNetworkingSubtest("startConnectNonexistent");
+}
+
+IN_PROC_BROWSER_TEST_F(ExtensionNetworkingPrivateApiTest, GetVisibleNetworks) {
+  RunNetworkingSubtest("getVisibleNetworks");
+}
+
+IN_PROC_BROWSER_TEST_F(ExtensionNetworkingPrivateApiTest,
+                       GetVisibleNetworksWifi) {
+  RunNetworkingSubtest("getVisibleNetworksWifi");
+}
+
+IN_PROC_BROWSER_TEST_F(ExtensionNetworkingPrivateApiTest, GetProperties) {
+  RunNetworkingSubtest("getProperties");
+}
+
+IN_PROC_BROWSER_TEST_F(ExtensionNetworkingPrivateApiTest,
+                       OnNetworksChangedEvent) {
+  RunNetworkingSubtest("onNetworksChangedEvent");
+}
+
+IN_PROC_BROWSER_TEST_F(ExtensionNetworkingPrivateApiTest,
+                       OnNetworkListChangedEvent) {
+  RunNetworkingSubtest("onNetworkListChangedEvent");
 }
 
 }  // namespace chromeos