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

	Commit message (Collapse)	Author	Age	Lines
...
\| \| \| \| \| * \| \| \| \| \| \|	(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]>
\| \| * \| \| \| \| \| \| \|	Use async-rediscache package for our redis caches	Sebastiaan Zeeff	2020-09-19	-1/+5
\| \| \|/ / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	I've migrated our redis caches over to the async-rediscache package that we've recently released (https://git.pydis.com/async-rediscache). The main functionality remains the same, although the package handles some things, like getting the active session, differently internally. The main changes you'll note for our bot are: - We create a RedisSession instance and ensure that it connects before we even create a Bot instance in `__main__.py`. - We are now actually using a connection pool instead of a single connection. - Our Bot subclass now has a new required kwarg: `redis_session`. - Bool values are now properly converted to and from typestrings. In addition, I've made sure that our MockBot passes a MagicMock for the new `redis_session` kwarg when creating a Bot instance for the spec_set. Signed-off-by: Sebastiaan Zeeff <[email protected]>
\| \| * \| \| \| \| \| \|	Merge branch 'master' into truncate-internal-eval	Kieran Siek	2020-09-16	-122/+1305
\| \| \|\\| \| \| \| \| \|
\| \| \| * \| \| \| \| \|	Merge branch 'master' into remove-wolfram	Leon Sandøy	2020-09-02	-0/+22
\| \| \| \|\ \ \ \ \ \
\| \| \| \| * \| \| \| \| \|	Fixed old tests and added 2 new ones	wookie184	2020-09-01	-0/+22
\| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \|	Removed ImagePaginator testing.	Xithrius	2020-08-31	-15/+0
\| \| \| \|/ / / / / /
\| \| \| * \| \| \| \| \|	Merge branch 'master' into feat/backend/707/alias-decorator	Shirayuki Nekomata	2020-08-26	-32/+55
\| \| \| \|\ \ \ \ \ \
\| \| \| * \| \| \| \| \| \|	Include root aliases in the command name conflict test	MarkKoz	2020-08-22	-0/+1
\| \| \| \| \|_\|_\|_\|/ / \| \| \| \|/\| \| \| \| \|
\| \| * \| \| \| \| \| \|	Change tests to work with the new file layout.	Numerlor	2020-07-15	-3/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	326beebe9b097731a39ecc9868e5e1f2bd762aae
\| \| * \| \| \| \| \| \|	Merge branch 'master' into truncate-internal-eval	Numerlor	2020-06-21	-179/+662
\| \| \|\ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	# Conflicts: # bot/utils/__init__.py
\| \| * \ \ \ \ \ \ \	Merge branch 'master' into truncate-internal-eval	Numerlor	2020-05-30	-33/+390
\| \| \|\ \ \ \ \ \ \ \
\| \| * \| \| \| \| \| \| \| \|	Add tests for `send_to_paste_service`.	Numerlor	2020-05-08	-0/+74
\| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \|	Adjust behaviour for new func usage.	Numerlor	2020-05-07	-14/+5
\| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \|	Remove tests from moved function.	Numerlor	2020-05-07	-15/+0
\| \| \| \|_\|_\|_\|_\|/ / / \| \| \|/\| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \|	Resolve conflicts in filtering.py	MarkKoz	2020-08-10	-9/+15
\| \|\ \ \ \ \ \ \ \ \ \| \| \| \|_\|_\|/ / / / / \| \| \|/\| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \|	Use format_user in token remover test	MarkKoz	2020-08-10	-1/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The point of format_user is to have a consistent format across the code base. That should apply to tests too.