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

Unified Diff: chromeos/network/network_configuration_handler_unittest.cc

Issue 2850993002: [CrOS Tether] Add the HasConnectedToHost network field to NetworkStateHandler. (Closed)
Patch Set: stevenjb@ comments. 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: chromeos/network/network_configuration_handler_unittest.cc
diff --git a/chromeos/network/network_configuration_handler_unittest.cc b/chromeos/network/network_configuration_handler_unittest.cc
index d07215f8b6c1822b49b6a46dcb69ce532a45b5ec..eb5b6ea2401f8ce88a3a7751c93bc54ae9d68d93 100644
--- a/chromeos/network/network_configuration_handler_unittest.cc
+++ b/chromeos/network/network_configuration_handler_unittest.cc
@@ -371,7 +371,8 @@ TEST_F(NetworkConfigurationHandlerTest, GetProperties_TetherNetwork) {
// coded to false.
network_state_handler_->AddTetherNetworkState(
kTetherGuid, "TetherNetworkName", "TetherNetworkCarrier",
- 100 /* battery_percentage */, 100 /* signal_strength */);
+ 100 /* battery_percentage */, 100 /* signal_strength */,
+ true /* has_connected_to_host */);
std::string expected_json =
"{\n "
@@ -383,7 +384,7 @@ TEST_F(NetworkConfigurationHandlerTest, GetProperties_TetherNetwork) {
"\"State\": \"\",\n "
"\"Tether.BatteryPercentage\": 100,\n "
"\"Tether.Carrier\": \"TetherNetworkCarrier\",\n "
- "\"Tether.HasConnectedToHost\": false,\n "
+ "\"Tether.HasConnectedToHost\": true,\n "
"\"Tether.SignalStrength\": 100,\n "
"\"Type\": \"wifi-tether\"\n"
"}\n";
« no previous file with comments | « chromeos/components/tether/tether_connector_unittest.cc ('k') | chromeos/network/network_connection_handler_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698