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

Unified Diff: chromeos/network/network_configuration_handler_unittest.cc

Issue 2852693004: [CrOS Tether] Create HostScanCache, which caches scan results and inserts them into the network sta… (Closed)
Patch Set: Fixed comment, added tests for TestHostResponseRecorder::Observer. 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";

Powered by Google App Engine
This is Rietveld 408576698