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

	Commit message (Collapse)	Author	Age	Lines
*	Implement description shortening to infraction notify tests	ks129	2020-06-12	-8/+27
\|
*	Remove deprecated avatar hash in `test_post_user`	ks129	2020-06-12	-3/+1
\|
*	Merge branch 'master' into mod-utils-tests	ks129	2020-06-11	-95/+715
\|\
\| *	Merge branch 'master' into bug/backend/906/converter-year-range	Joseph Banks	2020-06-10	-47/+824
\| \|\
\| \| *	Merge branch 'master' into staff_count_server	kwzrd	2020-06-05	-5/+0
\| \| \|\
\| \| \| *	Merge branch 'master' into stop_sending_avatar_hash	Leon Sandøy	2020-05-31	-31/+638
\| \| \| \|\
\| \| \| * \|	Fix failing tests related to avatar_hash	Leon Sandøy	2020-05-27	-5/+0
\| \| \| \| \|
\| \| * \| \|	Merge branch 'master' into staff_count_server	Leon Sandøy	2020-05-31	-0/+247
\| \| \|\ \ \ \| \| \| \| \|/ \| \| \| \|/\|
\| \| \| * \|	Merge branch 'master' into ban-kick-reason-length	Mark	2020-05-30	-31/+550
\| \| \| \|\ \
\| \| \| \| * \	Merge branch 'master' into test_antimalware	Sebastiaan Zeeff	2020-05-30	-35/+398
\| \| \| \| \|\ \
\| \| \| \| * \| \|	AntiMalware tests - Switched from monkeypatch to unittest.patch	Jannes Jonkers	2020-05-25	-2/+2
\| \| \| \| \| \| \|
\| \| \| \| * \| \|	AntiMalware Refactor - Moved embed descriptions into constants, added tests ↵	Jannes Jonkers	2020-05-11	-18/+30
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	for embed descriptions
\| \| \| \| * \| \|	AntiMalware Tests - Removed exact log content checks	Jannes Jonkers	2020-05-11	-19/+2
\| \| \| \| \| \| \|
\| \| \| \| * \| \|	AntiMalware Tests - added a missing case for no extensions in ↵	Jannes Jonkers	2020-05-11	-0/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	test_get_disallowed_extensions
\| \| \| \| * \| \|	AntiMalware Tests - extracted the method for determining disallowed ↵	Jannes Jonkers	2020-05-11	-6/+23
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	extensions and added a test for it.
\| \| \| \| * \| \|	AntiMalware Tests - implemented minor feedback	Jannes Jonkers	2020-05-11	-12/+12
\| \| \| \| \| \| \|
\| \| \| \| * \| \|	Update tests/bot/cogs/test_antimalware.py	MrGrote	2020-05-08	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-authored-by: Mark <[email protected]>
\| \| \| \| * \| \|	AntiMalware Tests - Switched to unittest.IsolatedAsyncioTestCase	Jannes Jonkers	2020-05-07	-29/+19
\| \| \| \| \| \| \|
\| \| \| \| * \| \|	AntiMalware Tests - Added unittest for txt file attachment	Jannes Jonkers	2020-05-07	-2/+23
\| \| \| \| \| \| \|
\| \| \| \| * \| \|	AntiMalware Tests - Added unittest for valid attachment	Jannes Jonkers	2020-05-07	-0/+9
\| \| \| \| \| \| \|
\| \| \| \| * \| \|	AntiMalware Tests - Added unittest message deletion log	Jannes Jonkers	2020-05-07	-4/+28
\| \| \| \| \| \| \|
\| \| \| \| * \| \|	AntiMalware Tests - Added unittest for cog setup	Jannes Jonkers	2020-05-07	-0/+10
\| \| \| \| \| \| \|
\| \| \| \| * \| \|	AntiMalware Tests - Added unittest for messages that were deleted in the ↵	Jannes Jonkers	2020-05-07	-1/+19
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	meantime.
\| \| \| \| * \| \|	AntiMalware Tests - Added unittest for the embed for a python file.	Jannes Jonkers	2020-05-07	-1/+24
\| \| \| \| \| \| \|
\| \| \| \| * \| \|	AntiMalware Tests - Added unittest for messages send by staff	Jannes Jonkers	2020-05-07	-1/+12
\| \| \| \| \| \| \|
\| \| \| \| * \| \|	AntiMalware Tests - Added unittests for deletion of message and ignoring of dms	Jannes Jonkers	2020-05-07	-1/+21
\| \| \| \| \| \| \|
\| \| \| \| * \| \|	AntiMalware Tests - Added unittest for message without attachment	Jannes Jonkers	2020-05-07	-0/+20
\| \| \| \| \| \| \|
\| \| \| * \| \| \|	Infr. Tests: Add `apply_infraction` awaiting assertion with args	ks129	2020-05-30	-0/+6
\| \| \| \| \| \| \|
\| \| \| * \| \| \|	Simplify infraction reason truncation tests	ks129	2020-05-30	-13/+7
\| \| \| \| \| \| \|
\| \| \| * \| \| \|	Infr Tests: Make `get_active_infraction` return `None`	ks129	2020-05-20	-1/+1
\| \| \| \| \| \| \|
\| \| \| * \| \| \|	Infr. Test: Replace `get_active_mock` return value	ks129	2020-05-20	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Replace `{"foo": "bar"}` with `{"id": 1}`
\| \| \| * \| \| \|	Infr. Tests: Replace `str` with `dict`	ks129	2020-05-20	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	To allow `.get`, I had to replace `str` return value with `dict`
\| \| \| * \| \| \|	Infr. Tests: Add more content to await comment	ks129	2020-05-20	-2/+2
\| \| \| \| \| \| \|
\| \| \| * \| \| \|	ModLog Tests: Fix embed description truncate test	ks129	2020-05-20	-1/+1
\| \| \| \| \| \| \|
\| \| \| * \| \| \|	Merge branch 'master' into ban-kick-reason-length	ks129	2020-05-20	-11/+183
\| \| \| \|\ \ \ \ \| \| \| \| \| \|_\|/ \| \| \| \| \|/\| \|
\| \| \| * \| \| \|	Infraction Tests: Small fixes	ks129	2020-05-20	-4/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	- Remove unnecessary space from placeholder - Rename `has_active_infraction` to `get_active_infraction`
\| \| \| * \| \| \|	ModLog Tests: Fix truncation tests docstring	ks129	2020-05-20	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-authored-by: Leon Sandøy <[email protected]>
\| \| \| * \| \| \|	(Infractions and ModLog Tests): Replaced `shortening` with `truncation`, ↵	ks129	2020-04-16	-5/+6
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	removed unnecessary type hint and added comment to kick truncation test about awaiting `kick`.
\| \| \| * \| \| \|	(ModLog Tests): Created reason shortening tests for `send_log_message`.	ks129	2020-04-16	-0/+29
\| \| \| \| \| \| \|
\| \| \| * \| \| \|	(Infraction Tests): Created reason shortening tests for ban and kick.	ks129	2020-04-16	-0/+54
\| \| \| \| \| \| \|
\| \| \| * \| \| \|	(Test Helpers): Added `__ge__` function to `MockRole` for comparing.	ks129	2020-04-15	-0/+4
\| \| \| \| \| \| \|
\| \| * \| \| \| \|	Fix busted test_information test.	Leon Sandøy	2020-05-30	-4/+8
\| \| \| \|_\|_\|/ \| \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	I wish this test didn't exist.
\| \| * \| \| \|	Add custom exceptions for each error state.	Leon Sandøy	2020-05-28	-3/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The bot can get into trouble in three distinct ways: - It has no Bot instance - It has no namespace - It has no parent instance. These happen only if you're using it wrong. To make the test more precise, and to add a little bit more readability (RuntimeError could be anything!), we'll introduce some custom exceptions for these three states. This addresses a review comment by @aeros.
\| \| * \| \| \|	Merge branch 'master' into redis_persistence	Leon Sandøy	2020-05-27	-10/+49
\| \| \|\ \ \ \
\| \| \| * \| \| \|	Add some tests for `in_whitelist_check`.	Leon Sandøy	2020-05-27	-0/+48
\| \| \| \| \| \| \|
\| \| \| * \| \| \|	Remove the now deprecated in_channel_check.	Leon Sandøy	2020-05-27	-8/+0
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This check was no longer being used anywhere, having been replaced by in_whitelist_check.
\| \| \| * \| \| \|	Find + change all InWhitelistCheckFailure imports	Leon Sandøy	2020-05-27	-2/+1
\| \| \| \| \| \| \|
\| \| \| * \| \| \|	Refactor the in_whitelist deco to a check.	Leon Sandøy	2020-05-27	-2/+2
\| \| \| \| \|/ / \| \| \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	We're moving the actual predicate into the `utils.checks` folder, just like we're doing with most of the other decorators. This is to allow us the flexibility to use it as a pure check, not only as a decorator. This commit doesn't actually change any functionality, just moves it around.
\| \| * \| \| \|	Floats are no longer permitted as RedisCache keys.	Leon Sandøy	2020-05-27	-5/+8
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Also added a test for this. This is the DRYest approach I could find. It's a little ugly, but I think it's probably good enough.
\| \| * \| \| \|	Refactor .increment and add lock test.	Leon Sandøy	2020-05-27	-64/+99
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The way we were doing the asyncio.Lock() stuff for increment was slightly problematic. @aeros has adviced us that it's better to just initialize the lock as None in __init__, and then initialize it inside the first coroutine that uses it instead. This ensures that the correct loop gets attached to the lock, so we don't end up getting errors like this one: RuntimeError: got Future <Future pending> attached to a different loop This happens because the lock and the actual calling coroutines aren't on the same loop. When creating a new test, test_increment_lock, we discovered that we needed a small refactor here and also in the test class to make this new test pass. So, now we're creating a DummyCog for every test method, and this will ensure the loop streams never cross. Cause we all know we must never cross the streams.