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

Unified Diff: chrome/browser/extensions/api/networking_private/networking_private_chromeos_apitest.cc

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
« no previous file with comments | « no previous file | chrome/test/data/extensions/api_test/networking_private/chromeos/test.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/extensions/api/networking_private/networking_private_chromeos_apitest.cc
diff --git a/chrome/browser/extensions/api/networking_private/networking_private_chromeos_apitest.cc b/chrome/browser/extensions/api/networking_private/networking_private_chromeos_apitest.cc
index 00c1c9e251b108e63a7ffbcabe4d0cc29392c815..fbe85e25092b1430319ca0af1d92fd1221da502e 100644
--- a/chrome/browser/extensions/api/networking_private/networking_private_chromeos_apitest.cc
+++ b/chrome/browser/extensions/api/networking_private/networking_private_chromeos_apitest.cc
@@ -300,6 +300,19 @@ class NetworkingPrivateChromeOSApiTest : public ExtensionApiTest {
content::RunAllPendingInMessageLoop();
}
+ void SetupTether() {
+ chromeos::NetworkStateHandler* network_state_handler =
+ chromeos::NetworkHandler::Get()->network_state_handler();
+ network_state_handler->SetTetherTechnologyState(
+ chromeos::NetworkStateHandler::TechnologyState::TECHNOLOGY_ENABLED);
+ network_state_handler->AddTetherNetworkState(
+ "tetherGuid1", "tetherName1", "tetherCarrier1",
+ 50 /* battery_percentage */, 75 /* signal_strength */);
+ network_state_handler->AddTetherNetworkState(
+ "tetherGuid2", "tetherName2", "tetherCarrier2",
+ 75 /* battery_percentage */, 100 /* signal_strength */);
+ }
+
void AddService(const std::string& service_path,
const std::string& name,
const std::string& type,
@@ -851,6 +864,31 @@ IN_PROC_BROWSER_TEST_F(NetworkingPrivateChromeOSApiTest, GetGlobalPolicy) {
EXPECT_TRUE(RunNetworkingSubtest("getGlobalPolicy")) << message_;
}
+IN_PROC_BROWSER_TEST_F(NetworkingPrivateChromeOSApiTest,
+ Tether_GetTetherNetworks) {
+ SetupTether();
+ EXPECT_TRUE(RunNetworkingSubtest("getTetherNetworks")) << message_;
+}
+
+IN_PROC_BROWSER_TEST_F(NetworkingPrivateChromeOSApiTest,
+ Tether_GetTetherNetworkProperties) {
+ SetupTether();
+ EXPECT_TRUE(RunNetworkingSubtest("getTetherNetworkProperties")) << message_;
+}
+
+IN_PROC_BROWSER_TEST_F(NetworkingPrivateChromeOSApiTest,
+ Tether_GetTetherNetworkManagedProperties) {
+ SetupTether();
+ EXPECT_TRUE(RunNetworkingSubtest("getTetherNetworkManagedProperties"))
+ << message_;
+}
+
+IN_PROC_BROWSER_TEST_F(NetworkingPrivateChromeOSApiTest,
+ Tether_GetTetherNetworkState) {
+ SetupTether();
+ EXPECT_TRUE(RunNetworkingSubtest("getTetherNetworkState")) << message_;
+}
+
// Tests subset of networking API for the networking API alias - to verify that
// using API methods and event does not cause access exceptions (due to
// missing permissions).
« no previous file with comments | « no previous file | chrome/test/data/extensions/api_test/networking_private/chromeos/test.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698