[Bug 28902] Implement Account.Service property

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Sun Jul 4 13:30:52 CEST 2010


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

Will Thompson <will.thompson at collabora.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Status Whiteboard|                            |r-, minor

--- Comment #3 from Will Thompson <will.thompson at collabora.co.uk> 2010-07-04 04:30:52 PDT ---
The spec doesn't say that Account.Service should fall back to the protocol; it
says that if it's the empty string, the service is determined by the protocol
name. I'm inclined to think that MC should report service-less accounts with
Account.Service = "".

The test doesn't test specifying Service at account creation time.

The patch named 'ibid' should be squashed into the preceding patch (which it
fixes).

+    assert account_props.Get(cs.ACCOUNT, 'Service') == 'fu-bar'

assertEquals

+    # OK, let's go online

but that's not what happens next in the test.

There are a load of unused imports in the test.

(I have not reviewed the branch on which this branch is stacked.)

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



More information about the telepathy-bugs mailing list