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

	Commit message (Collapse)	Author	Age	Lines
...
\| \| \| * \| \| \|	(Information Tests, discord.py 1.3.x Migration): In `InformationCogTests`, ↵	ks129	2020-04-03	-8/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	replaced `.callback` calls with direct command awaits.
\| \| \| * \| \| \|	(Information Tests, discord.py 1.3.x Migration): Moved from ↵	ks129	2020-04-03	-25/+29
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	`unittest.TestCase` to `unittest.IsolatedAsyncTestCase` rest of test case classes.
\| \| \| * \| \| \|	(Information Tests, discord.py 1.3.x Migration): Moved from ↵	ks129	2020-04-03	-4/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	`unittest.TestCase` to `unittest.IsolatedAsyncTestCase` in `InformationCogTests`.
\| \| \| * \| \| \|	(Snekbox Fix, discord.py 1.3.x Migration): Applied one reaction clear to tests.	ks123	2020-04-02	-2/+2
\| \| \| \| \| \| \|
\| \| \| * \| \| \|	(Patches, discord.py 1.3.x Migration): Removed patches due not longer necessary.	ks123	2020-04-02	-0/+0
\| \| \| \| \| \| \|
\| \| * \| \| \| \|	Refactor unit tests UserSyncerSyncTests to use changes made to UserSyncer in ↵	RohanJnr	2020-10-02	-7/+5
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	_syncers.py
\| \| * \| \| \| \|	update UserSyncerDiffTests Tests to use changes made to API calls.	RohanJnr	2020-10-02	-6/+32
\| \| \| \| \| \| \|
\| \| * \| \| \| \|	Merge branch 'master' into smart_syncing_users	Rohan Reddy Alleti	2020-10-01	-371/+138
\| \| \|\ \ \ \ \ \| \| \| \| \|/ / / \| \| \| \|/\| \| \|
\| \| \| * \| \| \|	PR #1151 Sync: remove confirmation	kwzrd	2020-09-26	-345/+14
\| \| \| \|\ \ \ \
\| \| \| \| * \ \ \	Merge branch 'master' into feat/backend/1075/remove-syncer-confirm	MarkKoz	2020-09-22	-972/+574
\| \| \| \| \|\ \ \ \
\| \| \| \| * \| \| \| \|	Fix test for sync message being edited	MarkKoz	2020-09-07	-1/+3
\| \| \| \| \| \| \| \| \|
\| \| \| \| * \| \| \| \|	Sync: remove confirmation	MarkKoz	2020-09-07	-345/+12
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The confirmation was intended to be a safe guard against cache issues that would cause a huge number of roles/users to deleted after syncing. With `wait_until_guild_available`, such cache issue shouldn't arise. Therefore, this feature is obsolete. Resolve #1075
\| \| * \| \| \| \| \| \|	modify tests to use paginated response.	RohanJnr	2020-09-27	-6/+37
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Silence tests: make _async_init attribute tests more robust	MarkKoz	2020-10-08	-7/+11
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Silence: remove _mod_log_channel attribute	MarkKoz	2020-10-08	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	It's only used as an argument to `SilenceNotifier`, so it doesn't need to be an instance attribute.
\| * \| \| \| \| \| \| \|	Silence: add to notifier when indefinite rather than temporary	MarkKoz	2020-10-08	-6/+6
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Accidentally swapped the logic in a previous commit during a refactor.
\| * \| \| \| \| \| \| \|	Silence tests: fix unawaited coro warnings	MarkKoz	2020-10-07	-2/+7
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Because the Scheduler is mocked, it doesn't actually do anything with the coroutines passed to the schedule() functions, hence the warnings.
\| * \| \| \| \| \| \| \|	Silence: rename _init_cog to _async_init	MarkKoz	2020-10-06	-13/+13
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Silence: refactor _silence	MarkKoz	2020-10-06	-16/+22
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	* Rename to `_silence_overwrites` * Reduce responsibilities to only setting permission overwrites * Log in `silence` instead * Add to notifier in `silence` instead
\| * \| \| \| \| \| \| \|	Silence: cancel init task when cog unloads	MarkKoz	2020-08-18	-2/+5
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Silence: rename caches	MarkKoz	2020-08-18	-20/+20
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Silence tests: remove unnecessary spec_set args	MarkKoz	2020-08-18	-3/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	It's not really necessary to set to True when mocking functions.
\| * \| \| \| \| \| \| \|	Silence tests: more accurately assert the silence cmd schedule a task	MarkKoz	2020-08-18	-3/+7
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Silence tests: add a test for rescheduling active silences	MarkKoz	2020-08-18	-0/+29
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Silence tests: test the rescheduler unsilences expired silences	MarkKoz	2020-08-18	-0/+14
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Silence tests: test the rescheduler adds permanent silence to notifier	MarkKoz	2020-08-18	-0/+14
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Silence tests: test the scheduler skips missing channels	MarkKoz	2020-08-18	-0/+25
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Silence tests: add a test for _init_cog rescheduling unsilences	MarkKoz	2020-08-18	-0/+7
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Silence tests: remove _reschedule patch for cog tests	MarkKoz	2020-08-18	-4/+0
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	They don't do anything because they patch the class rather than the instance. It's too late for patching the instance to work since the `setUp` fixture, which instantiates the cog, executes before the patches do. Patching `setUp` would work (and its done in the other test cases), but some tests in this case will need the unpatched function too. Patching it doesn't serve much benefit to most tests anyway, so it's not worth the effort trying to make them work where they aren't needed.
\| * \| \| \| \| \| \| \|	Silence tests: access everything via the silence module	MarkKoz	2020-08-18	-24/+23
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The module is imported anyway to keep imports short and clean. Using it in patch targets is shorter and allows for the two imports from the module to be removed.
\| * \| \| \| \| \| \| \|	Silence tests: add a subtest for the manual unsilence message	MarkKoz	2020-08-17	-4/+7
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Silence tests: assert against message constants	MarkKoz	2020-08-17	-6/+7
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Duplicating strings in assertions is redundant, closely coupled, and less maintainable.
\| * \| \| \| \| \| \| \|	Silence tests: also test that cache misses preserve other overwrites	MarkKoz	2020-08-17	-12/+16
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Silence tests: add a test for a mod alert on cache miss	MarkKoz	2020-08-17	-0/+7
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Silence tests: add a test for default overwrites on cache miss	MarkKoz	2020-08-17	-4/+19
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Use a False for `add_reactions` in the mock overwrite rather than None to be sure the default (also None) is actually set for it. Fix channels set by `_init_cog` not being mocked properly.
\| * \| \| \| \| \| \| \|	Silence tests: add a test for task cancellation	MarkKoz	2020-08-17	-0/+5
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Silence tests: add a test for deletion from the time cache	MarkKoz	2020-08-17	-0/+5
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Silence tests: add tests for scheduling tasks	MarkKoz	2020-08-17	-0/+12
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Silence tests: add a test for caching permanent times	MarkKoz	2020-08-17	-0/+6
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Silence tests: add a test for the time cache	MarkKoz	2020-08-17	-0/+15
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Silence tests: fix mock for _init_task	MarkKoz	2020-08-17	-2/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	An `AsyncMock` fails because it returns a coroutine which may only be awaited once. However, an `asyncio.Future` is perfect because it is easy to create and can be awaited repeatedly, just like the actual `asyncio.Task` that is being mocked.
\| * \| \| \| \| \| \| \|	Silence tests: create channel and overwrite in setUp for silence tests	MarkKoz	2020-08-17	-29/+17
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Reduce code redundancy by only defining them once.
\| * \| \| \| \| \| \| \|	Silence tests: merge unsilence fixture into setUp	MarkKoz	2020-08-17	-32/+20
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Now that there are separate test cases, there's no need to keep the fixtures separate.
\| * \| \| \| \| \| \| \|	Silence tests: autospec _reschedule and SilenceNotifier for cog tests	MarkKoz	2020-08-17	-4/+10
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Silence tests: separate test cases; refactor names & docstrings	MarkKoz	2020-08-17	-71/+95
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Silence tests: fix tests for _unsilence	MarkKoz	2020-08-17	-30/+45
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Add a fixture to set up mocks for a successful `unsilence` call. This reduces code redundancy.
\| * \| \| \| \| \| \| \|	Silence tests: fix test_unsilence_private_removed_notifier	MarkKoz	2020-08-17	-2/+5
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Silence tests: fix overwrite preservation test for silences	MarkKoz	2020-08-17	-11/+15
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Silence tests: fix command message tests	MarkKoz	2020-08-17	-15/+10
\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Silence tests: fix test_silence_private_for_false	MarkKoz	2020-08-17	-4/+13
\| \| \| \| \| \| \| \| \|