Bug 24261

Summary: Improved test coverage for plugin API
Product: Telepathy Reporter: Simon McVittie <smcv>
Component: mission-controlAssignee: Telepathy bugs list <telepathy-bugs>
Status: RESOLVED FIXED QA Contact: Telepathy bugs list <telepathy-bugs>
Severity: normal    
Priority: medium Keywords: patch
Version: unspecified   
Hardware: Other   
OS: All   
URL: http://git.collabora.co.uk/?p=user/smcv/telepathy-mission-control-smcv.git;a=shortlog;h=refs/heads/52-test-plugin-delay
Whiteboard:
i915 platform: i915 features:
Bug Depends on:    
Bug Blocks: 24120    

Description Simon McVittie 2009-10-01 12:13:41 UTC
While trying to find out how McdDispatcherContext::cancelled is set, I wrote some more tests. So far I haven't been able to trigger the "cancelled" code path, though.

For 5.2:

http://git.collabora.co.uk/?p=user/smcv/telepathy-mission-control-smcv.git;a=shortlog;h=refs/heads/52-test-plugin-delay

For master (merge is non-trivial due to a behaviour change):

http://git.collabora.co.uk/?p=user/smcv/telepathy-mission-control-smcv.git;a=shortlog;h=refs/heads/test-plugin-delay
Comment 1 Simon McVittie 2009-10-06 05:25:13 UTC
This blocks some of my McdDispatcher refactoring, I think (I don't want to merge it without this test coverage).
Comment 2 Simon McVittie 2009-10-08 06:59:30 UTC
Fixed in 5.2.6 and in master (thanks to Alberto and Sjoerd for 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.