[Bug 55666] only monitors SelfHandleChanged if the CM is obsolete

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Fri Oct 5 17:15:05 CEST 2012


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

--- Comment #3 from Simon McVittie <simon.mcvittie at collabora.co.uk> ---
Created attachment 68124
  --> https://bugs.freedesktop.org/attachment.cgi?id=68124&action=edit
_tp_dbus_properties_mixin_get_all: expose to internal  code

The self-handle test can't exercise certain situations without this,
except by pretending to be an obsolete CM, which means we don't test
the non-obsolete code path properly.

---

If we want the test that I'm about to attach on stable branches, we'll have to
do this, since the alternative is to add ABI.

If we're only adding the test to 0.21 then we could just rename to
tp_dbus_properties_mixin_get_all(), make it public and document it; but it's
dbus-glib-type-based so I think we should (skip) it.

Or I could add a public-API version in terms of GVariant if people want...

-- 
You are receiving this mail because:
You are the QA Contact for the bug.



More information about the telepathy-bugs mailing list