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

	Commit message (Collapse)	Author	Age	Lines
...
\| \| \| \| * \| \| \| \|	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 help_channel_rediscache	Leon Sandøy	2020-06-06	-9/+8
\| \| \|\ \ \ \ \ \ \
\| \| * \| \| \| \| \| \| \|	Add support for bool values in RedisCache	Leon Sandøy	2020-05-31	-1/+3
\| \| \| \|_\|/ / / / / \| \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	We're gonna need this for the help channel handling, and it seems like a reasonable type to support anyway. It requires a tiny bit of special handling, but nothing outrageous.
\| * \| \| \| \| \| \| \|	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
\| \| \|_\|/ / / / / \| \|/\| \| \| \| \| \|
\| * \| \| \| \| \| \|	Merge branch 'master' into bad-words-nick	ks129	2020-06-11	-60/+317
\| \|\ \ \ \ \ \ \
\| * \| \| \| \| \| \| \|	Remove redis namespace collision test	MarkKoz	2020-05-31	-10/+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
\| \| \| \| \| \| \| \| \|