[Bug 28366] Add DBusTube interface support

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=28366

--- Comment #16 from Dario Freddi <drf54321 at gmail.com> 2011-06-26 14:53:23 PDT ---
(In reply to comment #15)
> Well. There is no ready made test service, no. That's because there is no
> high-level client-side API for dbus tubes in tp-glib either, so nothing
> requires such a service for testing.
> 
> This only means that you have to create the test service yourself, probably by
> modifying tp-glib's StreamTube channel test service as I mentioned in a
> previous comment. tp-glib surely does have the needed low-level service classes
> for implementing a dbus tube channel (one could perhaps look at gabble for
> reference on how to use that). There's also TpGroupMixin for implementing the
> Group interface for testing the bus name monitoring stuff in addition to
> implementing the basic Channel stuff by deriving from TpBaseChannel.
> 
> In any case, automated regression tests are a must for this code. The fact that
> there is no ready-made test service already written by someone else is no
> excuse against that; after all we wouldn't have any unit tests at all if
> somebody didn't write the test service at first.

That's ok, I just wanted to check I was not stepping on other people's toes or
taking over work I shouldn't have done myself. I'll start
researching/implementing that tomorrow, then we'll eventually consider to port
this back to tp-glib if successful.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.