commit 8abd2f67616c3d138346781a49a40d4ae2bc7019
author mukesh agrawal <quiche@chromium.org> Mon Jan 30 14:56:14 2012 -0800
committer Gerrit <chrome-bot@google.com> Wed Feb 08 15:20:05 2012 -0800
tree 58836f7a6a8064030d04b8810fcda36c5317a374
parent 4d260daebf9af9eeba028ad852878e9f2611b4ba

shill: teach dbus_adaptor (and appropriate subclasses) how
to clear properties

BUG=chromium-os:24814
TEST=new unit tests

Change-Id: Ifad5ad0ad6c00d906bade7e1293540a58ac15625
Reviewed-on: https://gerrit.chromium.org/gerrit/15291
Reviewed-by: mukesh agrawal <quiche@chromium.org>
Tested-by: mukesh agrawal <quiche@chromium.org>
Commit-Ready: mukesh agrawal <quiche@chromium.org>

device_unittest.cc

diff --git a/device_unittest.cc b/device_unittest.cc
index c0c426b..8378f3f 100644
--- a/device_unittest.cc
+++ b/device_unittest.cc
@@ -120,6 +120,38 @@
   EXPECT_EQ(invalid_args(), error.name());
 }
 
+TEST_F(DeviceTest, ClearProperty) {
+  ::DBus::Error error;
+  EXPECT_TRUE(device_->powered());
+
+  EXPECT_TRUE(DBusAdaptor::DispatchOnType(device_->mutable_store(),
+                                          flimflam::kPoweredProperty,
+                                          PropertyStoreTest::kBoolV,
+                                          &error));
+  EXPECT_FALSE(device_->powered());
+
+  EXPECT_TRUE(DBusAdaptor::ClearProperty(device_->mutable_store(),
+                                         flimflam::kPoweredProperty,
+                                         &error));
+  EXPECT_TRUE(device_->powered());
+}
+
+TEST_F(DeviceTest, ClearReadOnlyProperty) {
+  ::DBus::Error error;
+  EXPECT_FALSE(DBusAdaptor::DispatchOnType(device_->mutable_store(),
+                                           flimflam::kAddressProperty,
+                                           PropertyStoreTest::kStringV,
+                                           &error));
+}
+
+TEST_F(DeviceTest, ClearReadOnlyDerivedProperty) {
+  ::DBus::Error error;
+  EXPECT_FALSE(DBusAdaptor::DispatchOnType(device_->mutable_store(),
+                                           flimflam::kIPConfigsProperty,
+                                           PropertyStoreTest::kStringsV,
+                                           &error));
+}
+
 TEST_F(DeviceTest, TechnologyIs) {
   EXPECT_FALSE(device_->TechnologyIs(Technology::kEthernet));
 }