diff options
author | Trygve Aaberge <trygveaa@gmail.com> | 2018-06-11 21:19:03 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-06-11 21:19:03 +0200 |
commit | eea510fb7839a0269fc637a9d91bb935bef60486 (patch) | |
tree | 20f093bf644a1604edeac26e89f3ca3faf24876c /_pytest/test_eventrouter.py | |
parent | dc15658f300f6ce2ff57ab3f493f6d72ee097db9 (diff) | |
download | wee-slack-eea510fb7839a0269fc637a9d91bb935bef60486.tar.gz |
Don't raise exception on missing event callbacks (#589)
These missing callbacks are not really errors or exceptions, and we
don't need to implement all of them. Raising an exception makes the
users think something isn't working, and is confusing.
Diffstat (limited to '_pytest/test_eventrouter.py')
-rw-r--r-- | _pytest/test_eventrouter.py | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/_pytest/test_eventrouter.py b/_pytest/test_eventrouter.py index b235512..94301d8 100644 --- a/_pytest/test_eventrouter.py +++ b/_pytest/test_eventrouter.py @@ -1,5 +1,5 @@ import pytest -from wee_slack import EventRouter, ProcessNotImplemented, SlackRequest +from wee_slack import EventRouter, SlackRequest def test_EventRouter(mock_weechat): # Sending valid json adds to the queue. @@ -24,12 +24,11 @@ def test_EventRouter(mock_weechat): assert len(e.queue) == 0 # Handling an event without an associated processor - # raises an exception. + # shouldn't raise an exception. e = EventRouter() # Create a function to test we are called e.receive_json('{"type": "testfunc"}') - with pytest.raises(ProcessNotImplemented): - e.handle_next() + e.handle_next() def test_EventRouterReceivedata(mock_weechat): |