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

	Commit message (Collapse)	Author	Age	Lines
...
\| \| \| * \| \| \| \| \| \| \| \| \|	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 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.