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

	Commit message (Collapse)	Author	Age	Lines
*	Reminders: fix conflict with the _can_modify check	MarkKoz	2020-09-09	-56/+93
\|\
\| *	Merge branch 'master' into remove-wolfram	Leon Sandøy	2020-09-02	-0/+22
\| \|\
\| \| *	Fixed old tests and added 2 new ones	wookie184	2020-09-01	-0/+22
\| \| \|
\| * \|	Removed ImagePaginator testing.	Xithrius	2020-08-31	-15/+0
\| \|/
\| *	Merge branch 'master' into feat/backend/707/alias-decorator	Shirayuki Nekomata	2020-08-26	-32/+55
\| \|\
\| \| *	Update tests for user commands	Joseph Banks	2020-08-14	-32/+55
\| \| \|
\| * \|	Include root aliases in the command name conflict test	MarkKoz	2020-08-22	-0/+1
\| \|/
\| *	Merge branch 'master' into whitelist_system	Leon Sandøy	2020-08-03	-3/+946
\| \|\
\| * \|	Fix broken antimalware tests.	Leon Sandøy	2020-07-29	-1/+1
\| \| \|
\| * \|	Restructure bot.filter_list_cache.	Leon Sandøy	2020-07-29	-5/+5
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This is an optimization designed to eliminate all the list comprehensions we were doing inside antimalware and filtering. The cache is now structured so that the content is the key and the metadata is the value.
\| * \|	Rename AllowDenyList to FilterLists	Leon Sandøy	2020-07-27	-1/+1
\| \| \|
\| * \|	Remove AntiMalWare constants, use cache data.	Leon Sandøy	2020-07-18	-9/+15
\| \| \| \| \| \| \| \| \| \| \| \|	Also updates the tests for this cog.
* \| \|	Decorators: clean up imports	MarkKoz	2020-07-31	-2/+2
\| \|/ \|/\|
* \|	Jam tests: use the MAX_CHANNELS constant more	MarkKoz	2020-07-22	-4/+4
\| \| \| \| \| \| \| \|	It's clearer to write MAX_CHANNELS - 2 than a literal 48.
* \|	Jam tests: assert equality of new category	MarkKoz	2020-07-22	-1/+2
\| \|
* \|	Jam tests: add subtests to non-existent category test	MarkKoz	2020-07-22	-7/+18
\| \| \| \| \| \| \| \| \| \|	The test has to account for not only the name not matching, but also a lack of available spaces for new channels.
* \|	Jam tests: fix category test	MarkKoz	2020-07-22	-14/+24
\| \|
* \|	Merge branch 'master' into jam-test	Mark	2020-07-22	-185/+1384
\|\ \
\| * \|	Edited tests to reflect changes (removed py formatting)	wookie184	2020-07-20	-3/+3
\| \| \|
\| * \|	Merge 'master' into 'kwzrd/incidents'; resolve config conflicts	kwzrd	2020-07-16	-39/+24
\| \|\\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Add upstream changes & resolve config conflicts introduced by #1041. In the conflicting parts of the diff, this commit also re-sorts constants which were added upstream to lexicographical order. This does not re-sort all constants added upstream.
\| \| *	Merge branch 'master' into dm_relay	Leon Sandøy	2020-07-15	-0/+12
\| \| \|\
\| \| \| *	edit snekbox tests to work with filtering	Slushie	2020-07-13	-0/+12
\| \| \| \|
\| \| * \|	Fix DuckPond tests now that send_webhook is gone.	Leon Sandøy	2020-07-12	-39/+12
\| \| \|/ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Some of the tests were failing because they were expecting send_webhook to be a method of the DuckPond cog, other tests simply were no longer applicable, and have been removed. https://github.com/python-discord/bot/issues/667
\| * \|	Incidents: reduce log level of 404 exception	kwzrd	2020-07-16	-2/+12
\| \| \| \| \| \| \| \| \| \| \| \|	Co-authored-by: MarkKoz <[email protected]>
\| * \|	Incidents: link `proxy_url` if attachment fails to download	kwzrd	2020-07-14	-37/+36
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Suggested by Mark during review. If the download fails, we fallback on showing an informative message, which will link the attachment cdn link. The attachment-handling logic was moved from the `archive` coroutine into `make_embed`, which now also returns the file, if available. In the end, this appears to be the smoothest approach. Co-authored-by: MarkKoz <[email protected]>
\| * \|	Incidents: add `download_file` helper & tests	kwzrd	2020-07-13	-0/+20
\| \| \| \| \| \| \| \| \| \| \| \|	Co-authored-by: MarkKoz <[email protected]>
\| * \|	Incidents tests: improve in-line comment wording	kwzrd	2020-07-09	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \|	Co-authored-by: MarkKoz <[email protected]>
\| * \|	Merge 'master' into 'kwzrd/incidents'; resolve mod pkg init conflict	kwzrd	2020-07-08	-24/+242
\| \|\\|
\| \| *	Fix imports in slowmode tests	Den4200	2020-07-06	-3/+3
\| \| \|
\| \| *	Improve set_slowmode tests by checking whether the channel was edited	Den4200	2020-07-06	-8/+22
\| \| \|
\| \| *	Add multiple test cases for set_slowmode tests	Den4200	2020-07-06	-10/+34
\| \| \|
\| \| *	Use local text_channel instead of instance attribute	Den4200	2020-07-06	-13/+10
\| \| \|
\| \| *	Add tests for set_slowmode	Den4200	2020-07-06	-0/+20
\| \| \|
\| \| *	Add tests for reset_slowmode	Den4200	2020-07-06	-3/+22
\| \| \|
\| \| *	Add tests for cog_check and get_slowmode	Den4200	2020-07-06	-0/+37
\| \| \|
\| \| *	Merge branch 'master' into issue926-paginate-long-lines	Kyle Stanley	2020-06-26	-191/+514
\| \| \|\
\| \| \| *	Merge branch 'master' into bug/backend/bot-50/sync-ignore-guilds	kwzrd	2020-06-22	-10/+85
\| \| \| \|\
\| \| \| \| *	Merge branch 'master' into logging-tests	Kieran Siek	2020-06-22	-200/+1047
\| \| \| \| \|\
\| \| \| \| * \|	Logging Tests: Simplify `DEBUG_MODE` `False` test	ks129	2020-05-22	-11/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	- Remove embed attributes checks - Replace `self.dev_log.assert_awaited_once_with` with `self.dev_log.assert_awaited_once`.
\| \| \| \| * \|	Merge branch 'master' into logging-tests	ks129	2020-05-17	-10/+178
\| \| \| \| \|\ \
\| \| \| \| * \| \|	Created tests for `bot.cogs.logging` connected message.	ks129	2020-04-14	-0/+42
\| \| \| \| \| \| \|
\| \| \| * \| \| \|	Sync: fix guild ID check	MarkKoz	2020-06-17	-16/+19
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Need to compare the IDs against each other rather than the Guild object against the ID.
\| \| \| * \| \| \|	Sync: ignore 404s in on_user_update	MarkKoz	2020-06-14	-7/+10
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	404s probably mean the user is from another guild.
\| \| \| * \| \| \|	Sync tests: test listeners ignore events from other guilds	MarkKoz	2020-06-14	-10/+54
\| \| \| \| \| \| \|
\| \| * \| \| \| \|	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
\| \| \|\ \ \ \ \
\| \| * \| \| \| \| \|	Improve LinePaginator to support long lines	Kyle Stanley	2020-06-04	-7/+34
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Incidents: archive incident attachments	kwzrd	2020-07-03	-0/+30
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	There is no handling of file types as explained in the `archive` docstring. Testing indicates that relaying incidents with e.g. a text file attachment is simply a noop in the Discord GUI. If there is at least one attachment, we always only relay the one at index 0, as it is believed the user-sent messages can only contain one attachment at maximum. This also adds an extra test asserting the behaviour when an incident with an attachment is archived. The existing test for `archive` is adjusted to assume no attachments. Joe helped me conceive & test this. Co-authored-by: Joseph Banks <[email protected]>
\| * \| \| \| \| \| \|	Incidents: relay incidents as embeds rather than raw content	kwzrd	2020-07-01	-21/+11
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This applies the previously defined `make_embed` function. As the `archive` function is now simpler, I decided to reduce the amount of whitespace ~ it's a lot more compact now. Tests are adjusted as appropriate.