Gabble is going to be pretty annoying to port to 'next' but we can at least get a head start on the tests.
Created attachment 86201 [details] [review] 01/10] placate coding-style check
Created attachment 86202 [details] [review] 02/10] Tests: replace GetSelfHandle calls with property-getting sed expression: s/\.GetSelfHandle()/.Properties.Get(cs.CONN, "SelfHandle")/g adjusted by hand to exclude Group.GetSelfHandle() calls.
Created attachment 86203 [details] [review] 03/10] Avoid calling Group.GetSelfHandle()
Created attachment 86204 [details] [review] 04/10] constants: factor out PREFIX This will mean we can switch over to im.telepathy1 more easily. --- I'd like to get this into all the CMs at some point.
Created attachment 86205 [details] [review] 05/10] tests: use constants instead of o.fd.T
Created attachment 86206 [details] [review] 06/10] Add inspect_contact_sync, inspect_contacts_sync and use in simple cases Telepathy 1.0 doesn't have InspectHandles.
Created attachment 86207 [details] [review] 07/10] groups-12792 test: remove unused function
Created attachment 86208 [details] [review] 08/10] MUC tests: modernize a bit We're better off using MembersChangedDetailed, which maps 1:1 into MembersChanged in Telepathy 1.0. Telepathy 1.0 no longer has RequestHandles, so cut out the middle-man and request MUC channels by TargetID. Hard-coding that the first two contacts we see are handles 2 and 3 is just horrible - let's not do that.
Created attachment 86209 [details] [review] 09/10] muc/roomlist.py: stop testing pre-Requests API We have equivalent test coverage for CreateChannel(). Also stop listening for the NewChannel signal.
Created attachment 86210 [details] [review] 10/10] Add get_contact_handle_sync, get_contact_handles_sync; use in simple cases
I'm probably going to concentrate on Haze or MC after these, so someone else is welcome to pick up the (large!) Gabble job.
Comment on attachment 86201 [details] [review] 01/10] placate coding-style check Review of attachment 86201 [details] [review]: ----------------------------------------------------------------- ++ (sorry)
Comment on attachment 86202 [details] [review] 02/10] Tests: replace GetSelfHandle calls with property-getting Review of attachment 86202 [details] [review]: ----------------------------------------------------------------- ++
Comment on attachment 86203 [details] [review] 03/10] Avoid calling Group.GetSelfHandle() Review of attachment 86203 [details] [review]: ----------------------------------------------------------------- ++
Comment on attachment 86204 [details] [review] 04/10] constants: factor out PREFIX Review of attachment 86204 [details] [review]: ----------------------------------------------------------------- ++
Comment on attachment 86205 [details] [review] 05/10] tests: use constants instead of o.fd.T Review of attachment 86205 [details] [review]: ----------------------------------------------------------------- ++
Comment on attachment 86206 [details] [review] 06/10] Add inspect_contact_sync, inspect_contacts_sync and use in simple cases Review of attachment 86206 [details] [review]: ----------------------------------------------------------------- ++
Comment on attachment 86207 [details] [review] 07/10] groups-12792 test: remove unused function Review of attachment 86207 [details] [review]: ----------------------------------------------------------------- ++
Comment on attachment 86208 [details] [review] 08/10] MUC tests: modernize a bit Review of attachment 86208 [details] [review]: ----------------------------------------------------------------- ++
Comment on attachment 86209 [details] [review] 09/10] muc/roomlist.py: stop testing pre-Requests API Review of attachment 86209 [details] [review]: ----------------------------------------------------------------- ++
Comment on attachment 86210 [details] [review] 10/10] Add get_contact_handle_sync, get_contact_handles_sync; use in simple cases Review of attachment 86210 [details] [review]: ----------------------------------------------------------------- ++
-- GitLab Migration Automatic Message -- This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity. You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/telepathy/telepathy-gabble/issues/263.
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.