dbus: verify object path of incoming signals

dbus/end_to_end_async_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <algorithm>
 #include <string>
 #include <vector>
 
 #include "base/bind.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 142 matching lines @@
     ASSERT_TRUE(success);
     message_loop_.Quit();
   }
 
   // Wait for the hey signal to be received.
   void WaitForTestSignal() {
     // OnTestSignal() will quit the message loop.
     message_loop_.Run();
   }
 
+  // Wait for the hey signal to be received, or a timeout.
+  void WaitForTestSignalOrTimeout(int timeout_ms) {
+    // OnTestSignal() will quit the message loop, we don't expect that
+    // to be called, so post a delayed task to quit the message loop
+    // for when it isn't.
+    message_loop_.PostDelayedTask(FROM_HERE,
+                                  MessageLoop::QuitClosure(),
+                                  timeout_ms);
+    message_loop_.Run();
+  }
+
   MessageLoop message_loop_;
   std::vector<std::string> response_strings_;
   scoped_ptr<base::Thread> dbus_thread_;
   scoped_refptr<dbus::Bus> bus_;
   dbus::ObjectProxy* object_proxy_;
   scoped_ptr<dbus::TestService> test_service_;
   // Text message from "Test" signal.
   std::string test_signal_string_;
 };
 
@@ skipping 122 matching lines @@
   // Send the test signal from the exported object.
   test_service_->SendTestSignal(kMessage);
   // Receive the signal with the object proxy. The signal is handled in
   // EndToEndAsyncTest::OnTestSignal() in the main thread.
   WaitForTestSignal();
   ASSERT_EQ(kMessage, test_signal_string_);
 }
 
 TEST_F(EndToEndAsyncTest, TestSignalFromRoot) {
   const char kMessage[] = "hello, world";
-  // Send the test signal from the root object path, to see if we can
-  // handle signals sent from "/", like dbus-send does.
+  // Object proxies are tied to a particular object path, if a signal
+  // arrives from a different object path like "/" the proxy should not
+  // handle it, and should leave it for another.
   test_service_->SendTestSignalFromRoot(kMessage);
-  // Receive the signal with the object proxy. The signal is handled in
-  // EndToEndAsyncTest::OnTestSignal() in the main thread.
-  WaitForTestSignal();
-  ASSERT_EQ(kMessage, test_signal_string_);
+  // Timeout while waiting to receive the signal, failing if we received it.
+  const int timeout_ms = 5000;
+  WaitForTestSignalOrTimeout(timeout_ms);

[satorux1, 2012/03/01 00:14:32]

Does this mean the test takes 5 seconds to perform? It's a bit too long for a
unit test. We should probably use action_timeout_ms() in
base/test/test_timeouts.h

[keybuk, 2012/03/01 00:15:22]

yes.

action_timeout_ms is 10s, which is even longer

but tiny_timeout_ms was too short to guarantee the signal got through dbus (the
test failed 3/7 times I tried)

[satorux1, 2012/03/01 00:23:46]

I see. Then let's remove this test. 

A better way to test this is to check if ObjectProxy rejects  signals from wrong
senders, but we don't have ability to observe the behavior from outside.

+  // Verify the signal was not received
+  ASSERT_TRUE(test_signal_string_.empty());
 }