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

	Commit message (Collapse)	Author	Age	Lines
...
\| \| \| \| * \| \| \| \| \| \|	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
\| \|\ \ \ \ \ \ \| \| \| \|_\|_\|/ / \| \| \|/\| \| \| \|
\| \| * \| \| \| \|	Use `Command`-object for `send_help`	Sebastiaan Zeeff	2020-05-17	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	As @mathsman5133 pointed out, it's better to use the `Command`-instance we typically already have in the current context than to rely on parsing the qualified name again. The invocation is now done as: `await ctx.send_help(ctx.command)`
\| \| * \| \| \| \|	Use `send_help` to invoke command help	Sebastiaan Zeeff	2020-05-17	-2/+1
\| \| \| \|/ / / \| \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	After the refactoring of the help command, we need to use the built-in method of calling the help command: `Context.send_help`. As an argument, the qualified name (a string containing the full command path, including parents) of the command can be passed. Examples: - await ctx.send_help("reminders edit") This would send a help embed with information on `!reminders edit` to the Context. - await ctx.send_help(ctx.command.qualified_name) This would extract the qualified name of the command, which is the full command path, and send a help embed to Context. - await ctx.send_help() This will send the main "root" help embed to the Context.
\| * / \| \| \|	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
\|/ / / /
* \| \| \|	Tests: change avatar_url_as assertion to use static_format	MarkKoz	2020-04-30	-1/+1
\| \| \| \|
* \| \| \|	Add DMChannel tests for in_whitelist decorator	Sebastiaan Zeeff	2020-04-27	-31/+63
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The `in_whitelist` decorator should not fail when a decorated command was called in a DMChannel; it should simply conclude that the user is not allowed to use the command. I've added a test case that uses a DMChannel context with User, not Member, objects. In addition, I've opted to display a test case description in the `subTest`: Simply printing the actual arguments and context is messy and does not actually show you the information you'd like. This description is enough to figure out which test is failing and what the gist of the test is.
* \| \| \|	Add mock for discord.DMChannels	Sebastiaan Zeeff	2020-04-27	-0/+21
\| \| \| \|
* \| \| \|	Exclude never-run lines from coverage	Sebastiaan Zeeff	2020-04-27	-2/+2
\| \| \| \|
* \| \| \|	Simplify `in_whitelisted_context` decorator API	Sebastiaan Zeeff	2020-04-23	-32/+32
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The API of the `in_whitelisted_context` decorator was a bit clunky: - The long parameter names frequently required multiline decorators - Despite `#bot-commands` being the defacto default, it needed to be passed - The name of the function, `in_whitelisted_context` is fairly long in itself To shorten the call length of the decorator, the parameter names were shortened by dropping the `whitelisted_` prefix. This means that the parameter names are now just `channels`, `categories`, and `roles`. This already means that all current usages of the decorator are reduced to one line. In addition, `#bot-commands` has now been made the default redirect channel for the decorator. This means that if no `redirect` was passed, users will be redirected to `bot-commands` to use the command. If needed, `None` (or any falsey value) can be passed to disable redirection. Passing another channel id will trigger that channel to be used as the redirection target instead of bot-commands. Finally, the name of the decorator was shortened to `in_whitelist`, which already communicates what it is supposed to do.
* \| \| \|	Add tests for `in_whitelisted_context` decorator	Sebastiaan Zeeff	2020-04-23	-0/+115
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	I have added tests for the new `in_whitelisted_context` decorator. They work by calling the decorator with different kwargs to generate a specific predicate callable. That callable is then called to assess if it comes to the right conclusion.