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

Issue 10540032: dbus: Add test_server.cc used for manual testing. (Closed)

Created:
8 years, 6 months ago by satorux1
Modified:
8 years, 6 months ago
Reviewers:
hashimoto
CC:
chromium-reviews
Visibility:
Public.

Description

dbus: Add test_server.cc used for manual testing. The server was used for investigating crbug.com/126217. end_to_end_async_unittest.cc runs both the server and the client in the same process. It's useful to have a server program that runs as a separate process. BUG=none TEST=out/Debug/dbus_test_server; (from another terminal) dbus-send --print-reply --type=method_call --dest=org.chromium.TestService /org/chromium/TestObject org.chromium.TestInterface.SlowEcho string:hello Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=141023

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+46 lines, -0 lines) Patch
M dbus/dbus.gyp View 1 chunk +18 lines, -0 lines 0 comments Download
A dbus/test_server.cc View 1 chunk +28 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
satorux1
8 years, 6 months ago (2012-06-06 19:15:42 UTC) #1
hashimoto
lgtm This test server is added to help developers doing manual tests, right? Is it ...
8 years, 6 months ago (2012-06-07 01:35:16 UTC) #2
satorux1
8 years, 6 months ago (2012-06-07 16:45:03 UTC) #3
On 2012/06/07 01:35:16, hashimoto wrote:
> lgtm
> 
> This test server is added to help developers doing manual tests, right?
> Is it possible to launch this server from dbus_unittests and use it in
> end_to_end_async_unittest?

I thought about it but not sure it's easily done (i.e. how do we launch the
binary from the test?) but it's a good thing to do. I'll file a bug.

Powered by Google App Engine
This is Rietveld 408576698