aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* EH Tests: Merge test classesGravatar ks1292020-05-19-11/+4
|
* EH Tests: Create test for `handle_check_failure`Gravatar ks1292020-05-19-0/+48
|
* EH Tests: Create test for `handle_input_error`Gravatar ks1292020-05-19-6/+36
|
* Error Handler: Changed way of help command get + send to avoid warningGravatar ks1292020-05-19-6/+4
| | | | Only get coroutine when this is gonna be awaited.
* EH Tests: Created test for `handle_user_input_error` `get_help_command`Gravatar ks1292020-05-19-0/+16
|
* EH Tests: Added test for `try_get_tag` `ResponseCodeError` ignoreGravatar ks1292020-05-19-0/+6
|
* EH Tests: Added test for `try_get_tag` `ctx.invoke` callingGravatar ks1292020-05-19-0/+7
|
* EH Tests: Added test for `try_get_tag` tag name converting failingGravatar ks1292020-05-19-0/+9
|
* EH Tests: Added test for `try_get_tag` successful tag name convertingGravatar ks1292020-05-19-0/+9
|
* EH Tests: Added test for `try_get_tag` error handlingGravatar ks1292020-05-19-2/+11
|
* EH Tests: Added test for `try_get_tag` checks failGravatar ks1292020-05-19-2/+8
|
* EH Tests: Added test for `try_get_tag` `invoked_from_error_handler`Gravatar ks1292020-05-19-0/+7
|
* EH Tests: Added test for `try_get_tag` `get_command` callingGravatar ks1292020-05-19-0/+19
|
* EH Tests: Added tests for `get_help_command`in `OtherErrorHandlerTests`Gravatar ks1292020-05-19-0/+32
|
* EH Tests: Cleanup `try_silence` testsGravatar ks1292020-05-18-39/+27
|
* EH Tests: Added test for `try_silence` no match messageGravatar ks1292020-05-18-0/+6
|
* EH Tests: Created `try_silence` test to test unsilence command callingGravatar ks1292020-05-18-0/+17
|
* EH Tests: Created `try_silence` test to test silence command callingGravatar ks1292020-05-18-0/+17
|
* EH Tests: Created `try_silence` test to check no permission with exceptGravatar ks1292020-05-18-0/+8
|
* EH Tests: Created `try_silence` test to check no permission responseGravatar ks1292020-05-18-2/+9
|
* EH Tests: Created `try_silence` test for `get_command`Gravatar ks1292020-05-18-0/+9
|
* EH Tests: Created first test for `try_silence`Gravatar ks1292020-05-18-0/+18
|
* EH Tests: Created test for all other errorsGravatar ks1292020-05-18-0/+8
|
* EH Tests: Added test for 3 errors handling on handlerGravatar ks1292020-05-18-1/+17
| | | | | | | These 3 errors is: - `ConversionError` - `MaxConcurrencyReached` - `ExtensionError`
* EH Tests: Added test for `CommandInvokeError` handling on handlerGravatar ks1292020-05-18-0/+22
|
* EH Tests: Added test for `CommandOnCooldown` handling on handlerGravatar ks1292020-05-18-0/+8
|
* EH Tests: Added test for `CheckFailure` handling on handlerGravatar ks1292020-05-18-0/+9
|
* EH Tests: Added test for `UserInputError` handling on handlerGravatar ks1292020-05-18-0/+9
|
* EH Tests: Added another test for `CommandNotFound` error handlingGravatar ks1292020-05-18-0/+17
| | | | | Added test for case when `Context.invoked_from_error_handler` is `True`
* Test Helpers: Added new attribute to `MockContext`Gravatar ks1292020-05-18-0/+2
| | | | Added `invoked_from_error_handler` attribute that is `False` default.
* Error Handler: Changed `CommandNotFound` error check for testingGravatar ks1292020-05-18-1/+1
| | | | Replaced `hasattr` with `getattr` for unit tests
* EH Tests: Remove class member `cog` + other small changesGravatar ks1292020-05-18-14/+15
| | | | | - Added `assertIsNone` to `test_error_handler_already_handled`. - Removed `ErrorHandlerTests.cog`, moved it to each test recreation.
* EH Tests: Created test for `CommandNotFound` error when call isn't by EHGravatar ks1292020-05-18-0/+49
|
* EH Tests: Created test cases set + already handled error testGravatar ks1292020-05-18-0/+22
| | | | | | Created test that make sure when error is already handled in local error handler, this don't await `ctx.send` to make sure that this don't move forward.
* Merge pull request #858 from python-discord/decorator-factory-mutability-tagGravatar kwzrd2020-05-17-0/+37
|\ | | | | Add mutability.md tag
| * Merge branch 'master' into decorator-factory-mutability-tagGravatar kwzrd2020-05-17-890/+2365
| |\ | |/ |/|
* | Merge pull request #949 from python-discord/help-command-fix-invocationGravatar kwzrd2020-05-17-41/+33
|\ \ | | | | | | Use `send_help` to ensure that our help command is correctly invoked
| * | Use `Command`-object for `send_help`Gravatar Sebastiaan Zeeff2020-05-17-21/+21
| | | | | | | | | | | | | | | | | | | | | | | | As @mathsman5133 pointed out, it's better to use the `Command`-instance we typically already have in the current context than to rely on parsing the qualified name again. The invocation is now done as: `await ctx.send_help(ctx.command)`
| * | Use `send_help` to invoke command helpGravatar Sebastiaan Zeeff2020-05-17-40/+32
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | After the refactoring of the help command, we need to use the built-in method of calling the help command: `Context.send_help`. As an argument, the qualified name (a string containing the full command path, including parents) of the command can be passed. Examples: - await ctx.send_help("reminders edit") This would send a help embed with information on `!reminders edit` to the Context. - await ctx.send_help(ctx.command.qualified_name) This would extract the qualified name of the command, which is the full command path, and send a help embed to Context. - await ctx.send_help() This will send the main "root" help embed to the Context.
* | Merge pull request #519 from mathsman5133/help-refactorGravatar Sebastiaan Zeeff2020-05-17-460/+269
|\ \ | | | | | | Refactor the !help command.
| * \ Merge branch 'master' into help-refactorGravatar Sebastiaan Zeeff2020-05-17-1/+1
| |\ \ | |/ / |/| |
* | | Merge pull request #946 from vivax3794/masterGravatar Leon Sandøy2020-05-17-1/+1
|\ \ \ | | | | | | | | add "solved" as a alias for "closed"
| * | | added "solved" as a alias for "closed"Gravatar vivax37942020-05-17-1/+1
|/ / /
| * | fix redirect_output decorator; remove ninja codeGravatar mathsman51332020-05-16-76/+66
| | | | | | | | | | | | | | | | | | - Lots of instance of `for c in ...` or `for a in ...` or `fmt` which are non-descriptive and sometimes cryptic. - Ves suggested running the command in an asyncio task for `@redirect_output`, rather than making a workaround which only applies to the help command. This fixes a fundamental flaw where the redirection message wouldn't be deleted until a further 60sec after the command has finished, which for `!help` could be up to 5min, meaning the invocation message could be sitting there for 6min, not the intended 60sec.
| * | Merge branch 'master' into help-refactorGravatar Leon Sandøy2020-05-15-447/+2127
| |\ \ | |/ / |/| |
* | | Remove @Admins ping from the #verification messageGravatar Leon Sandøy2020-05-14-1/+1
| | | | | | | | | This probably isn't necessary anymore. We get so many new users that someone is going to DM us very soon when something breaks. We've outgrown this, and it just adds noise to the #verification channel in the form of pings.
* | | Remove everyone-ping from mentions alertGravatar Sebastiaan Zeeff2020-05-14-1/+0
| | | | | | | | | The mentions alert that is sent out by the Verification cog currently pings `@everyone` despite being quite unactionable by most people receiving the ping. As it happens frequently, especially with the recent uptick in joins, I'm removing that ping to not bother our moderators as much.
* | | Merge pull request #899 from ks129/python-newsGravatar Sebastiaan Zeeff2020-05-13-55/+321
|\ \ \ | | | | | | | | Python News implemention
| * \ \ Merge branch 'master' into python-newsGravatar Sebastiaan Zeeff2020-05-13-219/+522
| |\ \ \ | |/ / / |/| | |
* | | | Expand guild whitelistGravatar S. Co12020-05-12-1/+7
| | | |