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

	Commit message (Collapse)	Author	Age	Lines
...
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Added 2 call check to `post_infraction` test.	Karlis S	2020-03-13	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| * \| \| \| \| \| \|	Revise docstrings for moderation util tests	MarkKoz	2020-03-12	-10/+16
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Removed `once` from `post_infraction` test due ↵	Karlis S	2020-03-12	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	tests failing.
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Added mock reset due fail.	ks123	2020-03-12	-0/+2
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Added call check for `ctx.send` in `post_user` test.	ks123	2020-03-12	-0/+1
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Move errors from booleans to actual errors in ↵	ks123	2020-03-12	-7/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	`post_user` test.
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Hard-coded `self.ctx` argument to `post_user` ↵	ks123	2020-03-12	-5/+5
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	test, renamed current `args` to `user`, applied this in code.
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Fixed formatting in `notify_infraction` test.	ks123	2020-03-12	-3/+1
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Removed unnecessary symbols from ↵	ks123	2020-03-12	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	`has_active_infraction` test `infraction_nr` variable and changes this to more unique number.
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Moved embed generating to test cases loop from ↵	ks123	2020-03-12	-10/+10
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	test cases listing, added icon to test cases in `notify_pardon` test
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Added params to variable in ↵	ks123	2020-03-12	-5/+7
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	`has_active_infraction` test.
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Create extra new tests set for `post_infraction` ↵	ks123	2020-03-12	-103/+60
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	testing, removed old.
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Moved mock resetting to beginning of subtest in ↵	ks123	2020-03-08	-4/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	`post_user` and `send_private_embed` test.
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Removed unnecessary `if` check from ↵	ks123	2020-03-08	-2/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	`send_private_embed` test
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Hard-coded args for `send_private_embed` test.	ks123	2020-03-08	-10/+6
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Added new check to `post_user` test (`ctx.send` ↵	ks123	2020-03-08	-35/+16
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	content test), improved test cases.
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Moved `expected_output` from `Dict` to ↵	ks123	2020-03-08	-15/+9
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	`discord.Embed` in `notify_pardon` test.
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Removed unnecessary `ctx.send` `side_effect` from ↵	ks123	2020-03-08	-11/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	`notify_pardon`, applied changes to test cases.
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Moved `notify_infraction` embed check from dict to ↵	ks123	2020-03-08	-28/+43
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	`Embed`.
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Removed unnecessary `ctx.send` mock resetting, ↵	ks123	2020-03-08	-4/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	moved `send_private_embed` mock reset to beginning of subtest.
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Removed unnecessary `ctx.send` `side_effect` and ↵	ks123	2020-03-08	-12/+6
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	removed these in test cases too in `notify_infraction` test.
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Removed `Dict` unpacking in `notify_infraction` test.	ks123	2020-03-08	-20/+20
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Cleaned up `has_active_infraction` test cases, ↵	ks123	2020-03-08	-30/+11
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	hard-coded args, moved mocks resetting to beginning of subtest, added `ctx.send` check only is infraction nr and type in sent string.
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Hard-coded API get request params for ↵	ks123	2020-03-08	-13/+6
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	`has_active_infraction` test.
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Removed large `utils` parts import, use import ↵	ks123	2020-03-08	-12/+10
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	`utils` instead and added `utils` before variables and function that was imported directly before.
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Fixed docstrings, added more information to these.	ks123	2020-03-08	-6/+8
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Fixed errors, added checks before assertions for ↵	ks123	2020-03-05	-1/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	errors
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Added additional assertions to `post_infraction` test.	ks123	2020-03-05	-8/+54
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Added `user.send` call assertion to ↵	ks123	2020-03-05	-0/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	`test_send_private_embed`.
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Added API POST call assertion to `test_post_user`.	ks123	2020-03-05	-5/+24
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Added new assertions to `notify_pardon`, added ↵	ks123	2020-03-05	-6/+24
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	`ctx.send` raising errors, added check for return values and `send_private_embed` call.
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Added new assertions to `post_infraction`, added ↵	ks123	2020-03-05	-11/+33
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	`ctx.send` raising errors, added check for return values and `send_private_embed` call.
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Added following new assertions to ↵	ks123	2020-03-05	-3/+27
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	`has_active_infraction` tests: `ctx.send` and `bot.api_client.get` calling.
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Use `bot.cogs.moderation.utils`'s `RULES_URL` ↵	ks123	2020-03-05	-2/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	instead creating new one
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Removed unnecessary `AsyncMock()` from `__init__` ↵	ks123	2020-03-05	-1/+0
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	(`self.bot.api_client.get`)
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Moved `has_active_infraction` tests to one test.	ks123	2020-03-05	-22/+35
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): `send_private_embed` moved exception creating from ↵	ks123	2020-03-05	-4/+6
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	cases testing to test cases listing, added side_effect resetting.
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Created tests for `post_infraction` function, ↵	ks123	2020-03-05	-1/+68
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	created __init__.py for moderation tests
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Created tests for `send_private_embed` function + ↵	ks123	2020-03-04	-3/+44
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fixed errors.
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Created tests for `post_user` function.	ks123	2020-03-04	-1/+59
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Created tests for `notify_pardon` function.	ks123	2020-03-04	-2/+39
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Created tests for `notify_infraction` function.	ks123	2020-03-04	-2/+91
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Created tests for `has_active_infraction` function	ks123	2020-03-03	-0/+24
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| * \| \| \| \| \| \|	(Moderation Utils Tests): Added imports, modified tests class instance and ↵	ks123	2020-03-03	-5/+7
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	created new params for tests class
\| \| \| \| \| \| * \| \| \| \| \| \|	Created file for moderation utils tests + added setUp to this.	ks123	2020-03-03	-0/+12
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| * \| \| \| \| \| \| \|	Fix cog_check tests	MarkKoz	2020-09-08	-8/+12
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| * \| \| \| \| \| \| \|	Fix tests for has_any_role_check	MarkKoz	2020-09-08	-16/+18
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| * \| \| \| \| \| \| \|	Rename role checks and decorators	MarkKoz	2020-09-08	-18/+18
\| \| \| \| \| \| \|_\|_\|_\|_\|/ / \| \| \| \| \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Make their names more in line with `has_any_role` for consistency.
\| \| \| \| * \| \| \| \| \| \| \|	Remove unit tests for duck pond	Sebastiaan Zeeff	2020-09-20	-548/+0
\| \| \| \| \| \|_\|_\|_\|_\|/ / \| \| \| \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	I've removed the unit tests for duckpond in concordance with the new policy for writing unit tests for the bot The tests were unnecessarily complicated, difficult to maintain, and slowed development. Signed-off-by: Sebastiaan Zeeff <[email protected]>
\| \| \| * \| \| \| \| \| \| \|	Remove vestigial RedisCache class definition	Sebastiaan Zeeff	2020-09-19	-265/+0
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	As we're now using the `async-rediscache` package, it's no longer necessary to keep the `RedisCache` defined in `bot.utils.redis_cache` around. I've removed the file containing it and the tests written for it. Signed-off-by: Sebastiaan Zeeff <[email protected]>