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

	Commit message (Collapse)	Author	Age	Lines
...
\| \| \| \| \| \| * \| \| \| \| \| \|	(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.
\| \| * \| \| \| \| \| \| \| \|	Fix conflict with disabling filter ping for DMs	MarkKoz	2020-08-02	-0/+173
\| \| \|\ \ \ \ \ \ \ \ \
\| \| * \| \| \| \| \| \| \| \| \|	Fix test for token remover log message	MarkKoz	2020-07-21	-2/+1
\| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \|	Fix user command tests	MarkKoz	2020-08-26	-13/+7
\| \| \|_\|_\|_\|_\|_\|/ / / / \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Mocking the bot commands channel constant no longer worked after switching to `in_whitelist_check`.
\| * \| \| \| \| \| \| \| \| \|	Update tests for user commands	Joseph Banks	2020-08-14	-32/+55
\| \| \|_\|/ / / / / / / \| \|/\| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Silence tests: make _async_init attribute tests more robust	MarkKoz	2020-10-08	-7/+11
\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Silence: remove _mod_log_channel attribute	MarkKoz	2020-10-08	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	It's only used as an argument to `SilenceNotifier`, so it doesn't need to be an instance attribute.
* \| \| \| \| \| \| \| \| \|	Silence: add to notifier when indefinite rather than temporary	MarkKoz	2020-10-08	-6/+6
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Accidentally swapped the logic in a previous commit during a refactor.
* \| \| \| \| \| \| \| \| \|	Silence tests: fix unawaited coro warnings	MarkKoz	2020-10-07	-2/+7
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Because the Scheduler is mocked, it doesn't actually do anything with the coroutines passed to the schedule() functions, hence the warnings.
* \| \| \| \| \| \| \| \| \|	Silence: rename _init_cog to _async_init	MarkKoz	2020-10-06	-13/+13
\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Silence: refactor _silence	MarkKoz	2020-10-06	-16/+22
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	* Rename to `_silence_overwrites` * Reduce responsibilities to only setting permission overwrites * Log in `silence` instead * Add to notifier in `silence` instead
* \| \| \| \| \| \| \| \| \|	Silence: cancel init task when cog unloads	MarkKoz	2020-08-18	-2/+5
\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Silence: rename caches	MarkKoz	2020-08-18	-20/+20
\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Silence tests: remove unnecessary spec_set args	MarkKoz	2020-08-18	-3/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	It's not really necessary to set to True when mocking functions.
* \| \| \| \| \| \| \| \| \|	Silence tests: more accurately assert the silence cmd schedule a task	MarkKoz	2020-08-18	-3/+7
\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Silence tests: add a test for rescheduling active silences	MarkKoz	2020-08-18	-0/+29
\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Silence tests: test the rescheduler unsilences expired silences	MarkKoz	2020-08-18	-0/+14
\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Silence tests: test the rescheduler adds permanent silence to notifier	MarkKoz	2020-08-18	-0/+14
\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Silence tests: test the scheduler skips missing channels	MarkKoz	2020-08-18	-0/+25
\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Silence tests: add a test for _init_cog rescheduling unsilences	MarkKoz	2020-08-18	-0/+7
\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Silence tests: remove _reschedule patch for cog tests	MarkKoz	2020-08-18	-4/+0
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	They don't do anything because they patch the class rather than the instance. It's too late for patching the instance to work since the `setUp` fixture, which instantiates the cog, executes before the patches do. Patching `setUp` would work (and its done in the other test cases), but some tests in this case will need the unpatched function too. Patching it doesn't serve much benefit to most tests anyway, so it's not worth the effort trying to make them work where they aren't needed.
* \| \| \| \| \| \| \| \| \|	Silence tests: access everything via the silence module	MarkKoz	2020-08-18	-24/+23
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The module is imported anyway to keep imports short and clean. Using it in patch targets is shorter and allows for the two imports from the module to be removed.
* \| \| \| \| \| \| \| \| \|	Silence tests: add a subtest for the manual unsilence message	MarkKoz	2020-08-17	-4/+7
\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Silence tests: assert against message constants	MarkKoz	2020-08-17	-6/+7
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Duplicating strings in assertions is redundant, closely coupled, and less maintainable.
* \| \| \| \| \| \| \| \| \|	Silence tests: also test that cache misses preserve other overwrites	MarkKoz	2020-08-17	-12/+16
\| \| \| \| \| \| \| \| \| \|