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

Unified Diff: chromeos/network/shill_property_handler_unittest.cc

Issue 181413006: Replace misc. network stub flags with more flexible ones (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address feedback Created 6 years, 10 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/shill_property_handler_unittest.cc
diff --git a/chromeos/network/shill_property_handler_unittest.cc b/chromeos/network/shill_property_handler_unittest.cc
index 431cccf3a695a4386d64a7126cbcc8164ff3ccb9..14cdf8d17cd5816d9d0cdc94b4f85782524ccb40 100644
--- a/chromeos/network/shill_property_handler_unittest.cc
+++ b/chromeos/network/shill_property_handler_unittest.cc
@@ -304,29 +304,29 @@ TEST_F(ShillPropertyHandlerTest, ShillPropertyHandlerTechnologyChanged) {
const int initial_technology_updates = 2; // Available and Enabled lists
EXPECT_EQ(initial_technology_updates, listener_->technology_list_updates());
- // Remove a technology. Updates both the Available and Enabled lists.
- manager_test_->RemoveTechnology(shill::kTypeWimax);
+ // Remove an enabled technology. Updates both the Available and Enabled lists.
+ manager_test_->RemoveTechnology(shill::kTypeWifi);
message_loop_.RunUntilIdle();
EXPECT_EQ(initial_technology_updates + 2,
listener_->technology_list_updates());
// Add a disabled technology.
- manager_test_->AddTechnology(shill::kTypeWimax, false);
+ manager_test_->AddTechnology(shill::kTypeWifi, false);
message_loop_.RunUntilIdle();
EXPECT_EQ(initial_technology_updates + 3,
listener_->technology_list_updates());
EXPECT_TRUE(shill_property_handler_->IsTechnologyAvailable(
- shill::kTypeWimax));
- EXPECT_FALSE(shill_property_handler_->IsTechnologyEnabled(shill::kTypeWimax));
+ shill::kTypeWifi));
+ EXPECT_FALSE(shill_property_handler_->IsTechnologyEnabled(shill::kTypeWifi));
// Enable the technology.
DBusThreadManager::Get()->GetShillManagerClient()->EnableTechnology(
- shill::kTypeWimax,
+ shill::kTypeWifi,
base::Bind(&base::DoNothing), base::Bind(&ErrorCallbackFunction));
message_loop_.RunUntilIdle();
EXPECT_EQ(initial_technology_updates + 4,
listener_->technology_list_updates());
- EXPECT_TRUE(shill_property_handler_->IsTechnologyEnabled(shill::kTypeWimax));
+ EXPECT_TRUE(shill_property_handler_->IsTechnologyEnabled(shill::kTypeWifi));
EXPECT_EQ(0, listener_->errors());
}

Powered by Google App Engine
This is Rietveld 408576698