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

	Commit message (Collapse)	Author	Age	Lines
...
\| * \| \| \| \| \| \| \|	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`.
\| * \| \| \| \| \| \| \|	Remove channel mentions from output discord messages.	Numerlor	2020-03-11	-8/+8
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	With the removal of the channel args, it's no longer necessary to mention the channel in the command output. Tests adjusted accordingly
\| * \| \| \| \| \| \| \|	Remove unnecessary args from test cases.	Numerlor	2020-03-11	-6/+6
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Needless call args which were constant were kept in the test cases, resulting in redundant code, the args were moved directly into the function call.
\| * \| \| \| \| \| \| \|	Add alert with silenced channels on `cog_unload`.	Numerlor	2020-03-10	-0/+11
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Remove `channel` arg from commands.	Numerlor	2020-03-10	-20/+10
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Retain previous channel overwrites.	Numerlor	2020-03-10	-4/+12
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Previously silencing a channel reset all overwrites excluding `send_messages` and unsilencing them removed all overwrites. This is prevented by getting the current overwrite and applying it with only send_messages changed.
\| * \| \| \| \| \| \| \|	Use `patch.object` instead of patch with direct `return_value`.	Numerlor	2020-03-09	-9/+2
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Move notifier to separate class.	Numerlor	2020-03-08	-40/+46
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Separating the notifier allows us to keep the Silence class and its methods to be more focused on the class' purpose, handling the logic of adding/removing channels and the loop itself behind `SilenceNotifier`'s interface.
\| * \| \| \| \| \| \| \|	Remove "Channel" from output string for consistency.	Numerlor	2020-03-08	-1/+1
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Use `Context.invoke` instead of calling `unsilence` directly.	Numerlor	2020-03-08	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Calling the command coro directly did unnecessary checks and made tests for the method harder to realize.
\| * \| \| \| \| \| \| \|	Add test for `unsilence` discord output.	Numerlor	2020-03-08	-0/+9
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Add test for `silence` discord output.	Numerlor	2020-03-08	-0/+24
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Add test case for Silence cog.	Numerlor	2020-03-08	-1/+10
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Add tests for FirstHash class.	Numerlor	2020-03-08	-0/+25
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Add handling for shh/unshh for `CommandNotFound`.	Numerlor	2020-03-08	-4/+32
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Restrict cog to moderators.	Numerlor	2020-03-08	-1/+7
\| \| \| \| \| \| \| \| \|