Bug 18404 - tests/twisted/roster/ensure.py has a race condition
Summary: tests/twisted/roster/ensure.py has a race condition
Status: RESOLVED FIXED
Alias: None
Product: Telepathy
Classification: Unclassified
Component: gabble (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: Simon McVittie
QA Contact: Telepathy bugs list
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-11-06 02:05 UTC by Simon McVittie
Modified: 2008-11-06 03:40 UTC (History)
0 users

See Also:
i915 platform:
i915 features:

Attachments

Description Simon McVittie 2008-11-06 02:05:33 UTC 
In tests/twisted/roster/ensure.py we expect two EnsureChannel calls to be processed together. If the first call is completely processed before the second is dispatched, the NewChannels signal might happen between the two returns, rather than after them both; this causes intermittent test failures.

My suggested workaround is to just delete the expectation of NewChannels (we know it works, and if it doesn't, it's telepathy-glib's fault anyway). I'm filing this as a bug rather than committing it because I don't want to block the 0.7.15 release on it.
Comment 1 Simon McVittie 2008-11-06 03:40:48 UTC 
Fixed in git.

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.