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

	Commit message (Collapse)	Author	Age	Lines
...
\| \| * \| \| \| \| \| \| \| \|	Merge branch 'master' into bad-words-nick	ks129	2020-06-11	-60/+317
\| \| \|\ \ \ \ \ \ \ \ \ \| \| \| \| \|_\|_\|_\|_\|_\|/ / \| \| \| \|/\| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \|	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
\| \| \| \| \| \|\ \ \ \ \ \
\| \| \| \| \| * \| \| \| \| \| \|	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.
\| \| \| * \| \| \| \| \| \| \|	Fix exception message assertions in converter tests	MarkKoz	2020-04-28	-7/+8
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The `msg` arg is for displaying a message when the assertion fails. To match against the exception's message, `assertRaisesRegex` must be used. Since all of the messages are meant to be interpreted literally rather than as regex, `re.escape` is used.
\| \| \| * \| \| \| \| \| \| \|	Use await instead of asyncio.run in converter tests	MarkKoz	2020-04-28	-28/+27
\| \| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \|	Remove redundant parenthesis from test values	MarkKoz	2020-04-28	-19/+19
\| \| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \|	Test for out of range datetime in the Duration converter	MarkKoz	2020-04-28	-0/+11
\| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \|	Remove redis namespace collision test	MarkKoz	2020-05-31	-10/+0
\| \| \| \|/ / / / / / / \| \| \|/\| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \|	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
\| \| \|_\|_\|_\|_\|/ / / \| \|/\| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \|	Change ValidPythonIdentifier tests to PackageName.	Numerlor	2020-07-22	-11/+10
\| \|_\|_\|_\|_\|/ / / \|/\| \| \| \| \| \| \|
* \| \| \| \| \| \| \|	Merge branch 'master' into test_antimalware	Sebastiaan Zeeff	2020-05-30	-35/+398
\|\ \ \ \ \ \ \ \ \| \| \|/ / / / / / \| \|/\| \| \| \| \| \|
\| * \| \| \| \| \| \|	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.
\| * \| \| \| \| \| \|	Clear cache in asyncSetUp instead of tests.	Leon Sandøy	2020-05-27	-6/+1
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Add a test for RuntimeErrors.	Leon Sandøy	2020-05-26	-0/+22
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This just tests that the various RuntimeErrors are reachable - that includes the error about not having a bot instance, the one about not being a class attribute, and the one about not having instantiated the class. This test addresses a concern raised by @MarkKoz in a review. I've decided not to test that actual contents of these RuntimeErrors, because I believe that sort of testing is a bit too brittle. It shouldn't break a test just to change the content of an error string.
\| * \| \| \| \| \| \|	Improve output of section name in config validation subtests	MarkKoz	2020-05-25	-1/+1
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Support validating collection types for constants	MarkKoz	2020-05-25	-8/+32
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This is a simple validation that only check the type of the collection. It does not validate the types inside the collection because that has proven to be quite complex.
\| * \| \| \| \| \| \|	Add support for Union type annotations for constants	MarkKoz	2020-05-24	-5/+12
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Note that `Optional[x]` is just an alias for `Union[None, x]` so this effectively supports `Optional` too. This was especially troublesome because the redis password must be unset/None in order to avoid authentication, but the test would complain that `None` isn't a `str`. Setting to an empty string would pass the test but then make redis authenticate and fail.
\| * \| \| \| \| \| \|	Improve error and error testing for increment	Leon Sandøy	2020-05-24	-0/+8
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Changed a RuntimeError to a KeyError (thanks @MarkKoz), and also added some tests to ensure that the right errors are raised whenever this method is used incorrectly.
\| * \| \| \| \| \| \|	MockBot needs to be aware of redis_ready	Leon Sandøy	2020-05-24	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Forgot to update the additional_spec_asyncs when changing the name of this Bot attribute to be public.
\| * \| \| \| \| \| \|	Add .increment and .decrement methods.	Leon Sandøy	2020-05-24	-0/+34
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Sometimes, we just want to store a counter in the cache. In this case, it is convenient to have a single method that will allow us to increment or decrement this counter. These methods allow you to decrement or increment floats and integers by an specified amount. By default, it'll increment or decrement by 1. Since this involves several API requests, we create an asyncio.Lock so that we don't end up with race conditions.
\| * \| \| \| \| \| \|	Make .items return ItemsView instead of AsyncIter	Leon Sandøy	2020-05-24	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	There really was no compelling reason why this method should return an AsyncIterator or than that `async for items in cache.items()` has nice readability, but there were a few concerns. One is a concern about race conditions raised by @SebastiaanZ, and @MarkKoz raised a concern that it was misleading to have an AsyncIterator that only "pretended" to be lazy. To address these concerns, I've refactored it to return a regular ItemsView instead. I also improved the docstring, and fixed the relevant tests.