diff options
author | Lance Stout <lancestout@gmail.com> | 2011-07-01 15:15:13 -0700 |
---|---|---|
committer | Lance Stout <lancestout@gmail.com> | 2011-07-01 15:18:10 -0700 |
commit | 2a2ac73845ffc8695e2bc55746f45e1a18d55e6c (patch) | |
tree | 7547644de0fa731d2d2e9f466fc0f33663262575 /tests/test_stream_xep_0030.py | |
parent | d8d9e8df16c07bd13bbac72e4445a2930407b244 (diff) | |
download | slixmpp-2a2ac73845ffc8695e2bc55746f45e1a18d55e6c.tar.gz slixmpp-2a2ac73845ffc8695e2bc55746f45e1a18d55e6c.tar.bz2 slixmpp-2a2ac73845ffc8695e2bc55746f45e1a18d55e6c.tar.xz slixmpp-2a2ac73845ffc8695e2bc55746f45e1a18d55e6c.zip |
So using sys.excepthook to catch errors only works once.
The error bubbles through the event processing loop, breaking it and
hanging the application.
Instead, there is now a .exception(e) method on XMLStream which may
be overridden or reassigned that will receive all unhandled exceptions
(read: not XMPPError) from event and stream handlers.
Diffstat (limited to 'tests/test_stream_xep_0030.py')
-rw-r--r-- | tests/test_stream_xep_0030.py | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/tests/test_stream_xep_0030.py b/tests/test_stream_xep_0030.py index c960fc7a..1666d3a1 100644 --- a/tests/test_stream_xep_0030.py +++ b/tests/test_stream_xep_0030.py @@ -12,7 +12,6 @@ class TestStreamDisco(SleekTest): """ def tearDown(self): - sys.excepthook = sys.__excepthook__ self.stream_close() def testInfoEmptyDefaultNode(self): @@ -531,11 +530,6 @@ class TestStreamDisco(SleekTest): raised_exceptions = [] - def catch_exception(*args, **kwargs): - raised_exceptions.append(True) - - sys.excepthook = catch_exception - self.stream_start(mode='client', plugins=['xep_0030', 'xep_0059']) @@ -544,8 +538,14 @@ class TestStreamDisco(SleekTest): iterator=True) results.amount = 10 + def run_test(): + try: + results.next() + except StopIteration: + raised_exceptions.append(True) + t = threading.Thread(name="get_items_iterator", - target=results.next) + target=run_test) t.start() self.send(""" |