Implement networkingPrivate.getNetworks

chrome/test/data/extensions/api_test/networking/test.js

 // 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.
 
 var callbackPass = chrome.test.callbackPass;
 var callbackFail = chrome.test.callbackFail;
 var assertTrue = chrome.test.assertTrue;
 var assertFalse = chrome.test.assertFalse;
 var assertEq = chrome.test.assertEq;
 
@@ skipping 103 matching lines @@
         chrome.networkingPrivate.getProperties(
           guid,
           callbackPass(function(properties) {
             assertEq("WiFi", properties.Type);
             assertEq(guid, properties.GUID);
             assertEq("wifi_created", properties.WiFi.SSID);
             assertEq("WEP-PSK", properties.WiFi.Security);
           }));
       }));
   },
+  function getNetworks() {
+    // Test 'type' and 'configured'.

[pneubeck (no reviews), 2014/05/15 18:28:37]

testing 'visible: true' too?

[stevenjb, 2014/05/15 20:12:41]

Hmm, we're not actually testing that and should, I will add that test.

+    chrome.networkingPrivate.getNetworks(
+      { "type": "WiFi", "visible": false, "configured": true },
+      callbackPass(function(result) {
+        assertEq([{
+          "Connectable": true,
+          "ConnectionState": "Connected",

[pneubeck (no reviews), 2014/05/15 18:28:37]

if such network states are common over several tests, we could move them to
global variables:
 var initialStateWiFi1 = ...
  ...

[stevenjb, 2014/05/15 20:12:41]

If I did that I'd want to move the entire default states to globals. TODO.

+          "GUID": "stub_wifi1",
+          "Name": "wifi1",
+          "Type": "WiFi",
+          "WiFi": {
+            "Security": "WEP-PSK",
+            "SignalStrength": 40
+          }
+        }, {
+          "Connectable": true,
+          "ConnectionState": "NotConnected",
+          "GUID": "stub_wifi2",
+          "Name": "wifi2_PSK",
+          "Type": "WiFi",
+          "WiFi": {
+            "Security": "WPA-PSK",
+            "SignalStrength": 80
+          }
+        }], result);
+        // Test 'limit'.
+        chrome.networkingPrivate.getNetworks(
+            { "type": "All", "visible": false, "configured": true, "limit": 1 },
+            callbackPass(function(result) {
+              assertEq([{
+                "ConnectionState": "Connected",
+                "Ethernet": {
+                  "Authentication": "None"
+                },
+                "GUID": "stub_ethernet",
+                "Name": "eth0",
+                "Type": "Ethernet"
+              }], result);
+            }));
+        }));
+  },
   function getVisibleNetworks() {
     chrome.networkingPrivate.getVisibleNetworks(
       "All",
       callbackPass(function(result) {
         assertEq([{
                     "ConnectionState": "Connected",
                     "Ethernet": {
                       "Authentication": "None"
                     },
                     "GUID": "stub_ethernet",
                     "Name": "eth0",
                     "Type": "Ethernet"
                   },
                   {
                     "Connectable": true,
                     "ConnectionState": "Connected",
                     "GUID": "stub_wifi1",
                     "Name": "wifi1",
                     "Type": "WiFi",
                     "WiFi": {
                       "Security": "WEP-PSK",
                       "SignalStrength": 40
                     }
                   },
                   {
                     "ConnectionState": "Connected",
                     "GUID": "stub_vpn1",
                     "Name": "vpn1",
-                    "Type": "VPN",
+                    "Type": "VPN"
                   },
                   {
                     "Connectable": true,
                     "ConnectionState": "NotConnected",
                     "GUID": "stub_wifi2",
                     "Name": "wifi2_PSK",
                     "Type": "WiFi",
                     "WiFi": {
                       "Security": "WPA-PSK",
                       "SignalStrength": 80
@@ skipping 128 matching lines @@
                        "Effective": "Unmanaged"
                      }
                    }
                  }, result);
       }));
   },
   function setProperties() {
     var done = chrome.test.callbackAdded();
     var network_guid = "stub_wifi2";
     chrome.networkingPrivate.getProperties(
-      network_guid,
-      callbackPass(function(result) {
-        assertEq(network_guid, result.GUID);
-        result.WiFi.Security = "WEP-PSK";
-        chrome.networkingPrivate.setProperties("stub_wifi2", result,
-          callbackPass(function() {
-            chrome.networkingPrivate.getProperties(
-              "stub_wifi2",
-              callbackPass(function(result) {
-                // Ensure that the property was set.
-                assertEq("WEP-PSK", result.WiFi.Security);
-                // Ensure that the GUID doesn't change.
-                assertEq(network_guid, result.GUID);
-                done();
+        network_guid,
+        callbackPass(function(result) {
+          assertEq(network_guid, result.GUID);
+          result.WiFi.Security = "WEP-PSK";
+          chrome.networkingPrivate.setProperties(
+              network_guid,
+              result,
+              callbackPass(function() {
+                chrome.networkingPrivate.getProperties(
+                    network_guid,
+                    callbackPass(function(result) {
+                      // Ensure that the property was set.
+                      assertEq("WEP-PSK", result.WiFi.Security);
+                      // Ensure that the GUID doesn't change.
+                      assertEq(network_guid, result.GUID);
+                      done();
+                    }));
               }));
-          }));
-      }));
+        }));
   },
   function getState() {
     chrome.networkingPrivate.getState(
       "stub_wifi2",
       callbackPass(function(result) {
         assertEq({
           "Connectable": true,
           "ConnectionState": "NotConnected",
           "GUID": "stub_wifi2",
           "Name": "wifi2_PSK",
@@ skipping 100 matching lines @@
     var listener =
         new privateHelpers.watchForCaptivePortalState('wifi', 'Online', done);
     chrome.test.sendMessage('notifyPortalDetectorObservers');
   },
 ];
 
 var testToRun = window.location.search.substring(1);
 chrome.test.runTests(availableTests.filter(function(op) {
   return op.name == testToRun;
 }));