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

	Commit message (Collapse)	Author	Lines
2020-03-17	Reword `unsilence` docstring.	Numerlor	-2/+1
	Co-authored-by: MarkKoz <[email protected]>
2020-03-17	Remove `channel` mentions from command docstrings.	Numerlor	-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]>
2020-03-17	Rename `cog_unload` tests.	Numerlor	-2/+2
	Previous names were undescriptive from testing phases.
2020-03-17	Fix `test_cog_unload` passing tests with invalid values.	Numerlor	-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.
2020-03-16	Handle and log `CommandErrors` on `.can_run`.	Numerlor	-1/+6

2020-03-15	Remove long indentation from docstrings.	Numerlor	-6/+6

2020-03-15	Set `_get_instance_vars_event` in test's `setUp`.	Numerlor	-0/+2

2020-03-15	Adjust tests for new calling behaviour.	Numerlor	-2/+2
	`.set_permissions` calls were changed to use kwargs directly instead of an overwrite, this reflects the changes in tests.
2020-03-15	Document returns values of private methods.	Numerlor	-0/+2

2020-03-15	Block commands until all instance vars are loaded.	Numerlor	-0/+4

2020-03-15	Assign created task to a var.	Numerlor	-1/+1

2020-03-15	Pass kwargs directly instead of a PermissionOverwrite.	Numerlor	-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.
2020-03-15	Change BadArgument error wording.	Numerlor	-1/+1
	Co-authored-by: MarkKoz <[email protected]>
2020-03-15	Add Silence cog load to docstring.	Numerlor	-1/+1

2020-03-15	Move coro execution outside of if condition.	Numerlor	-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]>
2020-03-15	Adjust docstring styling.	Numerlor	-2/+2
	Co-authored-by: MarkKoz <[email protected]>
2020-03-15	Use dict instead of a set and custom class.	Numerlor	-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.
2020-03-11	Add tests for `_notifier`.	Numerlor	-0/+19

2020-03-11	Improve commands help.	Numerlor	-3/+5

2020-03-11	Add logging to commands.	Numerlor	-0/+3

2020-03-11	Change various logging levels.	Numerlor	-7/+7

2020-03-11	Add tests for `remove_channel`.	Numerlor	-0/+18

2020-03-11	Remove `_current_loop` from setup.	Numerlor	-2/+1

2020-03-11	Add tests for `add_channel`.	Numerlor	-0/+21

2020-03-11	Create test case for `SilenceNotifier`	Numerlor	-1/+7

2020-03-11	Add tests for `_get_instance_vars`.	Numerlor	-1/+28

2020-03-11	Pass created channel instead of new object.	Numerlor	-1/+1
	Creating a new object caused the assert to fail because different objects were used.
2020-03-11	Reset `self.ctx` call history after every subtest.	Numerlor	-0/+1

2020-03-11	Fix erroneous `assert_called_once_with` calls.	Numerlor	-8/+7
	`assert_called_once_with` was being tested on call_args which always reported success.st.
2020-03-11	Add test for `cog_check`.	Numerlor	-0/+7

2020-03-11	Add test for `cog_unload`.	Numerlor	-1/+17

2020-03-11	Use patch decorators and assign names from `with` patches.	Numerlor	-11/+11

2020-03-11	Separate tests for permissions and `muted_channels.add` on `_silence`.	Numerlor	-5/+8

2020-03-11	Add tests for `_unsilence`	Numerlor	-1/+33

2020-03-11	Use async test case.	Numerlor	-14/+12
	This allows us to use coroutines with await directly instead of asyncio.run
2020-03-11	Add test for `_silence` method.	Numerlor	-1/+34

2020-03-11	Move adding of channel to `muted_channels` up.	Numerlor	-1/+1
	Before the channel was not added if `persistent` was `True`.
2020-03-11	Remove channel mentions from output discord messages.	Numerlor	-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
2020-03-11	Remove unnecessary args from test cases.	Numerlor	-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.
2020-03-10	Add alert with silenced channels on `cog_unload`.	Numerlor	-0/+11

2020-03-10	Remove `channel` arg from commands.	Numerlor	-20/+10

2020-03-10	Retain previous channel overwrites.	Numerlor	-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.
2020-03-09	Use `patch.object` instead of patch with direct `return_value`.	Numerlor	-9/+2

2020-03-08	Move notifier to separate class.	Numerlor	-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.
2020-03-08	Remove "Channel" from output string for consistency.	Numerlor	-1/+1

2020-03-08	Use `Context.invoke` instead of calling `unsilence` directly.	Numerlor	-1/+1
	Calling the command coro directly did unnecessary checks and made tests for the method harder to realize.
2020-03-08	Add test for `unsilence` discord output.	Numerlor	-0/+9

2020-03-08	Add test for `silence` discord output.	Numerlor	-0/+24

2020-03-08	Add test case for Silence cog.	Numerlor	-1/+10

2020-03-08	Add tests for FirstHash class.	Numerlor	-0/+25