commit f1a77fcb460c16e690306ea1e296d26ba7b90da2
author Alex Vakulenko <avakulenko@google.com> Fri Dec 11 16:14:39 2015 -0800
committer Alex Vakulenko <avakulenko@google.com> Tue Dec 15 07:43:36 2015 -0800
tree f2fb5c8ff121eb4918eb3ccd8ad00b8bc8d2d05a
parent 494b72473a1c436751003c249e1e4a9f81102329

Fix weaved unit tests

GetComponent() method was added to D-Bus object but the call wasn't
mocked out in D-Bus proxy tests, resulting in unwanted call and
broken tests.

Change-Id: I77f2baad7d337994e1d39b65158f0082e09776e0

buffet/dbus_command_proxy_unittest.cc

diff --git a/buffet/dbus_command_proxy_unittest.cc b/buffet/dbus_command_proxy_unittest.cc
index 0aa701d..5225f11 100644
--- a/buffet/dbus_command_proxy_unittest.cc
+++ b/buffet/dbus_command_proxy_unittest.cc
@@ -79,6 +79,8 @@
     // Use WillRepeatedly because GetName is used for logging.
     EXPECT_CALL(*command_, GetName())
         .WillRepeatedly(ReturnRefOfCopy<std::string>("robot.jump"));
+    EXPECT_CALL(*command_, GetComponent())
+        .WillRepeatedly(ReturnRefOfCopy<std::string>("myComponent"));
     EXPECT_CALL(*command_, GetState())
         .WillRepeatedly(Return(weave::Command::State::kQueued));
     EXPECT_CALL(*command_, GetOrigin())
@@ -149,6 +151,7 @@
   EXPECT_EQ(VariantDictionary{}, GetCommandAdaptor()->GetProgress());
   EXPECT_EQ(VariantDictionary{}, GetCommandAdaptor()->GetResults());
   EXPECT_EQ("robot.jump", GetCommandAdaptor()->GetName());
+  EXPECT_EQ("myComponent", GetCommandAdaptor()->GetComponent());
   EXPECT_EQ(kTestCommandId, GetCommandAdaptor()->GetId());
 }