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

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

Issue 694533007: Add 'setProperties' to InternetOptionsHandler (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase + feedback Created 6 years, 1 month 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
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 c3e8b622e658be9e776d70fe357f463940353040..efa8dc588204de440ba0f0e58c235ba89e5d5578 100644
--- a/chrome/test/data/extensions/api_test/networking/test.js
+++ b/chrome/test/data/extensions/api_test/networking/test.js
@@ -373,23 +373,64 @@ var availableTests = [
}, result);
}));
},
- function setProperties() {
+ function setWiFiProperties() {
var done = chrome.test.callbackAdded();
- var network_guid = "stub_wifi2_guid";
+ var network_guid = "stub_wifi1_guid";
+ chrome.networkingPrivate.getProperties(
+ network_guid,
+ callbackPass(function(result) {
+ assertEq(network_guid, result.GUID);
+ var new_properties = {
+ Priority: 1,
+ WiFi: {
+ AutoConnect: true
+ }
+ };
+ chrome.networkingPrivate.setProperties(
+ network_guid,
+ new_properties,
+ callbackPass(function() {
+ chrome.networkingPrivate.getProperties(
+ network_guid,
+ callbackPass(function(result) {
+ // Ensure that the properties were set.
+ assertEq(1, result['Priority']);
+ assertTrue('WiFi' in result);
+ assertTrue('AutoConnect' in result['WiFi']);
+ assertEq(true, result['WiFi']['AutoConnect']);
+ // Ensure that the GUID doesn't change.
+ assertEq(network_guid, result.GUID);
+ done();
+ }));
+ }));
+ }));
+ },
+ function setVPNProperties() {
+ var done = chrome.test.callbackAdded();
+ var network_guid = "stub_vpn1_guid";
chrome.networkingPrivate.getProperties(
network_guid,
callbackPass(function(result) {
assertEq(network_guid, result.GUID);
- result.WiFi.Security = "WEP-PSK";
+ var new_properties = {
+ Priority: 1,
+ VPN: {
+ Host: 'vpn.host1'
+ }
+ };
chrome.networkingPrivate.setProperties(
network_guid,
- result,
+ new_properties,
callbackPass(function() {
chrome.networkingPrivate.getProperties(
network_guid,
callbackPass(function(result) {
- // Ensure that the property was set.
- assertEq("WEP-PSK", result.WiFi.Security);
+ console.log('Result: ' + JSON.stringify(result));
+ // Ensure that the properties were set.
+ assertEq(1, result['Priority']);
+ assertTrue('VPN' in result);
+ assertTrue('Host' in result['VPN']);
+ assertEq('vpn.host1', result['VPN']['Host']);
// Ensure that the GUID doesn't change.
assertEq(network_guid, result.GUID);
done();
« no previous file with comments | « chrome/browser/ui/webui/options/chromeos/internet_options_handler.cc ('k') | chromeos/dbus/fake_shill_service_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698