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'.
+    chrome.networkingPrivate.getNetworks(
+      { "networkType": "WiFi", "configured": true },
+      callbackPass(function(result) {
+        assertEq([{
+          "Connectable": true,
+          "ConnectionState": "Connected",
+          "GUID": "stub_wifi1",
+          "Name": "wifi1",
+          "Type": "WiFi",
+          "WiFi": {
+            "Security": "WEP-PSK",
+            "SignalStrength": 40
+          }
+        }, {
+          "GUID": "stub_wifi2",
+          "Name": "wifi2_PSK",
+          "Type": "WiFi",
+          "WiFi": {
+            "Security": "WPA-PSK",
+          }
+        }], result);
+
+        // Test 'visible' (and 'configured').
+        chrome.networkingPrivate.getNetworks(
+          { "networkType": "WiFi", "visible": true, "configured": true },
+          callbackPass(function(result) {
+            assertEq([{
+              "Connectable": true,
+              "ConnectionState": "Connected",
+              "GUID": "stub_wifi1",
+              "Name": "wifi1",
+              "Type": "WiFi",
+              "WiFi": {
+                "Security": "WEP-PSK",
+                "SignalStrength": 40
+              }
+            }], result);
+
+            // Test 'limit'.
+            chrome.networkingPrivate.getNetworks(
+              { "networkType": "All", "configured": true, "limit": 1 },

[pneubeck (no reviews), 2014/05/16 13:18:55]

you should also have a test for 'configured:false'.
Maybe you can make ethernet unconfigured and change this test to
'configured:false' ?

Then you have all "interesting" values of the individual filter options covered.
(sorry that I didn't mention it on the first pass)

[stevenjb, 2014/05/16 17:35:23]

So, configured='false' is not the same as !configured, it means 'don't filter by
configured'. I removed 'configured' from this test for now, which doesn't affect
the result.

We really need a better way to add and maintain these tests. It's on my "TODO"
list for 37.

+              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;
 }));