From 7734aee7ad43bddc8bd1701f2d2773c4bbba6933 Mon Sep 17 00:00:00 2001 From: Lance Stout Date: Sat, 7 Apr 2012 17:19:39 -0400 Subject: Prevent roster_update from firing twice after retrieving the roster. --- tests/test_stream_roster.py | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) (limited to 'tests/test_stream_roster.py') diff --git a/tests/test_stream_roster.py b/tests/test_stream_roster.py index 2d0da173..652ea1ce 100644 --- a/tests/test_stream_roster.py +++ b/tests/test_stream_roster.py @@ -24,7 +24,11 @@ class TestStreamRoster(SleekTest): def roster_received(iq): events.append('roster_received') + def roster_update(iq): + events.append('roster_update') + self.xmpp.add_event_handler('roster_received', roster_received) + self.xmpp.add_event_handler('roster_update', roster_update) # Since get_roster blocks, we need to run it in a thread. t = threading.Thread(name='get_roster', target=self.xmpp.get_roster) @@ -62,8 +66,8 @@ class TestStreamRoster(SleekTest): # Give the event queue time to process. time.sleep(.1) - self.failUnless('roster_received' in events, - "Roster received event not triggered: %s" % events) + self.failUnless(events == ['roster_received', 'roster_update'], + "Wrong roster events fired: %s" % events) def testRosterSet(self): """Test handling pushed roster updates.""" -- cgit v1.2.3