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

	Commit message (Collapse)	Author	Age	Lines
...
\| \| \| \| \| \| \| * \| \| \| \| \| \| \|	Merge branch 'tags_overhaul' of https://github.com/RohanJnr/bot into ↵	RohanJnr	2020-02-29	-895/+2636
\| \| \| \| \| \| \| \|\ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	tags_overhaul
\| \| \| \| \| \| \| \| * \ \ \ \ \ \ \	Merge branch 'master' into tags_overhaul	Rohan	2020-02-29	-895/+2636
\| \| \| \| \| \| \| \| \|\ \ \ \ \ \ \ \
\| \| \| \| \| \| \| * \| \| \| \| \| \| \| \| \|	Caching all the tags when the bot has loaded(caching only once) insted of ↵	RohanJnr	2020-02-29	-21/+18
\| \| \| \| \| \| \| \|/ / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	caching it after the tags command is used.
\| \| \| \| \| \| \| * \| \| \| \| \| \| \| \|	Re-corrected the lines which I had changed by mistake	RohanJnr	2020-02-29	-1/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| * \| \| \| \| \| \| \| \|	added white spaces on statements before bullet points for proper rendering ↵	RohanJnr	2020-02-29	-75/+75
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	of points on github
\| \| \| \| \| \| \| * \| \| \| \| \| \| \| \|	Added all the tag files in resources and modified cogs/tags.py file to ↵	RohanJnr	2020-02-29	-84/+919
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	access the static tag files rather than sending an API get request. Removed all methods calling the API so the tags cannot be edited, added nor deleted.
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Add tests for `HushDurationConverter`.	Numerlor	2020-03-17	-0/+30
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Add tests ensuring permissions get preserved.	Numerlor	2020-03-17	-0/+31
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Use set `discard` instead of `remove`.	Numerlor	2020-03-17	-3/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Discard ignores non present values, allowing us to skip the KeyError suppress.
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Add docstring to test.	Numerlor	2020-03-17	-0/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Fix test name.	Numerlor	2020-03-17	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	`removed` was describing the opposite behaviour.
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Remove `alert_channel` mention from docstring.	Numerlor	2020-03-17	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	After removing the optional channel arg and changing output message channels we're only testing `ctx`'s `send`.
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Remove one indentation level.	Numerlor	2020-03-17	-3/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-authored-by: MarkKoz <[email protected]>
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Reword `unsilence` docstring.	Numerlor	2020-03-17	-2/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-authored-by: MarkKoz <[email protected]>
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Remove `channel` mentions from command docstrings.	Numerlor	2020-03-17	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	With the new behaviour of not accepting channels and muting the current one, it's no longer neccessary to keep the channel param in the docstring. Co-authored-by: MarkKoz <[email protected]>
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Rename `cog_unload` tests.	Numerlor	2020-03-17	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Previous names were undescriptive from testing phases.
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Fix `test_cog_unload` passing tests with invalid values.	Numerlor	2020-03-17	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The first assert - `asyncio_mock.create_task.assert_called_once_with` called `alert_channel`'s send resulting in an extra call. `send` on `alert_channel` was not tested properly because of a typo and a missing assert in the method call.
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Handle and log `CommandErrors` on `.can_run`.	Numerlor	2020-03-16	-1/+6
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Remove long indentation from docstrings.	Numerlor	2020-03-15	-6/+6
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Set `_get_instance_vars_event` in test's `setUp`.	Numerlor	2020-03-15	-0/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Adjust tests for new calling behaviour.	Numerlor	2020-03-15	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	`.set_permissions` calls were changed to use kwargs directly instead of an overwrite, this reflects the changes in tests.
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Document returns values of private methods.	Numerlor	2020-03-15	-0/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Block commands until all instance vars are loaded.	Numerlor	2020-03-15	-0/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Assign created task to a var.	Numerlor	2020-03-15	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Pass kwargs directly instead of a PermissionOverwrite.	Numerlor	2020-03-15	-9/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The `set_permissions` method creates a `PermissionOverwrite` from kwargs internally, so we can skip creating it ourselves and unpack the dict directly into kwargs.
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Change BadArgument error wording.	Numerlor	2020-03-15	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-authored-by: MarkKoz <[email protected]>
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Add Silence cog load to docstring.	Numerlor	2020-03-15	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Move coro execution outside of if condition.	Numerlor	2020-03-15	-8/+6
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This gives us a clearer look at the general flow control and what's getting executed. Comment was also moved to its relevant line. Co-authored-by: MarkKoz <[email protected]>
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Adjust docstring styling.	Numerlor	2020-03-15	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-authored-by: MarkKoz <[email protected]>
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Use dict instead of a set and custom class.	Numerlor	2020-03-15	-44/+8
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The FirstHash class is no longer necessary with only channels and the current loop in tuples. FirstHash was removed, along with its tests and tests were adjusted for new dict behaviour.
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Add tests for `_notifier`.	Numerlor	2020-03-11	-0/+19
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Improve commands help.	Numerlor	2020-03-11	-3/+5
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Add logging to commands.	Numerlor	2020-03-11	-0/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Change various logging levels.	Numerlor	2020-03-11	-7/+7
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Add tests for `remove_channel`.	Numerlor	2020-03-11	-0/+18
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Remove `_current_loop` from setup.	Numerlor	2020-03-11	-2/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Add tests for `add_channel`.	Numerlor	2020-03-11	-0/+21
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Create test case for `SilenceNotifier`	Numerlor	2020-03-11	-1/+7
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Add tests for `_get_instance_vars`.	Numerlor	2020-03-11	-1/+28
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Pass created channel instead of new object.	Numerlor	2020-03-11	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Creating a new object caused the assert to fail because different objects were used.
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Reset `self.ctx` call history after every subtest.	Numerlor	2020-03-11	-0/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Fix erroneous `assert_called_once_with` calls.	Numerlor	2020-03-11	-8/+7
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	`assert_called_once_with` was being tested on call_args which always reported success.st.
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Add test for `cog_check`.	Numerlor	2020-03-11	-0/+7
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Add test for `cog_unload`.	Numerlor	2020-03-11	-1/+17
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Use patch decorators and assign names from `with` patches.	Numerlor	2020-03-11	-11/+11
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Separate tests for permissions and `muted_channels.add` on `_silence`.	Numerlor	2020-03-11	-5/+8
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Add tests for `_unsilence`	Numerlor	2020-03-11	-1/+33
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Use async test case.	Numerlor	2020-03-11	-14/+12
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This allows us to use coroutines with await directly instead of asyncio.run
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Add test for `_silence` method.	Numerlor	2020-03-11	-1/+34
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| \| \| \| * \| \|	Move adding of channel to `muted_channels` up.	Numerlor	2020-03-11	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Before the channel was not added if `persistent` was `True`.