Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4591)

Unified Diff: chrome/test/data/extensions/api_test/networking/test.js

Issue 12676017: Adding policy support to the new network configuration stack. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed clang errors. Created 7 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/common/extensions/api/networking_private.json ('k') | chromeos/chromeos.gyp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/data/extensions/api_test/networking/test.js
diff --git a/chrome/test/data/extensions/api_test/networking/test.js b/chrome/test/data/extensions/api_test/networking/test.js
index 6df1e910968321226c2c9d5a9378badeec618fd0..0892c4e7b5bdf7134011ea2ad2c32b9055607462 100644
--- a/chrome/test/data/extensions/api_test/networking/test.js
+++ b/chrome/test/data/extensions/api_test/networking/test.js
@@ -25,7 +25,7 @@ var privateHelpers = {
var collectProperties = function(properties) {
var finishTest = function() {
chrome.networkingPrivate.onNetworksChanged.removeListener(
- self.watchForConnect);
+ self.onNetworkChange);
done();
};
var expectedState = expectedStates.pop();
@@ -35,18 +35,19 @@ var privateHelpers = {
};
this.onNetworkChange = function(changes) {
assertEq([network], changes);
- chrome.networkingPrivate.getProperties(network,
- collectProperties.bind(undefined));
+ chrome.networkingPrivate.getProperties(
+ network,
+ callbackPass(collectProperties));
};
chrome.networkingPrivate.onNetworksChanged.addListener(
- this.onNetworkChange);
+ this.onNetworkChange);
},
listListener: function(network, expected, done) {
var self = this;
this.listenForChanges = function(list) {
assertEq(expected, list);
chrome.networkingPrivate.onNetworkListChanged.removeListener(
- self.listenForChanges);
+ self.listenForChanges);
done();
};
}
@@ -78,7 +79,6 @@ var availableTests = [
chrome.networkingPrivate.getVisibleNetworks(
"All",
callbackPass(function(result) {
- assertTrue(!!result);
assertEq([{
"ConnectionState": "Connected",
"GUID": "stub_ethernet",
@@ -109,6 +109,7 @@ var availableTests = [
},
{
"Cellular": {
+ "ActivateOverNonCellularNetwork": false,
"ActivationState": "not-activated",
"NetworkTechnology": "GSM",
"RoamingState": "home"
@@ -133,7 +134,6 @@ var availableTests = [
chrome.networkingPrivate.getVisibleNetworks(
"WiFi",
callbackPass(function(result) {
- assertTrue(!!result);
assertEq([{
"ConnectionState": "Connected",
"GUID": "stub_wifi1",
@@ -176,7 +176,6 @@ var availableTests = [
chrome.networkingPrivate.getProperties(
"stub_wifi2",
callbackPass(function(result) {
- assertTrue(!!result);
assertEq({
"ConnectionState": "NotConnected",
"GUID": "stub_wifi2",
@@ -190,22 +189,70 @@ var availableTests = [
}, result);
}));
},
+ function getManagedProperties() {
+ chrome.networkingPrivate.getManagedProperties(
+ "stub_wifi2",
+ callbackPass(function(result) {
+ assertEq({
+ "ConnectionState": {
+ "Active": "NotConnected",
+ "Effective": "Unmanaged"
+ },
+ "GUID": "stub_wifi2",
+ "Name": {
+ "Active": "wifi2_PSK",
+ "Effective": "UserPolicy",
+ "UserPolicy": "My WiFi Network"
+ },
+ "Type": {
+ "Active": "WiFi",
+ "Effective": "UserPolicy",
+ "UserPolicy": "WiFi"
+ },
+ "WiFi": {
+ "AutoConnect": {
+ "Active": false,
+ "UserEditable": true
+ },
+ "Passphrase": {
+ "Effective": "UserSetting",
+ "UserEditable": true,
+ "UserSetting": "FAKE_CREDENTIAL_VPaJDV9x"
+ },
+ "SSID": {
+ "Active": "stub_wifi2",
+ "Effective": "UserPolicy",
+ "UserPolicy": "stub_wifi2"
+ },
+ "Security": {
+ "Active": "WPA-PSK",
+ "Effective": "UserPolicy",
+ "UserPolicy": "WPA-PSK"
+ },
+ "SignalStrength": {
+ "Active": 80,
+ "Effective": "Unmanaged"
+ }
+ }
+ }, result);
+ }));
+ },
function setProperties() {
var done = chrome.test.callbackAdded();
chrome.networkingPrivate.getProperties(
"stub_wifi2",
- function(result) {
+ callbackPass(function(result) {
result.WiFi.Security = "WEP-PSK";
chrome.networkingPrivate.setProperties("stub_wifi2", result,
- function() {
+ callbackPass(function() {
chrome.networkingPrivate.getProperties(
"stub_wifi2",
- function(result) {
+ callbackPass(function(result) {
assertEq("WEP-PSK", result.WiFi.Security);
done();
- });
- });
- });
+ }));
+ }));
+ }));
},
function getState() {
chrome.networkingPrivate.getState(
@@ -213,7 +260,7 @@ var availableTests = [
callbackPass(function(result) {
assertEq({
"ConnectionState": "NotConnected",
- "GUID": "",
+ "GUID": "stub_wifi2",
"Name": "wifi2_PSK",
"Type": "WiFi",
"WiFi": {
« no previous file with comments | « chrome/common/extensions/api/networking_private.json ('k') | chromeos/chromeos.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698