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

Unified Diff: chromeos/dbus/flimflam_manager_client_unittest.cc

Issue 10177001: Reimplement CrosSetOfflineMode without Libcros (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: _ Created 8 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/dbus/flimflam_manager_client_unittest.cc
diff --git a/chromeos/dbus/flimflam_manager_client_unittest.cc b/chromeos/dbus/flimflam_manager_client_unittest.cc
index 6dcc0df2fd3ac6c78ded8a50b858acc91d1812e1..a3b0c6ff94a127aea777b48df2428646b58f153c 100644
--- a/chromeos/dbus/flimflam_manager_client_unittest.cc
+++ b/chromeos/dbus/flimflam_manager_client_unittest.cc
@@ -187,6 +187,22 @@ TEST_F(FlimflamManagerClientTest, SetProperty) {
message_loop_.RunAllPending();
}
+TEST_F(FlimflamManagerClientTest, CallSetPropertyAndBlock) {
+ // Create response.
+ scoped_ptr<dbus::Response> response(dbus::Response::CreateEmpty());
+ // Set expectations.
+ base::StringValue value("portal list");
+ PrepareForMethodCall(flimflam::kSetPropertyFunction,
+ base::Bind(ExpectStringAndValueArguments,
+ flimflam::kCheckPortalListProperty,
+ &value),
+ response.get());
+ // Call method.
+ const bool result = client_->CallSetPropertyAndBlock(
+ flimflam::kCheckPortalListProperty, value);
+ EXPECT_TRUE(result);
stevenjb 2012/04/23 21:42:06 Shouldn't need this.
hashimoto 2012/04/24 04:37:54 Done.
+}
+
TEST_F(FlimflamManagerClientTest, RequestScan) {
// Create response.
scoped_ptr<dbus::Response> response(dbus::Response::CreateEmpty());

Powered by Google App Engine
This is Rietveld 408576698