commit 8a56ad63b73d221a0f20fd61d1f27bcc443eec50
author Chris Masone <cmasone@chromium.org> Sat Jul 02 15:27:57 2011 -0700
committer Chris Masone <cmasone@chromium.org> Sun Jul 03 10:58:12 2011 -0700
tree 314944097500e34b0966281f975ed9950e1e147a
parent 27c4aa55b33d3a3836cf70c8f7094bce1c5ead8c

[shill] Move some simple unit tests to property_store_unittest.cc

BUG=None
TEST=unit tests

Change-Id: Ib5ee4b3c2654c193227a3b789cbb6f40e5222b85
Reviewed-on: http://gerrit.chromium.org/gerrit/3617
Tested-by: Chris Masone <cmasone@chromium.org>
Reviewed-by: Darin Petkov <petkov@chromium.org>

property_store_unittest.cc

diff --git a/property_store_unittest.cc b/property_store_unittest.cc
index 03b6ee1..2dd049c 100644
--- a/property_store_unittest.cc
+++ b/property_store_unittest.cc
@@ -22,6 +22,7 @@
 using std::string;
 using std::map;
 using std::vector;
+using ::testing::Values;
 
 namespace shill {
 
@@ -62,4 +63,26 @@
 
 PropertyStoreTest::~PropertyStoreTest() {}
 
+TEST_P(PropertyStoreTest, TestProperty) {
+  // Ensure that an attempt to write unknown properties returns InvalidProperty.
+  PropertyStore store;
+  ::DBus::Error error;
+  EXPECT_FALSE(DBusAdaptor::DispatchOnType(&store, "", GetParam(), &error));
+  EXPECT_EQ(invalid_prop_, error.name());
+}
+
+INSTANTIATE_TEST_CASE_P(
+    PropertyStoreTestInstance,
+    PropertyStoreTest,
+    Values(PropertyStoreTest::kBoolV,
+           PropertyStoreTest::kByteV,
+           PropertyStoreTest::kStringV,
+           PropertyStoreTest::kInt16V,
+           PropertyStoreTest::kInt32V,
+           PropertyStoreTest::kUint16V,
+           PropertyStoreTest::kUint32V,
+           PropertyStoreTest::kStringsV,
+           PropertyStoreTest::kStringmapV,
+           PropertyStoreTest::kStringmapsV));
+
 }  // namespace shill