bot - A mirrored copy of the python-discord/bot repository.

	Commit message (Collapse)	Author	Age	Lines
...
\| \| \| \| \| * \| \| \| \| \| \|	Merge branch 'master' into redis_persistence	Leon Sandøy	2020-05-23	-4/+7
\| \| \| \| \| \|\\| \| \| \| \| \|
\| \| \| \| \| * \| \| \| \| \| \|	Complete asyncified test suite for RedisCache	Leon Sandøy	2020-05-23	-94/+112
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This commit just alters existing code to work with the new interface, and with async. All tests are passing successfully.
\| \| \| \| \| * \| \| \| \| \| \|	Finish .set and .get, and add tests.	Leon Sandøy	2020-05-23	-12/+26
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The .set and .get will accept ints, floats, and strings. These will be converted into "typestrings", which is basically just a simple format that's been invented for this object. For example, an int looks like `b"i\|2423"`. Note how it is still stored as a bytestring (like everything in Redis), but because of this prefix we are able to coerce it into the type we want on the way out of the db.
\| \| \| \| \| * \| \| \| \| \| \|	Use autospecced mocks in MockBot for the stats and aiohttp session	MarkKoz	2020-05-22	-0/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This will help catch anything that tries to get/set an attribute/method which doesn't exist. It'll also catch missing/too many parameters being passed to methods.
\| \| \| \| \| * \| \| \| \| \| \|	Properly mock the redis pool in MockBot	MarkKoz	2020-05-22	-9/+19
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Because some of the redis pool/connection methods return futures rather than being coroutines, the redis pool had to be mocked using the CustomMockMixin so it could take advantage of `additional_spec_asyncs` to use AsyncMocks for these future-returning methods.
\| \| \| \| \| * \| \| \| \| \| \|	Fix unawaited coro warning when instantiating Bot for MockBot's spec	MarkKoz	2020-05-22	-10/+14
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The fix is to mock the loop and pass it to the Bot. It will then set it as `self.loop` rather than trying to get an event loop from asyncio. The `create_task` patch has been moved to this loop mock rather than being done in MockBot to ensure that it applies to anything calling it when instantiating the Bot.
\| \| \| \| \| * \| \| \| \| \| \|	Improve aiohttp context manager mocking in snekbox tests	MarkKoz	2020-05-22	-3/+12
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	I'm not sure how it even managed to work before. It was calling the `post` coroutine (without specifying a URL) and then changing `__aenter__`. Now, a separate mock is created for the context manager and the `post` simply returns that mocked context manager.
\| \| \| \| \| * \| \| \| \| \| \|	Fix assertion for `create_task` in duck pond tests	MarkKoz	2020-05-22	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The assertion wasn't using the assertion method. Furthermore, it was testing a non-existent function `create_loop` rather than `create_task`.
\| \| \| \| \| * \| \| \| \| \| \|	Set up async testbed	Leon Sandøy	2020-05-22	-189/+128
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| * \| \| \| \| \| \|	Test suite for the redis dict.	Leon Sandøy	2020-05-17	-0/+189
\| \| \| \| \| \| \|_\|/ / / / \| \| \| \| \| \|/\| \| \| \| \|
\| \| \| \| * \| \| \| \| \| \|	AntiMalware tests - Switched from monkeypatch to unittest.patch	Jannes Jonkers	2020-05-25	-2/+2
\| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| * \| \| \| \| \| \|	AntiMalware Refactor - Moved embed descriptions into constants, added tests ↵	Jannes Jonkers	2020-05-11	-18/+30
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	for embed descriptions
\| \| \| \| * \| \| \| \| \| \|	AntiMalware Tests - Removed exact log content checks	Jannes Jonkers	2020-05-11	-19/+2
\| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| * \| \| \| \| \| \|	AntiMalware Tests - added a missing case for no extensions in ↵	Jannes Jonkers	2020-05-11	-0/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	test_get_disallowed_extensions
\| \| \| \| * \| \| \| \| \| \|	AntiMalware Tests - extracted the method for determining disallowed ↵	Jannes Jonkers	2020-05-11	-6/+23
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	extensions and added a test for it.
\| \| \| \| * \| \| \| \| \| \|	AntiMalware Tests - implemented minor feedback	Jannes Jonkers	2020-05-11	-12/+12
\| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| * \| \| \| \| \| \|	Update tests/bot/cogs/test_antimalware.py	MrGrote	2020-05-08	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-authored-by: Mark <[email protected]>
\| \| \| \| * \| \| \| \| \| \|	AntiMalware Tests - Switched to unittest.IsolatedAsyncioTestCase	Jannes Jonkers	2020-05-07	-29/+19
\| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| * \| \| \| \| \| \|	AntiMalware Tests - Added unittest for txt file attachment	Jannes Jonkers	2020-05-07	-2/+23
\| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| * \| \| \| \| \| \|	AntiMalware Tests - Added unittest for valid attachment	Jannes Jonkers	2020-05-07	-0/+9
\| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| * \| \| \| \| \| \|	AntiMalware Tests - Added unittest message deletion log	Jannes Jonkers	2020-05-07	-4/+28
\| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| * \| \| \| \| \| \|	AntiMalware Tests - Added unittest for cog setup	Jannes Jonkers	2020-05-07	-0/+10
\| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| * \| \| \| \| \| \|	AntiMalware Tests - Added unittest for messages that were deleted in the ↵	Jannes Jonkers	2020-05-07	-1/+19
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	meantime.
\| \| \| \| * \| \| \| \| \| \|	AntiMalware Tests - Added unittest for the embed for a python file.	Jannes Jonkers	2020-05-07	-1/+24
\| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| * \| \| \| \| \| \|	AntiMalware Tests - Added unittest for messages send by staff	Jannes Jonkers	2020-05-07	-1/+12
\| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| * \| \| \| \| \| \|	AntiMalware Tests - Added unittests for deletion of message and ignoring of dms	Jannes Jonkers	2020-05-07	-1/+21
\| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| * \| \| \| \| \| \|	AntiMalware Tests - Added unittest for message without attachment	Jannes Jonkers	2020-05-07	-0/+20
\| \| \| \| \|/ / / / / /
\| \| \| * \| \| \| \| \| \|	Infr. Tests: Add `apply_infraction` awaiting assertion with args	ks129	2020-05-30	-0/+6
\| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \|	Simplify infraction reason truncation tests	ks129	2020-05-30	-13/+7
\| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \|	Infr Tests: Make `get_active_infraction` return `None`	ks129	2020-05-20	-1/+1
\| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \|	Infr. Test: Replace `get_active_mock` return value	ks129	2020-05-20	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Replace `{"foo": "bar"}` with `{"id": 1}`
\| \| \| * \| \| \| \| \| \|	Infr. Tests: Replace `str` with `dict`	ks129	2020-05-20	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	To allow `.get`, I had to replace `str` return value with `dict`
\| \| \| * \| \| \| \| \| \|	Infr. Tests: Add more content to await comment	ks129	2020-05-20	-2/+2
\| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \|	ModLog Tests: Fix embed description truncate test	ks129	2020-05-20	-1/+1
\| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \|	Merge branch 'master' into ban-kick-reason-length	ks129	2020-05-20	-11/+183
\| \| \| \|\ \ \ \ \ \ \ \| \| \| \| \| \|/ / / / / \| \| \| \| \|/\| \| \| \| \|
\| \| \| * \| \| \| \| \| \|	Infraction Tests: Small fixes	ks129	2020-05-20	-4/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	- Remove unnecessary space from placeholder - Rename `has_active_infraction` to `get_active_infraction`
\| \| \| * \| \| \| \| \| \|	ModLog Tests: Fix truncation tests docstring	ks129	2020-05-20	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-authored-by: Leon Sandøy <[email protected]>
\| \| \| * \| \| \| \| \| \|	(Infractions and ModLog Tests): Replaced `shortening` with `truncation`, ↵	ks129	2020-04-16	-5/+6
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	removed unnecessary type hint and added comment to kick truncation test about awaiting `kick`.
\| \| \| * \| \| \| \| \| \|	(ModLog Tests): Created reason shortening tests for `send_log_message`.	ks129	2020-04-16	-0/+29
\| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \|	(Infraction Tests): Created reason shortening tests for ban and kick.	ks129	2020-04-16	-0/+54
\| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \|	(Test Helpers): Added `__ge__` function to `MockRole` for comparing.	ks129	2020-04-15	-0/+4
\| \| \| \| \|_\|_\|/ / / \| \| \| \|/\| \| \| \| \|
\| \| * \| \| \| \| \| \|	Fix failing tests related to avatar_hash	Leon Sandøy	2020-05-27	-5/+0
\| \| \| \|/ / / / / \| \| \|/\| \| \| \| \|
\| \| * \| \| \| \| \|	Merge branch 'master' into eval-timeout-increase	Dennis Pham	2020-05-19	-3/+2
\| \| \|\ \ \ \ \ \ \| \| \| \| \|_\|_\|/ / \| \| \| \|/\| \| \| \|
\| \| * \| \| \| \| \|	Change tests to use the new timeout constant	Numerlor	2020-05-15	-1/+5
\| \| \| \|_\|/ / / \| \| \|/\| \| \| \|
\| * / \| \| \| \|	Improve LinePaginator to support long lines	Kyle Stanley	2020-06-04	-7/+34
\| \|/ / / / /
* \| \| \| \| \|	EH Tests: Create test for `handle_unexpected_error`	ks129	2020-05-19	-2/+37
\| \| \| \| \| \|
* \| \| \| \| \|	EH Tests: Create test for `ErrorHandler` `setup` function	ks129	2020-05-19	-1/+11
\| \| \| \| \| \|
* \| \| \| \| \|	EH Tests: Create tests for `handle_api_error`	ks129	2020-05-19	-0/+33
\| \| \| \| \| \|
* \| \| \| \| \|	EH Tests: Merge test classes	ks129	2020-05-19	-11/+4
\| \| \| \| \| \|
* \| \| \| \| \|	EH Tests: Create test for `handle_check_failure`	ks129	2020-05-19	-0/+48
\| \| \| \| \| \|