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

Unified Diff: chrome/test/data/extensions/api_test/networking_private/chromeos/test.js

Issue 2847453002: [CrOS Tether] Remove the ManagedTetherProperties dictionary. (Closed)
Patch Set: Fixed broken OncValidator test. Created 3 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
Index: chrome/test/data/extensions/api_test/networking_private/chromeos/test.js
diff --git a/chrome/test/data/extensions/api_test/networking_private/chromeos/test.js b/chrome/test/data/extensions/api_test/networking_private/chromeos/test.js
index 69dde01398a12f2aabd7b18f64ba2fda36febefa..3cd6f2e3111e2cd782441b52f2bbef9c5ce909de 100644
--- a/chrome/test/data/extensions/api_test/networking_private/chromeos/test.js
+++ b/chrome/test/data/extensions/api_test/networking_private/chromeos/test.js
@@ -81,6 +81,18 @@ var privateHelpers = {
};
chrome.networkingPrivate.onPortalDetectionCompleted.addListener(
self.onPortalDetectionCompleted);
+ },
+ verifyTetherNetwork: function(
+ properties, expectedGuid, expectedName, expectedBatteryPercentage,
+ expectedCarrier, expectedSignalStrength) {
+ //assertEq(NetworkType.Tether, properties.Type);
+ assertEq(expectedGuid, properties.GUID);
+ assertEq(expectedName,
+ properties.Name.hasOwnProperty('Active') ? properties.Name.Active
+ : properties.Name);
+ assertEq(expectedBatteryPercentage, properties.Tether.BatteryPercentage);
+ assertEq(expectedCarrier, properties.Tether.Carrier);
+ assertEq(expectedSignalStrength, properties.Tether.SignalStrength);
}
};
@@ -923,6 +935,41 @@ var availableTests = [
}, result);
}));
},
+ function getTetherNetworks() {
+ chrome.networkingPrivate.getNetworks(
+ {networkType: 'Tether'},
+ callbackPass(function(tetherNetworks) {
+ assertEq(2, tetherNetworks.length);
+ privateHelpers.verifyTetherNetwork(tetherNetworks[0], 'tetherGuid1',
+ 'tetherName1', 50, 'tetherCarrier1', 75);
+ privateHelpers.verifyTetherNetwork(tetherNetworks[1], 'tetherGuid2',
+ 'tetherName2', 75, 'tetherCarrier2', 100);
+ }));
+ },
+ function getTetherNetworkProperties() {
+ chrome.networkingPrivate.getProperties(
+ 'tetherGuid1',
+ callbackPass(function(tetherNetwork) {
+ privateHelpers.verifyTetherNetwork(tetherNetwork, 'tetherGuid1',
+ 'tetherName1', 50, 'tetherCarrier1', 75);
+ }));
+ },
+ function getTetherNetworkManagedProperties() {
+ chrome.networkingPrivate.getManagedProperties(
+ 'tetherGuid1',
+ callbackPass(function(tetherNetwork) {
+ privateHelpers.verifyTetherNetwork(tetherNetwork, 'tetherGuid1',
+ 'tetherName1', 50, 'tetherCarrier1', 75);
+ }));
+ },
+ function getTetherNetworkState() {
+ chrome.networkingPrivate.getState(
+ 'tetherGuid1',
+ callbackPass(function(tetherNetwork) {
+ privateHelpers.verifyTetherNetwork(tetherNetwork, 'tetherGuid1',
+ 'tetherName1', 50, 'tetherCarrier1', 75);
+ }));
+ },
];
chrome.test.getConfig(function(config) {

Powered by Google App Engine
This is Rietveld 408576698