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

	Commit message (Collapse)	Author	Age	Lines
...
\| \| * \| \| \| \| \| \| \|	Get up to 3 signatures of a symbol	Numerlor	2019-11-02	-16/+14
\| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \|	Get symbol description by searching for a dd tag instead of traversing the ↵	Numerlor	2019-11-02	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	siblings
\| \| * \| \| \| \| \| \| \|	Do not cut off description in code blocks	Numerlor	2019-11-02	-1/+7
\| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \|	Grammar check comment	Numerlor	2019-10-21	-1/+1
\| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \|	Allow embeds to not include signatures in case the symbol is a module	Numerlor	2019-10-21	-11/+14
\| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \|	Don't include a signature and only get first paragraphs when scraping when ↵	Numerlor	2019-10-21	-10/+18
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	symbol is a module
\| \| * \| \| \| \| \| \| \|	remove "function" from NO_OVERRIDE_GROUPS	Numerlor	2019-10-20	-1/+0
\| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \|	Auto delete messages when docs are not found	Numerlor	2019-10-20	-2/+8
\| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \|	show renamed duplicates in embed footer	Numerlor	2019-10-20	-8/+13
\| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \|	add handling for duplicate symbols in docs inventories	Numerlor	2019-10-20	-4/+35
\| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| * \|	Keeps access token alive, only revokes it on extension unload.	Jens	2019-12-05	-20/+32
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Hard-coded version number to 1.0.0.
\| \| \| \| \| \| \| \| * \|	Fix linting error	Jens	2019-10-26	-0/+1
\| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| * \|	Resolved merge conflicts.	Jens	2019-10-26	-1605/+2753
\| \| \| \| \| \| \| \| \|\ \ \| \| \| \| \|_\|_\|_\|_\|/ / \| \| \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Add retry loop to token request. Revoke access token after usage.
\| \| \| \| \| \| \| \| * \|	Add reddit environment variable, change User-Agent and fix lint problem	Jens	2019-10-22	-2/+4
\| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| * \|	Merge branch 'master' of https://github.com/python-discord/bot into ↵	Jens	2019-10-16	-92/+291
\| \| \| \| \| \| \| \| \|\ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	reddit-api-oauth
\| \| \| \| \| \| \| \| * \ \	Resolve merge conflict	Jens	2019-10-15	-4/+2
\| \| \| \| \| \| \| \| \|\ \ \
\| \| \| \| \| \| \| \| \| * \| \|	Add Reddit OAuth tasks and refactor code	Jens	2019-10-09	-6/+77
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| * \| \| \|	Unload cog on auth error and fix linting warnings	Jens	2019-10-15	-19/+16
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| * \| \| \|	Add Reddit OAuth tasks and refactor code	Jens	2019-10-14	-6/+76
\| \| \| \|_\|_\|_\|_\|/ / / / \| \| \|/\| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| *	Specify assertion to be a tuple comparison	kwzrd	2019-11-14	-1/+1
\| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| *	Make complex test cases namedtuples, recognize between various authors, pass ↵	kwzrd	2019-11-14	-12/+43
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	config to subTest
\| \| \| \| \| \| \| \| \| \| *	Hold recent_messages in a list to respect type hint, set config in setUp	kwzrd	2019-11-14	-5/+8
\| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| *	Adjust attachments rule unit test to correcty build the arguments for the ↵	kwzrd	2019-11-14	-7/+11
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	tested rule
\| \| \| \| \| \| \| \| \| \| *	Fix bug in attachments rule where last_message could potentially count twice ↵	kwzrd	2019-11-14	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	in the sum of attachments
\| \| \| \| \| \| \| \| \| \| *	Use async_test helper to simplify coro testing	kwzrd	2019-11-13	-9/+14
\| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| *	Adjust attachments rule to use MockMessage, restructure test cases	kwzrd	2019-11-13	-20/+23
\| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| *	Adjust links rule to use proper MockMessage	kwzrd	2019-11-13	-14/+10
\| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| *	Merge branch 'master' into unittest-mentions	kwzrd	2019-11-13	-179/+187
\| \| \| \| \| \| \| \| \| \| \|\ \| \|_\|_\|_\|_\|_\|_\|_\|_\|_\|/ \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Testing methodology was adjusted in upstream repo. Merging the relevant changes.
* \| \| \| \| \| \| \| \| \| \|	Enhancements for `tests.helpers` and our test suite (#660)	scragly	2019-11-14	-157/+134
\|\ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Enhancements for `tests.helpers` and our test suite Co-authored-by: null <[email protected]>
\| * \ \ \ \ \ \ \ \ \ \	Merge branch 'master' into unittest-helpers-improvements	scragly	2019-11-14	-36/+167
\| \|\ \ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #656 from python-discord/checkpoint-changes	Mark	2019-11-10	-22/+53
\|\ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Forward pings in checkpoint to mod-alerts and reword periodic checkpoint ping
\| * \ \ \ \ \ \ \ \ \ \ \	Merge branch 'master' into checkpoint-changes	Mark	2019-11-10	-540/+677
\| \|\ \ \ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \|	[kaizen] Remove now duplicate channel check	Derek	2019-11-09	-19/+18
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Unindent subsequent lines after check
\| * \| \| \| \| \| \| \| \| \| \| \|	Forward user/role pings in checkpoint to mod-alerts	Derek	2019-11-09	-2/+34
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	[kaizen] limit on_message listener to verification channel
\| * \| \| \| \| \| \| \| \| \| \| \|	Reword periodic #checkpoint message	Derek	2019-11-04	-1/+1
\| \| \|_\|_\|_\|_\|_\|_\|/ / / / \| \|/\| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \| \|	Prevent await warnings for MockBot's create_task	Sebastiaan Zeeff	2019-11-13	-1/+19
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Previously, the coroutine object passed to `MockBot.loop.create_task` would trigger a `RuntimeWarning` for not being awaited as we do not actually create a task for it. To prevent these warnings, coroutine objects passed will now automatically be closed.
\| \| * \| \| \| \| \| \| \| \| \|	Prevent setting unknown attributes on d.py mocks	Sebastiaan Zeeff	2019-11-13	-158/+101
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Our custom `discord.py` now follow the specifications of the object they are mocking more strictly by using the `spec_set` instead of the `spec` kwarg to initialize the specifications. This means that trying to set an attribute that does not follow the specifications will now also result in an `AttributeError`. To make sure we are not trying to set illegal attributes during the default initialization of the mock objects, I've changed the way we handle default values of parameters. This does introduce a breaking change: Instead of passing a `suffix_id`, the `id` attribute should now be passed using the exact name. `id`. This commit also makes sure existing tests follow this change.
\| \| * \| \| \| \| \| \| \| \| \|	Allow `name` attribute to be set during Mock init	Sebastiaan Zeeff	2019-11-13	-0/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The `name` keyword argument has a special meaning for the default mockobjects provided by `unittest.mock`. This means that by default, the common d.py `name` attribute can't be set during initalization of one of our custom Mock-objects by passing it to the constructor. Since it's unlikely for us to make use of the special `name` feature of mocks and more likely to want to set the d.py `name` attribute, I added special handling of the `name` kwarg.
\| \| * \| \| \| \| \| \| \| \| \|	Prevent unwanted logging while running tests	Sebastiaan Zeeff	2019-11-13	-1/+14
\| \| \|/ / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Previously, logging messages would output to std.out. when running individual test files (instead of running the entire suite). To prevent this, I've added a `for`-loop to `tests.helpers` that sets the level of all registered loggers to `CRITICAL`. The reason for adding this to `tests.helpers` is simple: It's the most common file to be imported in individual tests, increasing the chance of the code being run for individual test files. A small downside of this way of handling logging is that when we are trying to assert logging messages are being emitted, we need to set the logger explicitly in the `self.assertLogs` context manager. This is a small downside, though, and probably good practice anyway. There was one test in `tests.bot.test_api` that did not do this, so I have changed this to make the test compatible with the new set-up.
\| \| \| \| \| \| \| \| \| \| *	Adjust incorrect type hint	kwzrd	2019-11-09	-1/+1
\| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| *	Use MockMessage instead of custom FakeMessage	kwzrd	2019-11-09	-15/+11
\| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| *	Adjust type hint to correctly represent internal type	kwzrd	2019-11-09	-1/+1
\| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| *	Use range to build mock mentions list	kwzrd	2019-11-09	-1/+1
\| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| *	Adjust docstring asterisk to backtick for consistency	kwzrd	2019-11-09	-1/+1
\| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| *	Add missing docstring	kwzrd	2019-11-09	-0/+1
\| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| \| \| \| *	Add unit test for mentions antispam rule	kwzrd	2019-11-09	-0/+98
\| \|_\|_\|_\|_\|_\|_\|_\|_\|/ \|/\| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Merge pull request #641 from kwzrd/unittest-links	Mark	2019-11-08	-0/+101
\|\ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Add unit test for links antispam rule
\| * \ \ \ \ \ \ \ \ \	Merge branch 'master' into unittest-links	Mark	2019-11-08	-608/+1193
\| \|\ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \|	Merge pull request #653 from python-discord/talent-pooling	Sebastiaan Zeeff	2019-11-08	-14/+13
\|\ \ \ \ \ \ \ \ \ \ \ \| \|_\|_\|/ / / / / / / / \|/\| \| \| \| \| \| \| \| \| \|	Allow helpers to nominate people
\| * \| \| \| \| \| \| \| \| \|	Refactor bigbrother to use constants	kosayoda	2019-11-08	-5/+5
\| \| \| \| \| \| \| \| \| \| \|