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

	Commit message (Collapse)	Author	Age	Lines
...
\| \| \| * \| \| \| \| \| \| \|	Incidents tests: assert webhook username is de-clyded	kwzrd	2020-06-20	-0/+19
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	See: a8b4e394d9da57287cd9497cd9bb0a97fa467e84
\| \| \| * \| \| \| \| \| \| \|	Merge branch 'origin/master' into kwzrd/incidents	kwzrd	2020-06-20	-9/+50
\| \| \| \|\ \ \ \ \ \ \ \ \| \| \| \| \| \|_\|_\|_\|/ / / \| \| \| \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Adds the `sub_clyde` function, which we'll apply to our webhook.
\| \| \| * \| \| \| \| \| \| \|	Incidents tests: remove unnecessary patch	kwzrd	2020-06-20	-1/+0
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This is already being patched at class-level.
\| \| \| * \| \| \| \| \| \| \|	Incidents: try-except Signal creation	kwzrd	2020-06-20	-4/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Suggested by Mark during review. This follows the "ask for forgiveness rather than permission" paradigm, ends up being less code to read, and may be seen as more logical / safer. The `ALLOWED_EMOJI` set was renamed to `ALL_SIGNALS` as this now better communicates the set's purpose. Tests adjusted as appropriate. Co-authored-by: MarkKoz <[email protected]>
\| \| \| * \| \| \| \| \| \| \|	Incidents tests: write tests for `on_raw_reaction_add`	kwzrd	2020-06-19	-0/+128
\| \| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \|	Incidents tests: write tests for `crawl_incidents`	kwzrd	2020-06-19	-0/+58
\| \| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \|	Incidents tests: add helper for mocking async for-loops	kwzrd	2020-06-19	-0/+37
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	See the docstring. This does not make the ambition to be powerful enough to be included in `tests.helpers`, and is only intended for local purposes.
\| \| \| * \| \| \| \| \| \| \|	Merge branch 'origin/master' into kwzrd/incidents	kwzrd	2020-06-19	-107/+302
\| \| \| \|\ \ \ \ \ \ \ \
\| \| \| * \| \| \| \| \| \| \| \|	Incidents tests: write tests for `process_event`	kwzrd	2020-06-18	-1/+101
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This also breaks the helpers import statement into a vertical list, as the amount of imports has grown too much. I still believe that this is a preferred alternative to accessing the helpers via module namespace, as we use them a lot, and the added visual noise would be annoying to read - their names are already descriptive enough.
\| \| \| * \| \| \| \| \| \| \| \|	Incidents tests: write tests for `make_confirmation_task`	kwzrd	2020-06-13	-0/+35
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \| \|	Incidents tests: write tests for `on_message`	kwzrd	2020-06-13	-0/+27
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \| \|	Incidents tests: test non-404 error response	kwzrd	2020-06-13	-0/+20
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \| \|	Incidents tests: write tests for `resolve_message`	kwzrd	2020-06-13	-0/+56
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \| \|	Incidents tests: move `mock_404` into module namespace	kwzrd	2020-06-13	-5/+6
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This will be useful for others tests as well.
\| \| \| * \| \| \| \| \| \| \| \|	Incidents tests: write tests for `archive`	kwzrd	2020-06-13	-2/+63
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \| \|	Incidents tests: set up base class for `Incidents`	kwzrd	2020-06-13	-2/+21
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	For cleanliness, I've decided to make a separate class for each method. Since most tests will want to have an `Incident` instance ready, they can inherit the `setUp` from `TestIncidents`, which does not make any assertions on its own.
\| \| \| * \| \| \| \| \| \| \| \|	Incidents tests: improve mock `Signal` name & move def	kwzrd	2020-06-13	-6/+6
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Let's make it clear that this is our own mock. We also move the definition to the top of the module.
\| \| \| * \| \| \| \| \| \| \| \|	Incidents tests: use our own helper mocks	kwzrd	2020-06-13	-16/+14
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	No reason to build own MagicMocks as we already have helpers that more accurately mimic the mocked behaviour.
\| \| \| * \| \| \| \| \| \| \| \|	Incidents tests: write tests for module-level helpers	kwzrd	2020-06-12	-0/+135
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \| \|	Incidents tests: create new test module	kwzrd	2020-06-12	-0/+0
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \| \|	Jam Tests: space out lines for readability	MarkKoz	2020-07-22	-0/+6
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \| \|	Jam Tests: remove default_args attribute	MarkKoz	2020-07-22	-4/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Kind of redundant since it's only used by two tests.
\| \| * \| \| \| \| \| \| \| \| \|	Jam Tests: re-arrange tests to follow definition order in the cog	MarkKoz	2020-07-22	-10/+10
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \| \|	Jam Tests: fix utils patch	MarkKoz	2020-07-22	-4/+5
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	stop needs to be called on the patcher, not the mock. Furthermore, using addCleanup is safer than tearDown because the latter may not be called if an exception is raised in setUp.
\| \| * \| \| \| \| \| \| \| \| \|	Jam Tests: Fix `test_duplicate_member_provided` assertions	ks129	2020-06-13	-1/+3
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \| \|	Jam Tests: Fix wrong function name and convert them to mocks	ks129	2020-06-13	-1/+3
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \| \|	Jam Tests: Simplify and make tests more secure	ks129	2020-06-13	-6/+4
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \| \|	Jam Tests: Update `Context` to `Guild` for tests too	ks129	2020-06-13	-19/+19
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \| \|	Jam Tests: Apply recent command splitting to `test_jam_roles_adding`	ks129	2020-06-11	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \| \|	Jam Tests: Remove unnecessary `Context` mock resets	ks129	2020-06-11	-2/+0
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \| \|	Jam Tests: Implement default arguments	ks129	2020-06-11	-4/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	To avoid repeating same arguments, added default arguments that is unpacked on function call.
\| \| * \| \| \| \| \| \| \| \| \|	Jam Tests: Apply recent changes to `test_category_dont_exist`	ks129	2020-06-11	-2/+1
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \| \|	Jam Tests: Simplify and update `test_category_channel_exist`	ks129	2020-06-11	-3/+1
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \| \|	Jam Tests: Merge text and voice channel creation tests	ks129	2020-06-11	-21/+24
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \| \|	Jam Tests: Apply recent changes to overwrites test	ks129	2020-06-11	-2/+1
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \| \|	Jam Tests: Add more assertions to result message sending test	ks129	2020-06-11	-1/+7
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \| \|	Jam Tests: Make early exiting test more secure	ks129	2020-06-11	-2/+6
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \| \|	Jam Tests: Use class member of patch instead decorator on most of tests	ks129	2020-06-11	-21/+19
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \| \|	Jam Tests: Fix `test_result_sending` docstring	ks129	2020-06-11	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-authored-by: Mark <[email protected]>
\| \| * \| \| \| \| \| \| \| \| \|	Jam Tests: Create test for `setup` function	ks129	2020-06-11	-1/+11
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \| \|	Jam Tests: Create test for successful `ctx.send` calling	ks129	2020-06-11	-0/+6
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \| \|	Jam Tests: Create test for team jam roles adding	ks129	2020-06-11	-0/+14
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \| \|	Jam Tests: Create test for team voice channel creating	ks129	2020-06-11	-0/+12
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \| \|	Jam Tests: Make text channel creation test more specific	ks129	2020-06-11	-1/+8
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \| \|	Jam Tests: Created test for channel overwrites	ks129	2020-06-11	-0/+24
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \| \|	Jam Tests: Created test for creating text channel for team	ks129	2020-06-11	-0/+5
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \| \|	Jam Tests: Created test that make sure when category exist, don't create	ks129	2020-06-11	-0/+8
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \| \|	Jam Tests: Added utils.get assert to category creating test	ks129	2020-06-11	-0/+1
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \| \|	Jam Tests: Created test for category creating when not exist	ks129	2020-06-11	-2/+14
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \| \|	Jam Tests: Created tests for removing duplicate team members	ks129	2020-06-11	-0/+9
\| \| \| \| \| \| \| \| \| \| \| \|