[Telepathy] [Bug 18404] New: tests/twisted/roster/ensure.py has a race condition

[bugzilla-daemon at freedesktop.org]

http://bugs.freedesktop.org/show_bug.cgi?id=18404

           Summary: tests/twisted/roster/ensure.py has a race condition
           Product: Telepathy
           Version: unspecified
          Platform: Other
        OS/Version: All
            Status: ASSIGNED
          Severity: normal
          Priority: medium
         Component: telepathy-gabble
        AssignedTo: simon.mcvittie at collabora.co.uk
        ReportedBy: simon.mcvittie at collabora.co.uk
         QAContact: telepathy at lists.freedesktop.org


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.


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