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

	Commit message (Collapse)	Author	Age	Lines
...
\| \| * \| \| \| \| \| \| \|	Add more valid tokens to test the regex with	MarkKoz	2020-05-25	-3/+5
\| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \|	Test token regex won't match non-base64 characters	MarkKoz	2020-05-25	-4/+3
\| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \|	Fix a test needlessly being a coroutine	MarkKoz	2020-05-13	-1/+1
\| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \|	Use subtests for valid ID/timestamp tests and test non-ASCII inputs	MarkKoz	2020-05-11	-18/+25
\| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \|	Clean up token remover test imports	MarkKoz	2020-05-11	-16/+12
\| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \|	Replace deprecated assertion methods	MarkKoz	2020-05-11	-2/+2
\| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \|	Refactor `TokenRemoverSetupTests` and add a more thorough test	MarkKoz	2020-05-11	-4/+8
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The test now ensures the cog is instantiated and that the instance is passed as an argument to `add_cog`.
\| \| * \| \| \| \| \| \| \|	Test TokenRemover.take_action	MarkKoz	2020-05-11	-43/+30
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	* Remove `bot.get_cog` mocks in `setUp` * Mock the logger cause it's easier to assert logs * Remove subtests * Assert helper functions were called * Create an autospec for ModLog
\| \| * \| \| \| \| \| \| \|	Test TokenRemover.format_log_message	MarkKoz	2020-05-11	-0/+16
\| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \|	Test token remover's message deletion	MarkKoz	2020-05-11	-0/+9
\| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \|	Simplify token remover's message mock	MarkKoz	2020-05-11	-9/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	* Rely on default values for the author * Set the content to a non-empty string
\| \| * \| \| \| \| \| \| \|	Avoid instantiating the cog when testing static/class methods	MarkKoz	2020-05-11	-10/+5
\| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \|	Correct the return type annotation for the autospec decorator	MarkKoz	2020-05-11	-2/+2
\| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \|	Test token regex matches valid tokens	MarkKoz	2020-05-11	-0/+21
\| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \|	Test is_maybe_token	MarkKoz	2020-05-11	-7/+24
\| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \|	Test `is_maybe_token` returns False for missing parts	MarkKoz	2020-05-11	-0/+10
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	In practice, this won't ever happen since the regex wouldn't match strings with missing parts. However, the function does check it so may as well test it. It's not necessarily bound to always use inputs from the regex either I suppose.
\| \| * \| \| \| \| \| \| \|	Fix autospec decorator when used with multiple attributes	MarkKoz	2020-05-11	-16/+12
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The original approach of messing with the `attribute_name` didn't work for reasons I won't discuss here (would require knowledge of patcher internals). The new approach doesn't use patch.multiple but mimics it by applying multiple patch decorators to the function. As a consequence, this can no longer be used as a context manager.
\| \| * \| \| \| \| \| \| \|	Test token regex doesn't match invalid tokens	MarkKoz	2020-05-11	-7/+25
\| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \|	Test `find_token_in_message` returns the found token	MarkKoz	2020-05-11	-0/+24
\| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \|	Test `find_token_in_message` returns None if no matches found	MarkKoz	2020-05-11	-0/+14
\| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \|	Fix test for token remover ignoring bot messages	MarkKoz	2020-05-11	-4/+9
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	It's not possible to test this via asserting the return value of `on_message` since it never returns anything. Instead, the actual relevant unit, `find_token_in_message,` should be tested.
\| \| * \| \| \| \| \| \| \|	Allow using arbitrary parameter names with the autospec decorator	MarkKoz	2020-05-11	-2/+15
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This gives the caller more flexibility. Sometimes attribute names are too long or they don't follow a naming scheme accepted by the linter.
\| \| * \| \| \| \| \| \| \|	Test token remover skips messages without tokens	MarkKoz	2020-05-11	-0/+11
\| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \|	Test token remover takes action if a token is found	MarkKoz	2020-05-11	-1/+13
\| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \|	Add a test helper function to patch multiple attributes with autospecs	MarkKoz	2020-05-11	-0/+9
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This helper reduces redundancy/boilerplate by setting default values. It also has the consequence of shortening the length of the invocation, which makes it faster to use and easier to read.
\| \| * \| \| \| \| \| \| \|	Test on_message_edit of token remover uses on_message	MarkKoz	2020-05-11	-2/+10
\| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \|	Merge pull request #978 from ItsDrike/unsilence-scheduler	Mark	2020-06-12	-4/+14
\| \|\ \ \ \ \ \ \ \ \ \| \| \|_\|_\|_\|_\|_\|_\|_\|/ \| \|/\| \| \| \| \| \| \| \|	Use Scheduler instead of asyncio.sleep on silence cog
\| \| * \| \| \| \| \| \| \|	Test for channel not silenced message	ItsDrike	2020-06-06	-4/+14
\| \| \| \|_\|_\|_\|/ / / \| \| \|/\| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	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 staff_count_server	Leon Sandøy	2020-05-31	-0/+247
\| \| \| \|\ \ \ \ \ \ \| \| \| \| \| \|/ / / / \| \| \| \| \|/\| \| \| \|
\| \| \| * \| \| \| \| \|	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
\| \| \|/ / / / / / \| \|/\| \| \| \| \| \|
* \| \| \| \| \| \| \|	In LinePaginator, truncate words that exceed scale_to_size	Kyle Stanley	2020-06-26	-7/+5
\| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \|	Update LinePaginator.add_line() tests	Kyle Stanley	2020-06-26	-5/+5
\| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \|	Merge branch 'master' into issue926-paginate-long-lines	Kyle Stanley	2020-06-04	-40/+645
\|\ \ \ \ \ \ \ \ \| \| \|_\|_\|/ / / / \| \|/\| \| \| \| \| \|
\| * \| \| \| \| \| \|	Merge branch 'master' into stop_sending_avatar_hash	Leon Sandøy	2020-05-31	-31/+638
\| \|\ \ \ \ \ \ \ \| \| \| \|_\|/ / / / \| \| \|/\| \| \| \| \|
\| \| * \| \| \| \| \|	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
\| \| \| \|\\| \| \| \| \|
\| \| \| \| * \| \| \| \|	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.