Marco Rougeth

Middleware example should be async

3

There are two examples of middlewares at the documentation, a function and a class middlewares. The function middleware should be `async` and the class middleware doesn't actually work.

Bottery ignores errors from Telegram API. If, for example, an invalid token is used, no message is shown and, the process breaks because it tries to continue even if something...

ref: https://dustingram.com/articles/2018/03/16/markdown-descriptions-on-pypi https://pypi.org/project/bottery/

The Messenger platform isn't running message handler for all messages that arrives in batch. At [platform/messenger/engine.py#L46](https://github.com/rougeth/bottery/blob/master/bottery/platform/messenger/engine.py#L46), we are getting only the first item of `entry`: ```python { "object":"page", "entry":[ {...

Session of the bot fixture must be closed after used. Probably here: https://github.com/rougeth/bottery/blob/master/tests/test_bottery.py#L22 ``` tests/test_bottery.py::test_default_properties[loop-AbstractEventLoop] bottery/tests/test_bottery.py:22: RuntimeWarning: coroutine 'ClientSession.close' was never awaited bot.session.close() tests/test_bottery.py::test_default_properties[session-ClientSession] bottery/tests/test_bottery.py:22: RuntimeWarning: coroutine 'ClientSession.close' was never...

Use pytest-deadfixtures on CI

1

The `pytest-deadfixtures` plugin search for duplicated and not used fixtures.

Multiplatform

8

Would be great to run the same bot for telegram, messenger, slack, etc.

Doc of the dreams

2

Check list to help to remember what should or must be at the documentation of Bottery. Suggestions are **really** welcome. - [ ] A "big" tutorial - [ ] How...

Increase coverage for bottery/conf/settings.py

2

Missing lines: 11-25, 32, 35-37 *See Travis log to confirm the missing lines:* https://travis-ci.org/rougeth/bottery