| Summary: | Haze: reduce Telepathy 0.x dependence in regression tests | ||
|---|---|---|---|
| Product: | Telepathy | Reporter: | Simon McVittie <smcv> |
| Component: | haze | Assignee: | Simon McVittie <smcv> |
| Status: | RESOLVED FIXED | QA Contact: | Telepathy bugs list <telepathy-bugs> |
| Severity: | normal | ||
| Priority: | medium | ||
| Version: | git master | ||
| Hardware: | Other | ||
| OS: | All | ||
| Whiteboard: | |||
| i915 platform: | i915 features: | ||
| Attachments: |
1/4] Tests: replace GetSelfHandle calls with property-getting
2/4] haze_im_channel_get_property: fix a TP_SEAL error 3/4] simple-caps: stop relying on GetContactCapabilities 4/4] im-channel.c: include config.h first |
||
|
Description
Simon McVittie
2013-09-20 16:44:54 UTC
Created attachment 86211 [details] [review] 1/4] Tests: replace GetSelfHandle calls with property-getting sed expression: s/\.GetSelfHandle()/.Properties.Get(cs.CONN, "SelfHandle")/g adjusted by hand to add "import constants as cs" where needed. Created attachment 86212 [details] [review] 2/4] haze_im_channel_get_property: fix a TP_SEAL error Created attachment 86213 [details] [review] 3/4] simple-caps: stop relying on GetContactCapabilities 0.99.1 doesn't have it. Created attachment 86214 [details] [review] 4/4] im-channel.c: include config.h first Comment on attachment 86211 [details] [review] 1/4] Tests: replace GetSelfHandle calls with property-getting Review of attachment 86211 [details] [review]: ----------------------------------------------------------------- ++ Comment on attachment 86212 [details] [review] 2/4] haze_im_channel_get_property: fix a TP_SEAL error Review of attachment 86212 [details] [review]: ----------------------------------------------------------------- ++ Comment on attachment 86213 [details] [review] 3/4] simple-caps: stop relying on GetContactCapabilities Review of attachment 86213 [details] [review]: ----------------------------------------------------------------- ++ Comment on attachment 86214 [details] [review] 4/4] im-channel.c: include config.h first Review of attachment 86214 [details] [review]: ----------------------------------------------------------------- ++ |
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.